[pisa-src] r1516 - trunk/community-operator/co_server.c

  • From: Jan Marten <jan.marten@xxxxxxxxxxxxxx>
  • To: pisa-src@xxxxxxxxxxxxx
  • Date: Wed, 04 Nov 2009 15:46:46 +0100

Author: marten
Date: Wed Nov  4 15:46:45 2009
New Revision: 1516

Log:
Fixed a memory leak in co_server.c and
moved the successful default case out of the else brackets.

Modified:
   trunk/community-operator/co_server.c

Modified: trunk/community-operator/co_server.c
==============================================================================
--- trunk/community-operator/co_server.c        Wed Nov  4 15:00:57 2009        
(r1515)
+++ trunk/community-operator/co_server.c        Wed Nov  4 15:46:45 2009        
(r1516)
@@ -205,20 +205,22 @@
        if (sent == -1) {
                PISA_ERROR("sendto failed.\n");
                free(cert);
+               free(packet);
                return 0;
        }
 
        if (sent != sizeof(co_packet)) {
                PISA_ERROR("sent bytes (%i) do not match expected value 
(%i).\n", sent,
                                sizeof(co_packet));
+               free(cert);
+               free(packet);
                return 0;
-       } else {
-               PISA_INFO("the request has been answered.\n");
-               return 1;
        }
 
+       PISA_INFO("the request has been answered.\n");
        free(cert);
        free(packet);
+       return 1;
 }
 
 int main(void)

Other related posts:

  • » [pisa-src] r1516 - trunk/community-operator/co_server.c - Jan Marten