Changes in ipc.c [edc767b:171ef85]

File:

• ipc.c (modified) (5 diffs)

ipc.c

@@ -209 +209 @@
 }
 
-/* Return just one line. Returns NULL if something broke, an empty string
-   on temporary "errors" (EAGAIN and friends). */
 static char *ipc_readline( int fd )
 {
-        char buf[513], *eol;
+        char *buf, *eol;
         int size;
+        
+        buf = g_new0( char, 513 );
         
         /* Because this is internal communication, it should be pretty safe
@@ -221 +221 @@
            sockets and limites message length, messages should always be
            complete. Saves us quite a lot of code and buffering. */
-        size = recv( fd, buf, sizeof( buf ) - 1, MSG_PEEK );
+        size = recv( fd, buf, 512, MSG_PEEK );
         if( size == 0 || ( size < 0 && !sockerr_again() ) )
                 return NULL;
@@ -229 +229 @@
                 buf[size] = 0;
         
-        if( ( eol = strstr( buf, "\r\n" ) ) == NULL )
+        eol = strstr( buf, "\r\n" );
+        if( eol == NULL )
                 return NULL;
         else
                 size = eol - buf + 2;
+        
+        g_free( buf );
+        buf = g_new0( char, size + 1 );
         
         if( recv( fd, buf, size, 0 ) != size )
                 return NULL;
         else
-                return g_strndup( buf, size - 2 );
+                buf[size-2] = 0;
+        
+        return buf;
 }
 
@@ -248 +254 @@
                 cmd = irc_parse_line( buf );
                 if( cmd )
-                {
                         ipc_command_exec( data, cmd, ipc_master_commands );
-                        g_free( cmd );
-                }
-                g_free( buf );
         }
         else
@@ -270 +272 @@
                 cmd = irc_parse_line( buf );
                 if( cmd )
-                {
                         ipc_command_exec( data, cmd, ipc_child_commands );
-                        g_free( cmd );
-                }
-                g_free( buf );
         }
         else