[Nagios-checkins] SF.net SVN: nagios:[2283] nagioscore/trunk/base/query-handler.c

[<ag...@us...>]

Revision: 2283
          http://nagios.svn.sourceforge.net/nagios/?rev=2283&view=rev
Author:   ageric
Date:     2012-09-21 13:14:47 +0000 (Fri, 21 Sep 2012)
Log Message:
-----------
query-handler: Avoid coredumps on incomplete queries

If we get no data from iocache_use_delim(), we must return early
(without errors) so we don't try to dereference a NULL pointer.

Signed-off-by: Andreas Ericsson <ae...@op...>

Modified Paths:
--------------
    nagioscore/trunk/base/query-handler.c

Modified: nagioscore/trunk/base/query-handler.c
===================================================================
--- nagioscore/trunk/base/query-handler.c	2012-09-21 13:14:29 UTC (rev 2282)
+++ nagioscore/trunk/base/query-handler.c	2012-09-21 13:14:47 UTC (rev 2283)
@@ -91,6 +91,9 @@
 		 * to the handler
 		 */
 		buf = iocache_use_delim(ioc, "\0", 1, &len);
+		if(!buf)
+			return 0;
+
 		if(*buf != '@') {
 			/* bad request, so nuke the socket */
 			nsock_printf(sd, "400: Bad request");

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.