Re: [patch] openssl/md2 issue autoconf files configure.ac configure nse_openssl.cc

[olli hauer <ohauer () gmx de>]

On 2010-08-27 01:23, David Fifield wrote:
> On Fri, Aug 20, 2010 at 10:10:33PM +0200, olli hauer wrote:
> > Has someone a workaround if openssl is build with SCTP support?
> > I got the following build failure if SCTP is anabled in openssl.
> >
> > In file included from /usr/local/include/openssl/dtls1.h:78,
> >                  from /usr/local/include/openssl/ssl.h:1193,
> >                  from service_scan.cc:106:
> > /usr/include/netinet/sctp.h:57: error: redefinition of 'struct sctp_chunkhdr'
> > ./libdnet-stripped/include/dnet/sctp.h:39: error: previous definition of 'struct
> > sctp_chunkhdr'
> > gmake[1]: *** [service_scan.o] Error 1
> > gmake[1]: Leaving directory `/data/work/usr/ports/security/nmap/work/nmap-5.35DC1'
> > gmake: *** [all] Error 2
> > *** Error code 1
>
> Daniel, is this a naming conflict with libdnet's own sctp_chunkhdr?
> Might it have to be renamed?
>
> David Fifield


I may have found the issue.
sctp_chunkhdr is not defined in openssl-1.x but since Nov 2006 in FreeBSD.
(Sorry, I haven't mention FreeBSD in the first post)


If openssl-1.x is build on FreeBSD with SCTP support, then this lines are
present in "openssl/dtls1.h"

#ifdef OPENSSL_SCTP
#include <netinet/sctp.h>
#endif


In netinet/sctp.h sctp_chunkhdr is defined as

#define SCTP_PACKED __attribute__((packed))
struct sctp_chunkhdr {
    uint8_t chunk_type; /* chunk type */
    uint8_t chunk_flags;    /* chunk flags */
    uint16_t chunk_length;  /* chunk length */
    /* optional params follow */
}             SCTP_PACKED;

Source:
http://svn.freebsd.org/viewvc/base/release/7.0.0/sys/netinet/sctp.h?view=log

Where in nmap-5.x sctp_chunkhdr is defined as follows in
nmap-5.x/libdnet-stripped/include/dnet/sctp.h

struct sctp_chunkhdr {
    uint8_t     sch_type;   /* chunk type */
    uint8_t     sch_flags;  /* chunk flags */
    uint16_t    sch_length; /* chunk length */
} __attribute__((__packed__));


Indeed a quick fix for me is to rename sctp_chunkhdr to a different
name in this files.
- traceroute.cc
- scan_engine.cc
- libdnet-stripped/include/dnet/sctp.h


PS:
Is the nmap repo somewhere available as read only to test against HEAD?

--
olli
_______________________________________________
Sent through the nmap-dev mailing list
http://cgi.insecure.org/mailman/listinfo/nmap-dev
Archived at http://seclists.org/nmap-dev/