libextractor

commit a666aaa2851d695efef37bc431645522a6d8e7f9
parent a3f6574cddae1374561a05280a6277422532fe0c
Author: Christian Grothoff <christian@grothoff.org>
Date:   Sun, 15 Oct 2017 19:41:43 +0200

fix potential assign-after-free

Diffstat:
M
ChangeLog
 | 
3
++-
M
src/main/extractor_ipc_gnu.c
 | 
2
++

2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/ChangeLog b/ChangeLog
@@ -1,5 +1,6 @@
 Sun Oct 15 19:36:41 CEST 2017
-	Fix potential file descriptor leak (on error handling path). -CG
+	Fix potential file descriptor leak (on error handling path).
+	Fix potential assign-after-free (on IPC error handling path). -CG
 
 Fri Oct 13 12:30:37 CEST 2017
 	Properly check read error in NSF plugin (from signedness confusion) found by Leon Zhao. -CG
diff --git a/src/main/extractor_ipc_gnu.c b/src/main/extractor_ipc_gnu.c
@@ -545,6 +545,7 @@ EXTRACTOR_IPC_channel_recv_ (struct EXTRACTOR_Channel **channels,
 	      LOG_STRERROR ("realloc");
 	      EXTRACTOR_IPC_channel_destroy_ (channel);
 	      channels[i] = NULL;
+              continue;
 	    }
 	  channel->mdata = ndata;
 	}
@@ -563,6 +564,7 @@ EXTRACTOR_IPC_channel_recv_ (struct EXTRACTOR_Channel **channels,
                channel->plugin->libname);
 	  EXTRACTOR_IPC_channel_destroy_ (channel);
 	  channels[i] = NULL;
+          continue;
 	}
       else
 	{