At 2009-09-30 11:16:57 -0500, stef-l...@memberwebs.com wrote: > > I've now added tests for sys/ioctl.h and net/if.h even though these > headers seemed to be common to all the unixes investigated.
Thanks. I've marked this ready for committer now. > FWIW, there are checks for various bad netmasks. I incorporated these > techniques after seeing them in the corresponding postfix code. [...] > > BTW, there's also fallback code. If none of the methods work on a > given OS, then the ifaddrs code just lists 127.0.0.1/8 and ::1/128. Both of these look good. > + /* > + * Wrong address family. We allow only one case: if the file > + * has IPv4 and the port is IPv6, promote the file address to > + * IPv6 and try to match that way. > + */ I still think this comment should be fixed in _both_ places it now occurs, but oh well. (Note: I have not tested under Windows, and I've not tested test_ifaddrs.c. Also, there are a few lines which introduce trailing whitespace.) -- ams -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers