BUILD : 1.7.9 (793) BUGS : 378 NOTES : Applied patch

git-svn-id: svn://svn.anope.org/anope/trunk@793 31f1291d-b8d6-0310-a050-a5561fc1590b


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

Changes

@@ -21,6 +21,9 @@ Provided by crazytoon <crazy@anope.org> - 2005
 05/08 F Removed "IRCDFILE" from the Makefile's                            [ #00]
 05/10 F Removed BOT LIST from BOT_BOT_SYNTAX                              [ #00]
 
+Provided by Sascha Just <bugs.anope@nmg.own-hero.de> - 2005
+05/13 F Nicer used of fgets, dont free the buffer.                        [#378]
+
 Anope Version 1.7.9
 -------------------
 Provided by Anope Dev. <dev@anope.org> - 2005

src/modules/ns_noop.c

@@ -257,8 +257,7 @@ int mLoadData(void)
         alog("ns_noop: WARNING: Can not open database file! (it might not exist, this is not fatal)");
         ret = 1;
     } else {
-        while (!feof(in)) {
-            fgets(buffer, 1500, in);
+        while (fgets(buffer, 1500, in)) {
             name = myStrGetToken(buffer, ' ', 0);
             if (name) {
                 len = strlen(name);

src/modules/os_info.c

@@ -188,7 +188,6 @@ int myAddNickInfo(User * u)
         } else {
             moduleNoticeLang(s_NickServ, u, OINFO_SYNTAX);
         }
-        free(text);
     }
     return MOD_CONT;
 }
@@ -257,7 +256,6 @@ int myAddChanInfo(User * u)
         } else {
             moduleNoticeLang(s_ChanServ, u, OCINFO_SYNTAX);
         }
-        free(text);
     }
     return MOD_CONT;
 }
@@ -296,7 +294,6 @@ int myNickInfo(User * u)
                 }
                 free(nick);
             }
-            free(text);
         }
     }
     return MOD_CONT;
@@ -330,7 +327,6 @@ int myChanInfo(User * u)
                 }
                 free(chan);
             }
-            free(text);
         }
     }
     return MOD_CONT;
@@ -361,8 +357,7 @@ int mLoadData(void)
         alog("os_info: WARNING: can not open the database file! (it might not exist, this is not fatal)");
         ret = 1;
     } else {
-        while (!feof(in)) {
-            fgets(buffer, 1500, in);
+        while (fgets(buffer, 1500, in)) {
             type = myStrGetToken(buffer, ' ', 0);
             name = myStrGetToken(buffer, ' ', 1);
             info = myStrGetTokenRemainder(buffer, ' ', 2);

version.log

@@ -8,10 +8,14 @@
 VERSION_MAJOR="1"
 VERSION_MINOR="7"
 VERSION_PATCH="9"
-VERSION_BUILD="792"
+VERSION_BUILD="793"
 
 # $Log$
 #
+# BUILD : 1.7.9 (793)
+# BUGS  : 378
+# NOTES : Applied patch
+#
 # BUILD : 1.7.9 (792)
 # BUGS  : N/A
 # NOTES : Added a check for last_quit when infoing a suspended nick, this should never happen, but it seems a few networks have odd databases which are using the suspend flag for something else, i think its related to networks that have merged databases, but i dont know, anyway, we check for it before using it now. (this dosnt fix the fact they are using a nc->flag that core wants for something else)