Parcourir la source

Fix: Memory leak due to calling GetErrorMessage() twice by accident

GetErrorMessage() was accidently called twice during error handling but
from the created error messages only one could be freed later.

Signed-off-by: Stefan Mätje <stefan.maetje@esd.eu>
Stefan Mätje il y a 6 ans
Parent
commit
3e285b30fc
1 fichiers modifiés avec 2 ajouts et 3 suppressions
  1. 2 3
      source/src/ports/generic_networkhandler.c

+ 2 - 3
source/src/ports/generic_networkhandler.c

@@ -192,9 +192,8 @@ EipStatus NetworkHandlerInitialize(void) {
                sizeof(struct sockaddr) ) ) == -1 ) {
     int error_code = GetSocketErrorNumber();
     char *error_message = GetErrorMessage(error_code);
-    OPENER_TRACE_ERR( "error with UDP unicast bind: %d - %s\n", error_code,
-                      GetErrorMessage(
-                        error_code) );
+    OPENER_TRACE_ERR( "error with UDP unicast bind: %d - %s\n",
+                     error_code, error_message);
     FreeErrorMessage(error_message);
     return kEipStatusError;
   }