diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-09 10:04:44 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-10 10:17:38 -0400 |
| commit | bb1487fef5d19a7b1536fc42e25a08f7d4c43e1b (patch) | |
| tree | eeee7524c49b3f3dcb0cdfa5f8c6758a985e6bb3 /src/monitor | |
| parent | a40b812b63e9f32c946761cb9cb7315ee57376ed (diff) | |
| download | sssd-bb1487fef5d19a7b1536fc42e25a08f7d4c43e1b.tar.gz sssd-bb1487fef5d19a7b1536fc42e25a08f7d4c43e1b.tar.xz sssd-bb1487fef5d19a7b1536fc42e25a08f7d4c43e1b.zip | |
Check the correct variable for NULL after creating timer
In several places, we were creating a new timer and assigning it
to the tev variable, but then we were checking for NULL from the
te variable (which, incidentally, is guaranteed never to be NULL
in this situation)
https://fedorahosted.org/sssd/ticket/523
Diffstat (limited to 'src/monitor')
| -rw-r--r-- | src/monitor/monitor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c index e52d79349..bda45dc47 100644 --- a/src/monitor/monitor.c +++ b/src/monitor/monitor.c @@ -1435,7 +1435,7 @@ static void process_config_file(struct tevent_context *ev, rw_ctx->file_ctx = file_ctx; tev = tevent_add_timer(ev, rw_ctx, tv, rewatch_config_file, rw_ctx); - if (te == NULL) { + if (tev == NULL) { DEBUG(0, ("Could not restore inotify watch. Quitting!\n")); close(file_ctx->mt_ctx->inotify_fd); kill(getpid(), SIGTERM); @@ -1488,7 +1488,7 @@ static void rewatch_config_file(struct tevent_context *ev, cb->filename, err, strerror(err))); tev = tevent_add_timer(ev, ev, tv, rewatch_config_file, rw_ctx); - if (te == NULL) { + if (tev == NULL) { DEBUG(0, ("Could not restore inotify watch. Quitting!\n")); close(file_ctx->mt_ctx->inotify_fd); kill(getpid(), SIGTERM); |