summaryrefslogtreecommitdiffstats
path: root/plugins/check_tcp.c
diff options
context:
space:
mode:
authorjuliopedreira <julio.pedreira@gmail.com>2013-09-10 08:11:12 (GMT)
committerHolger Weiss <holger@zedat.fu-berlin.de>2013-09-10 10:16:46 (GMT)
commitcfb50add532c1b5c80ea8214340b0a908a7fab65 (patch)
tree2a678c2bab011fb4e54718c353d0e7b7e672914e /plugins/check_tcp.c
parent5bfca4b34d15ef40239ed6074b9ff9c061022946 (diff)
downloadmonitoring-plugins-cfb50add532c1b5c80ea8214340b0a908a7fab65.tar.gz
check_tcp: Don't close connection too early
Closing the connection because the bytes received are less than the buffer size assumes that all the bytes will be received in one go. This is not always true!
Diffstat (limited to 'plugins/check_tcp.c')
-rw-r--r--plugins/check_tcp.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/plugins/check_tcp.c b/plugins/check_tcp.c
index ce966c3..b024821 100644
--- a/plugins/check_tcp.c
+++ b/plugins/check_tcp.c
@@ -277,10 +277,7 @@ main (int argc, char **argv)
277 memcpy(&status[len], buffer, i); 277 memcpy(&status[len], buffer, i);
278 len += i; 278 len += i;
279 279
280 /* stop reading if user-forced or data-starved */ 280 /* stop reading if user-forced */
281 if(i < sizeof(buffer) || (maxbytes && len >= maxbytes))
282 break;
283
284 if (maxbytes && len >= maxbytes) 281 if (maxbytes && len >= maxbytes)
285 break; 282 break;
286 } 283 }