From ab6d1223c9e957d9f179327cd6016ac1780d23fe Mon Sep 17 00:00:00 2001 From: "vandebo@chromium.org" Date: Tue, 7 Sep 2010 19:39:37 +0000 Subject: Don't call freeaddrinfo(NULL) to avoid crash on FreeBSD. Apparently freeaddrinfo(NULL) is undefined (or at least FreeBSD reads the appropriate RFCs that way) and FreeBSD crashes in this case. BUG=FreeBSD crash TEST=no crash after change on FreeBSD. Review URL: http://codereview.chromium.org/3360012 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58735 0039d316-1c4b-4281-b951-d872f2087c98 --- net/base/host_resolver_proc.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/net/base/host_resolver_proc.cc b/net/base/host_resolver_proc.cc index 86304dc..d235ce9 100644 --- a/net/base/host_resolver_proc.cc +++ b/net/base/host_resolver_proc.cc @@ -223,8 +223,10 @@ int SystemHostResolverProc(const std::string& host, } } if (should_retry) { - freeaddrinfo(ai); - ai = NULL; + if (ai != NULL) { + freeaddrinfo(ai); + ai = NULL; + } err = getaddrinfo(host.c_str(), NULL, &hints, &ai); } -- cgit v1.1