getaddrinfo() not correctly accepting some flag combinations

[anevilyak]

Haiku's getaddrinfo() returns an error indicating invalid flags with the following sample code:

struct addrinfo *ai;
struct addrinfo hints;
memset (&hints, '\0', sizeof (hints));
hints.ai_flags = AI_PASSIVE | AI_ADDRCONFIG;
hints.ai_socktype = SOCK_STREAM;
int e = getaddrinfo (NULL, "113", &hints, &ai);

All the documentation I've read indicates that this set of flags is correct though. Removing AI_ADDRCONFIG from the flags results in normal behavior though. I suspect this is due to some hack to temporarily disable IPv6 functionality, but haven't looked more closely. The error returned was 3, which gai_strerror() indicates as being "invalid flags".

[axeld]

Doesn't really look like it. In any case, you could easily try to revert hrev35942 which introduced ignoring IPv6 for AF_UNSPEC, and check it out.

[anevilyak]

Will see what I can find hopefully sometime this week, thanks for the tip!

[anevilyak]

Had a look, and it looks like a number of our AI_* defines are either bogus or out of date. Unfortunately I'm uncertain where our getaddrinfo() actually came from, though its code looks near identical to that used in FreeBSD. Our netdb.h looks quite different though as far as the AI_* defines go. Ours has AI_MASK (used to validate the flags in getaddrinfo()) hardwired to 0x00000007 for some reason, whereas FreeBSD's looks like:

#define AI_MASK \
    (AI_PASSIVE | AI_CANONNAME | AI_NUMERICHOST | AI_NUMERICSERV | \
    AI_ADDRCONFIG)

Also, some of the other AI_* values differ between us and FreeBSD's, though I'm uncertain if that's due to theirs simply being a newer revision or what. In any case, what would be the best course of action to update this, since I'm uncertain if our current libbind source is vendor branched somewhere?

[anevilyak]

On second thought, maybe it's simply a case of our netdb.h being out of date? The getaddrinfo()/getnameinfo() code that we have looks like it would work correctly with the updated flag defs (assuming libbind was recompiled).

[kaliber]

Replying to anevilyak:

Had a look, and it looks like a number of our AI_* defines are either bogus or out of date.

I reported that 13 months ago, see ticket #3945 ...

[anevilyak]

Didn't see that one, thanks.