Bug 133779 - [maintainer] security/nmap - fix ncat host resolver
Summary: [maintainer] security/nmap - fix ncat host resolver
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Wesley Shields
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-16 14:20 UTC by Daniel Roethlisberger
Modified: 2009-04-21 16:40 UTC (History)
1 user (show)

See Also:


Attachments
nmap-ncat-fix.diff (1.75 KB, patch)
2009-04-16 14:20 UTC, Daniel Roethlisberger
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Roethlisberger 2009-04-16 14:20:03 UTC
Ncat uses getaddrinfo(3) to resolve hostnames and always passes it a numerical
service string.  FreeBSD getaddrinfo(3) until recently was broken with
numerical service strings (see PR bin/51827).  This patch adds a workaround for
FreeBSD versions with broken getaddrinfo(3), fixing Ncat where it is currently
unable to resolve any hostnames.

svn add files/patch-ncat__ncat_core.c

How-To-Repeat: ncat -v localhost 80
Comment 1 Wesley Shields freebsd_committer freebsd_triage 2009-04-16 15:02:02 UTC
Responsible Changed
From-To: freebsd-ports-bugs->wxs

I'll take it.
Comment 2 Wesley Shields freebsd_committer freebsd_triage 2009-04-16 15:04:35 UTC
If I am reading this patch correctly this does not apply to the upcoming
7.2 release.  If this is incorrect please let me know so I can get
portmgr approval to commit it, otherwise it will wait until after the
freeze is over.

-- WXS
Comment 3 Daniel Roethlisberger 2009-04-16 16:00:36 UTC
Wesley Shields <wxs@FreeBSD.org> 2009-04-16:
> If I am reading this patch correctly this does not apply to the
> upcoming 7.2 release.  If this is incorrect please let me know
> so I can get portmgr approval to commit it, otherwise it will
> wait until after the freeze is over.

Correct.  RELENG_7_2 has the fixed getaddrinfo(3); the Ncat patch
is only required for compatibility with 7.1 and earlier release
branches (and 8.0 before the end of March 2009).  I should have
stated that more clearly.

-- 
Daniel Roethlisberger
http://daniel.roe.ch/
Comment 4 Wesley Shields freebsd_committer freebsd_triage 2009-04-16 16:23:09 UTC
On Thu, Apr 16, 2009 at 05:00:36PM +0200, Daniel Roethlisberger wrote:
> Wesley Shields <wxs@FreeBSD.org> 2009-04-16:
> > If I am reading this patch correctly this does not apply to the
> > upcoming 7.2 release.  If this is incorrect please let me know
> > so I can get portmgr approval to commit it, otherwise it will
> > wait until after the freeze is over.
> 
> Correct.  RELENG_7_2 has the fixed getaddrinfo(3); the Ncat patch
> is only required for compatibility with 7.1 and earlier release
> branches (and 8.0 before the end of March 2009).  I should have
> stated that more clearly.

Great, thank you for the clarification.  I will queue this up to go into
the tree after the freeze has ended.  Thank you for your work on this.

-- WXS
Comment 5 dfilter service freebsd_committer freebsd_triage 2009-04-21 16:37:05 UTC
wxs         2009-04-21 15:36:56 UTC

  FreeBSD ports repository

  Modified files:
    security/nmap        Makefile 
  Added files:
    security/nmap/files  patch-ncat__ncat_core.c 
  Log:
  - Add patch to work around versions of FreeBSD with broken
    getaddrinfo(3) (7.1 and earlier, and -CURRENT before the end of March
    2009).
  
  PR:             ports/133779
  Submitted by:   Daniel Roethlisberger <daniel@roe.ch> (maintainer)
  
  Revision  Changes    Path
  1.115     +1 -0      ports/security/nmap/Makefile
  1.1       +33 -0     ports/security/nmap/files/patch-ncat__ncat_core.c (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 6 Wesley Shields freebsd_committer freebsd_triage 2009-04-21 16:37:11 UTC
State Changed
From-To: open->closed

Committed. Thanks!