apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: Bug to fix or not in 1.2.x - recvfrom?
Date Mon, 15 Oct 2007 08:02:33 GMT
On Mon, Oct 15, 2007 at 01:17:46AM -0500, William Rowe wrote:
> [more details about that universal recvfrom issue in my prior note, I 
> thought
> I had sent this hours ago...]
> 
>     apr_sockaddr_ip_get(&ip_addr, from);
>     fromport = from->port;
>     ABTS_STR_EQUAL(tc, addr, ip_addr);
>     ABTS_INT_EQUAL(tc, 7771, fromport);
> 
> Because we weren't randomizing this, we hadn't noted that fromport was
> not being filled in by apr_socket_recvfrom() (although the address is).
> 
> On Linux, ipv6 loopback enabled;
> 
> ./testall -v testsockets
> testsockets         : |Line 178: expected <7771>, but saw <4242>
> FAILED 1 of 6

Well, yes, you merged a test case without the corresponding bug fix that 
it was testing for (both were part of r467600).

If you revert the changes you made to testsockets.c, and backport 
r467600 in its entirety, I don't see any problems here on Linux.

joe

Mime
View raw message