hook_fn casts were hiding UB (#265)

[solanum.git] / extensions / no_locops.c

diff --git a/extensions/no_locops.c b/extensions/no_locops.c
index 7fc7d1b6ac75e8050a490ae63d40c506b8c4fb44..077a4653e302fefd901b40ca30484f72defe0e5c 100644 (file)
--- a/extensions/no_locops.c
+++ b/extensions/no_locops.c
@@ -14,18 +14,19 @@
 
 static const char no_locops_desc[] = "Disables local operators";
 
-static void h_nl_umode_changed(hook_data_umode_changed *);
+static void h_nl_umode_changed(void *);
 
 mapi_hfn_list_av1 nl_hfnlist[] = {
-       { "umode_changed", (hookfn) h_nl_umode_changed },
+       { "umode_changed", h_nl_umode_changed },
        { NULL, NULL }
 };
 
 DECLARE_MODULE_AV2(no_locops, NULL, NULL, NULL, NULL, nl_hfnlist, NULL, NULL, no_locops_desc);
 
 static void
-h_nl_umode_changed(hook_data_umode_changed *hdata)
+h_nl_umode_changed(void *data)
 {
+       hook_data_umode_changed *hdata = data;
        struct Client *source_p = hdata->client;
 
        if (MyClient(source_p) && source_p->umodes & UMODE_LOCOPS)