providers/ident: fix some nasty crashes

[solanum.git] / modules / m_userhost.c

diff --git a/modules/m_userhost.c b/modules/m_userhost.c
index e39866fed674b309a7b0e030cb68337186df1dea..76e6cd17e7d60bcfdb711bdcdbc8eaee261c1b69 100644 (file)
--- a/modules/m_userhost.c
+++ b/modules/m_userhost.c
@@ -34,18 +34,20 @@
 #include "modules.h"
 #include "s_conf.h"
 
-static char buf[BUFSIZE];
-
-static int m_userhost(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
 static const char userhost_desc[] =
        "Provides the USERHOST command to show a user's host";
 
+static char buf[BUFSIZE];
+
+static void m_userhost(struct MsgBuf *, struct Client *, struct Client *, int, const char **);
+
 struct Message userhost_msgtab = {
        "USERHOST", 0, 0, 0, 0,
        {mg_unreg, {m_userhost, 2}, mg_ignore, mg_ignore, mg_ignore, {m_userhost, 2}}
 };
 
 mapi_clist_av1 userhost_clist[] = { &userhost_msgtab, NULL };
+
 DECLARE_MODULE_AV2(userhost, NULL, NULL, userhost_clist, NULL, NULL, NULL, NULL, userhost_desc);
 
 /*
@@ -53,7 +55,7 @@ DECLARE_MODULE_AV2(userhost, NULL, NULL, userhost_clist, NULL, NULL, NULL, NULL,
  * the need for complicated requests like WHOIS. It returns user/host
  * information only (no spurious AWAY labels or channels).
  */
-static int
+static void
 m_userhost(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *source_p, int parc, const char *parv[])
 {
        struct Client *target_p;
@@ -109,6 +111,4 @@ m_userhost(struct MsgBuf *msgbuf_p, struct Client *client_p, struct Client *sour
        }
 
        sendto_one(source_p, "%s", buf);
-
-       return 0;
 }