1
80%
master: 80%

Ran 26 Sep 2020 11:58PM UTC

Files 7

Run time 0s

Badge

Embed ▾

Committed 26 Sep 2020 11:57PM UTC coverage: 80.374% (+0.09%) from 80.282%

Job # 2.3

Build Type

push

travis-ci

Committed by

ioquatix

Commit Message

Prevent race condition in Notifier#run / #stop

There is a potential race condition when a notifier is run in a new
thread and then immediately stopped in the original thread.  If
`Notifier#stop` sets `@stop = true` before `Notifier#run` sets
`@stop = false`, then `Notifier#run` will loop until `Notifier#stop` is
called again.  Furthermore, if `Notifier#run` acquires the `@running`
mutex before `Notifier#stop` does (but after `Notifier#stop` sets
`@stop = true`), then `Notifier#stop` will get stuck waiting for the
mutex while `Notifier#run` infinitely loops.

This commit modifies `Notifier#run` to assume `@stop == false` when it
begins, and to reset `@stop = false` only after its loop terminates,
thus eliminating the race.

Run Details

172 of 214 relevant lines covered (80.37%)

9.26 hits per line

guard / rb-inotify / 83 / 1
80%
master: 80%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 83.1 (2.3)

guard / rb-inotify / 83 / 1 80% master: 80%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 83.1 (2.3)

guard / rb-inotify / 83 / 1
80%
master: 80%

README BADGES
x