Fix crash caused by hostname in socksParentProxy

Solar Flare <[email protected]> Mon, 5 May 2014 18:59:30 +0800
Newsgroups gmane.comp.web.polipo.user
Message-ID <CAAvra1SnUtG83MSCxBcHv3Bt53xM+KcpqcqX_0W9oBWPZ-KjLg@mail.gmail.com>
I use polipo to bridge from socks proxy to http. Recently, I noticed the
1.1.0 version was released. But an old crash bug I suffered is still exist.
If I use the hostname of socks proxy in config along with the built-in dns
resolver, polipo always crashes during the first connection. I tried to fix
it today and here is the patch.

Cheers
Solar

------------------------------------------------------------------------------
Is your legacy SCM system holding you back? Join Perforce May 7 to find out:
&#149; 3 signs your SCM is hindering your productivity
&#149; Requirements for releasing software faster
&#149; Expert tips and advice for migrating your SCM now
http://p.sf.net/sfu/perforce

_______________________________________________
Polipo-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/polipo-users
0001-Fix-crash-caused-by-hostname-in-socksParentProxy.patch (application/octet-stream, 727 B)
From 48e18e5a3d46807b740e241e83679335ed082932 Mon Sep 17 00:00:00 2001
From: Solar Flare <[email protected]>
Date: Mon, 5 May 2014 18:33:59 +0800
Subject: [PATCH] Fix crash caused by hostname in socksParentProxy

---
 socks.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/socks.c b/socks.c
index ce8f600..0cfd236 100644
--- a/socks.c
+++ b/socks.c
@@ -233,7 +233,7 @@ socksDnsHandler(int status, GethostbynameRequestPtr grequest)
             return 1;
         }
         do_gethostbyname(grequest->addr->string + 1, grequest->count + 1,
-                         httpServerConnectionDnsHandler, request);
+                         socksDnsHandler, request);
         return 1;
     }
 
-- 
1.7.10.4