Skip to content

Commit

Permalink
sdbus: fix watcher is_active() callback
Browse files Browse the repository at this point in the history
Problem: the internal sdbus watcher is_active() callback calls
ev_is_active() on a flux_watcher_t, which is accepted because
ev_is_active() is a macro that casts its pointer argument.

Although flux_watcher_is_active() is not currently used in the
sdbus module (the only user of the watcher), fix the code in
case it ever is.
  • Loading branch information
garlick committed Dec 11, 2024
1 parent f49dbda commit bf55981
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/modules/sdbus/watcher.c
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ static void op_stop (flux_watcher_t *w)
static bool op_is_active (flux_watcher_t *w)
{
struct sdbus_watcher *sdw = watcher_get_data (w);
return ev_is_active (sdw->prep);
return flux_watcher_is_active (sdw->prep);
}

static void op_destroy (flux_watcher_t *w)
Expand Down

0 comments on commit bf55981

Please sign in to comment.