Nmap Development mailing list archives
Re: [Patch] Traceroute, Windows localhost scan fix
From: Kris Katterjohn <katterjohn () gmail com>
Date: Fri, 20 Apr 2007 17:58:38 -0500
Eddie Bell wrote:
Hey Kris, On 20/04/07, Kris Katterjohn <katterjohn () gmail com> wrote:Hey Eddie, I was looking at your patch, and the checking of pd that's being added doesn't look necessary because my_pcap_open_live() only returns a good descriptor.The problem is that if pcap_open_live fails then nmap quits. pcap_open_live will fail on Windows if it a trace is executed against localhost. So we have a test before-hand to return if this is the case. pd is initially set to NULL. The check of pd is there so other functions know if pcap_open_live was executed, otherwise we get a segmentation fault when traceroute tries to access it.
Ah, thanks. Sorry, I really should've applied the patch and looked rather than an just look at the patch -- it makes it much more clear. Begin sick doesn't help either :) Thanks again, Kris Katterjohn _______________________________________________ Sent through the nmap-dev mailing list http://cgi.insecure.org/mailman/listinfo/nmap-dev Archived at http://SecLists.Org
Current thread:
- [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Kris Katterjohn (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Kris Katterjohn (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 21)
- Re: [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Kris Katterjohn (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Kris Katterjohn (Apr 20)
- Re: [Patch] Traceroute, Windows localhost scan fix Eddie Bell (Apr 25)
- Re: [Patch] Traceroute, Windows localhost scan fix Fyodor (Apr 25)