aboutsummaryrefslogtreecommitdiffstats
path: root/capture_sync.c
diff options
context:
space:
mode:
authorAnders Broman <anders.broman@ericsson.com>2012-11-05 09:18:43 +0000
committerAnders Broman <anders.broman@ericsson.com>2012-11-05 09:18:43 +0000
commite57bf90ebeb17863f005f82009771510bb8b7362 (patch)
tree7067e0bc4fdb41911da0e3e653caf6a18c77c18e /capture_sync.c
parentdc284b0c0492ed87a4e7d9bf83705d76f042b6e3 (diff)
From Bill Parker:
code fails to check/test return value from select() https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7861 svn path=/trunk/; revision=45917
Diffstat (limited to 'capture_sync.c')
-rw-r--r--capture_sync.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/capture_sync.c b/capture_sync.c
index 2d071e7814..a1d0aa29bd 100644
--- a/capture_sync.c
+++ b/capture_sync.c
@@ -1575,6 +1575,10 @@ pipe_read_block(int pipe_fd, char *indicator, int len, char *msg,
/* we have a problem here, try to read some more bytes from the pipe to debug where the problem really is */
memcpy(msg, header, sizeof(header));
newly = read(pipe_fd, &msg[sizeof(header)], len-sizeof(header));
+ if (newly < 0) { /* error */
+ g_log(LOG_DOMAIN_CAPTURE, G_LOG_LEVEL_DEBUG,
+ "read from pipe %d: error(%u): %s", pipe_fd, errno, g_strerror(errno));
+ }
*err_msg = g_strdup_printf("Unknown message from dumpcap, try to show it as a string: %s",
msg);
return -1;