Various small fixes, part #2.

git-svn-id: http://anope.svn.sourceforge.net/svnroot/anope/trunk@1670 5417fbe8-f217-4b02-8779-1006273d7864

src/core/bs_info.c

@@ -262,7 +262,7 @@ void send_bot_channels(User * u, BotInfo * bi)
     }
 
     if (*buf)
-        notice_user(s_BotServ, u, buf);
+        notice_user(s_BotServ, u, "%s", buf);
     return;
 }
 

src/events.c

@@ -512,12 +512,12 @@ int Module::AddEventHook(EvtHook *evh)
 	return status;
 }
 
-int Module::DelEventHandler(const char *name)
+int Module::DelEventHandler(const char *sname)
 {
     EvtMessage *evm;
     int status;
 
-    evm = findEventHandler(EVENT, name);
+    evm = findEventHandler(EVENT, sname);
     if (!evm) {
         return MOD_ERR_NOEXIST;
     }
@@ -529,12 +529,12 @@ int Module::DelEventHandler(const char *name)
     return status;
 }
 
-int Module::DelEventHook(const char *name)
+int Module::DelEventHook(const char *sname)
 {
     EvtHook *evh;
     int status;
 
-    evh = findEventHook(EVENTHOOKS, name);
+    evh = findEventHook(EVENTHOOKS, sname);
     if (!evh) {
         return MOD_ERR_NOEXIST;
     }

src/helpserv.c

@@ -50,7 +50,7 @@ void helpserv_init(void)
  */
 void helpserv(User * u, char *buf)
 {
-    char *cmd, *s;
+    const char *cmd, *s;
 
     cmd = strtok(buf, " ");
 

src/hostserv.c

@@ -46,7 +46,6 @@ void moduleAddHostServCmds(void)
 void get_hostserv_stats(long *nrec, long *memuse)
 {
     long count = 0, mem = 0;
-    int i;
     HostCore *hc;
 
     for (hc = head; hc; hc = hc->next) {
@@ -89,7 +88,7 @@ void hostserv_init(void)
  */
 void hostserv(User * u, char *buf)
 {
-    char *cmd, *s;
+    const char *cmd, *s;
 
     cmd = strtok(buf, " ");
 
@@ -177,12 +176,12 @@ HostCore *createHostCorelist(HostCore * next, char *nick, char *vIdent,
  * @param found If found
  * @return HostCore
  */
-HostCore *findHostCore(HostCore * head, char *nick, bool* found)
+HostCore *findHostCore(HostCore * phead, char *nick, bool* found)
 {
     HostCore *previous, *current;
 
     *found = false;
-    current = head;
+    current = phead;
     previous = current;
 
     if (!nick) {
@@ -201,7 +200,7 @@ HostCore *findHostCore(HostCore * head, char *nick, bool* found)
             current = current->next;
         }
     }
-    if (current == head) {
+    if (current == phead) {
         return NULL;
     } else {
         return previous;
@@ -209,7 +208,7 @@ HostCore *findHostCore(HostCore * head, char *nick, bool* found)
 }
 
 /*************************************************************************/
-HostCore *insertHostCore(HostCore * head, HostCore * prev, char *nick,
+HostCore *insertHostCore(HostCore * phead, HostCore * prev, char *nick,
                          char *vIdent, char *vHost, const char *creator,
                          int32 tmp_time)
 {
@@ -252,8 +251,8 @@ HostCore *insertHostCore(HostCore * head, HostCore * prev, char *nick,
         }
         newCore->time = tmp_time;
         if (prev == NULL) {
-            tmp = head;
-            head = newCore;
+            tmp = phead;
+            phead = newCore;
             newCore->next = tmp;
         } else {
             tmp = prev->next;
@@ -261,18 +260,18 @@ HostCore *insertHostCore(HostCore * head, HostCore * prev, char *nick,
             newCore->next = tmp;
         }
     }
-    return head;
+    return phead;
 }
 
 /*************************************************************************/
-HostCore *deleteHostCore(HostCore * head, HostCore * prev)
+HostCore *deleteHostCore(HostCore * phead, HostCore * prev)
 {
 
     HostCore *tmp;
 
     if (prev == NULL) {
-        tmp = head;
-        head = head->next;
+        tmp = phead;
+        phead = phead->next;
     } else {
         tmp = prev->next;
         prev->next = tmp->next;
@@ -284,7 +283,7 @@ HostCore *deleteHostCore(HostCore * head, HostCore * prev)
         free(tmp->vIdent);
     }
     free(tmp);
-    return head;
+    return phead;
 }
 
 /*************************************************************************/

src/memoserv.c

@@ -54,7 +54,7 @@ void ms_init(void)
  */
 void memoserv(User * u, char *buf)
 {
-    char *cmd, *s;
+    const char *cmd, *s;
 
     cmd = strtok(buf, " ");
     if (!cmd) {
@@ -380,7 +380,7 @@ static void new_memo_mail(NickCore * nc, Memo * m)
     fprintf(mail->pipe, getstring2(NULL, MEMO_MAIL_TEXT2), m->sender,
             m->number);
     fprintf(mail->pipe, "\n\n");
-    fprintf(mail->pipe, getstring2(NULL, MEMO_MAIL_TEXT3));
+    fprintf(mail->pipe, "%s", getstring2(NULL, MEMO_MAIL_TEXT3));
     fprintf(mail->pipe, "\n\n");
     fprintf(mail->pipe, "%s", m->text);
     fprintf(mail->pipe, "\n");
@@ -423,7 +423,7 @@ void rsend_notify(User * u, Memo * m, const char *chan)
             sprintf(text, fmt, chan);
         } else {
             fmt = getstring(na, MEMO_RSEND_NICK_MEMO_TEXT);
-            sprintf(text, fmt);
+            sprintf(text, "%s", fmt);
         }
 
         /* Send notification */

src/news.c

@@ -330,7 +330,7 @@ int do_randomnews(User * u)
 void do_news(User * u, short type)
 {
     int is_servadmin = is_services_admin(u);
-    char *cmd = strtok(NULL, " ");
+    const char *cmd = strtok(NULL, " ");
     const char *type_name;
     int *msgs;
 

src/nickserv.c

@@ -134,7 +134,7 @@ void ns_init(void)
 
 void nickserv(User * u, char *buf)
 {
-    char *cmd, *s;
+    const char *cmd, *s;
 
     cmd = strtok(buf, " ");
 

src/operserv.c

@@ -116,8 +116,8 @@ void os_init(void)
 
 void operserv(User * u, char *buf)
 {
-    char *cmd;
-    char *s;
+    const char *cmd;
+    const char *s;
 
     alog("%s: %s: %s", s_OperServ, u->nick, buf);
 
@@ -308,7 +308,6 @@ void load_os_dbase(void)
     }
 
     if (ver >= 7) {
-        uint32 tmp32;
         SAFE(read_int32(&maxusercnt, f));
         SAFE(read_int32(&tmp32, f));
         maxusertime = tmp32;