[Nmh-commits] nmh/sbr client.c

[Ken Hornstein]

CVSROOT:        /sources/nmh
Module name:    nmh
Changes by:     Ken Hornstein <kenh>    09/01/16 15:48:10

Modified files:
        sbr            : client.c 

Log message:
        Move freeaddrinfo() to inside of our loop; this prevents a memory leak
        and also prevents it from being called with a NULL pointer.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/nmh/sbr/client.c?cvsroot=nmh&r1=1.4&r2=1.5

Patches:
Index: client.c
===================================================================
RCS file: /sources/nmh/nmh/sbr/client.c,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -b -r1.4 -r1.5
--- client.c    16 Jan 2009 02:28:55 -0000      1.4
+++ client.c    16 Jan 2009 15:48:09 -0000      1.5
@@ -2,7 +2,7 @@
 /*
  * client.c -- connect to a server
  *
- * $Id: client.c,v 1.4 2009/01/16 02:28:55 kenh Exp $
+ * $Id: client.c,v 1.5 2009/01/16 15:48:09 kenh Exp $
  *
  * This code is Copyright (c) 2002, by the authors of nmh.  See the
  * COPYRIGHT file in the root directory of the nmh distribution for
@@ -111,9 +111,10 @@
 
            close(sd);
        }
-    }
 
     freeaddrinfo(res);
+    }
+
     client_freelist(ap);
     strncpy (response, "no servers available", len_response);
     return NOTOK;