Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

in6addr_any missing from HPE NonStop NSE #2146

Closed
rsbeckerca opened this issue Dec 3, 2017 · 4 comments
Closed

in6addr_any missing from HPE NonStop NSE #2146

rsbeckerca opened this issue Dec 3, 2017 · 4 comments
Labels
build

Comments

@rsbeckerca
Copy link
Contributor

I found a compile problem in the test suite for curl 7.57.0 (may be old as this is the first time we're trying to build with IPv6 enabled. I have a patch for this that I can provide in a pull request if desired, although the in6.h and in6addr_any may be distinct issues making the patch not entirely perfect.

I did this

When building curl on NSK-TANDEM-NSE, the definition in6addr_any is not found in in.h. In addition, IPv6 is defined in in6.h on this platform, which did not seem to be an issue for the main curl build.

    struct sockaddr_in6  sa6;
                         ^
"/home/git/curl/tests/server/server_sockaddr.h", line 34: error(163):
          incomplete type is not allowed


      me.sa6.sin6_addr = in6addr_any;
                         ^
"/home/git/curl/tests/server/rtspd.c", line 1321: error(114): identifier
          "in6addr_any" is undefined

I expected the following

Clean compile - rather obviously.

curl/libcurl version

7.57.0 (nse-tandem-nsk) libcurl/7.57.0 OpenSSL/1.0.2m zlib/1.2.8
Release-Date: [unreleased]
Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp smtp smtps telnet tftp
Features: NTLM NTLM_WB SSL libz TLS-SRP UnixSockets HTTPS-proxy

operating system

HPE NonStop J06.20
@rsbeckerca
Copy link
Contributor Author

I did miss a few required sources in this which I can add as part of a pull request:

  • lib/curl_addrinfo.h
  • lib/hostcheck.c
  • lib/hostip.c
  • lib/urldata.h

@bagder bagder added the build label Dec 4, 2017
@bagder
Copy link
Member

bagder commented Dec 4, 2017

note that in6addr_any is only used by the test suite servers, not by curl or libcurl

@rsbeckerca
Copy link
Contributor Author

Yes, however, in6.h is required for the above lib files because of sockaddr_in6, which on this platform is also in in6h.

@bagder
Copy link
Member

bagder commented Dec 4, 2017

aha!

@bagder bagder closed this as completed in 76ebd54 Dec 5, 2017
JohnDeHelian pushed a commit to JohnDeHelian/curl that referenced this issue Dec 7, 2017
@rsbeckerca
configure: check for netinet/in6.h
92e63c8 
Needed by HPE NonStop NSE and NSX systems

Fixes curl#2146
Closes curl#2155
@lock lock bot locked as resolved and limited conversation to collaborators May 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
build
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bagder @rsbeckerca