diff options
author | Michael Mann <mmann78@netscape.net> | 2017-12-27 21:08:26 -0500 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-12-28 04:16:43 +0000 |
commit | 5c52ab7e45241af5c936ac391cac386d224f2e86 (patch) | |
tree | d59b56c52e46004fa2c63337a027612da5720105 /extcap_spawn.c | |
parent | 22f17f918c20c6934edfdffd96942e45c0b2e28b (diff) |
extcap: Have extcap_spawn_sync use return value from ws_read_string_from_pipe
Continuation from I162b1322fd970777d2156d869e95b005b0f045ac
Change-Id: I5c93ad7eb5145577403721f3dd1ee8e836a4a828
Ping-Bug: 12845
Reviewed-on: https://code.wireshark.org/review/25025
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'extcap_spawn.c')
-rw-r--r-- | extcap_spawn.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/extcap_spawn.c b/extcap_spawn.c index 887694afca..c5659181ad 100644 --- a/extcap_spawn.c +++ b/extcap_spawn.c @@ -122,8 +122,11 @@ gboolean extcap_spawn_sync(gchar *dirname, gchar *command, gint argc, gchar **ar WaitForSingleObject(processInfo.hProcess, INFINITE); buffer = (gchar*)g_malloc(BUFFER_SIZE); - ws_read_string_from_pipe(child_stdout_rd, buffer, BUFFER_SIZE); - local_output = g_strdup_printf("%s", buffer); + status = ws_read_string_from_pipe(child_stdout_rd, buffer, BUFFER_SIZE); + if (status) + { + local_output = g_strdup_printf("%s", buffer); + } g_free(buffer); CloseHandle(child_stdout_rd); @@ -133,7 +136,6 @@ gboolean extcap_spawn_sync(gchar *dirname, gchar *command, gint argc, gchar **ar CloseHandle(processInfo.hProcess); CloseHandle(processInfo.hThread); - status = TRUE; } else status = FALSE; |