From: Sean Carey <carey@basho.com>
Date: Fri, 15 Feb 2013 22:39:18 +0000 (+0100)
Subject: BUG/MEDIUM: config: fix parser crash with bad bind or server address
X-Git-Tag: v1.5-dev18~88
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=58ea039115f3faaf29529e0df97f4562436fdd09;p=thirdparty%2Fhaproxy.git

BUG/MEDIUM: config: fix parser crash with bad bind or server address

If an address is improperly formated on a bind or server address
and haproxy is built for using getaddrinfo, then a crash may occur
upon the call to freeaddrinfo().

Thanks to Jon Meredith for helping me patch this for SmartOS,
I am not a C/GDB wizard.
---

diff --git a/src/standard.c b/src/standard.c
index b14b70ba27..380f7bedcc 100644
--- a/src/standard.c
+++ b/src/standard.c
@@ -600,7 +600,8 @@ struct sockaddr_storage *str2ip(const char *str)
 			}
 		}
 
-		freeaddrinfo(result);
+		if (result)
+			freeaddrinfo(result);
 	}
 #endif
 	/* unsupported address family */