alpha net-yy test failures due to port decode errors

Mike Frysinger vapier at gentoo.org
Tue Mar 3 00:07:12 UTC 2015


On 03 Mar 2015 02:24, Dmitry V. Levin wrote:
> On Mon, Mar 02, 2015 at 02:32:53PM -0500, Mike Frysinger wrote:
> > done.  the first run is with the +20 hack while the second is w/out.
> > 
> > TCP:[127.0.0.1:44261->127.0.0.1:41995]
> > 0000: 02  01  00  00  ac  e5  a4  0b | 7f  00  00  01  00  00  00  00  
> > 0010: 00  00  00  00  00  00  00  00 | 7f  00  00  01  00  00  00  00  
> > 0020: 00  00  00  00  00  00  00  00 | 00  00  00  00  00  b6  dc  f6  
> > 0030: 00  fc  ff  ff  00  00  00  00 | 00  00  00  00  00  00  00  00  
> > 0040: ed  03  00  00  53  b8  80  05 | 
> > TCP:[127.0.0.1:39667->127.0.0.1:60672]
> > 0000: 02  01  00  00  9a  f3  ed  8e | 7f  00  00  01  00  00  00  00  
> > 0010: 00  00  00  00  00  00  00  00 | 7f  00  00  01  00  00  00  00  
> > 0020: 00  00  00  00  00  00  00  00 | 00  00  00  00  80  de  dc  f6  
> > 0030: 00  fc  ff  ff  00  00  00  00 | 00  00  00  00  00  00  00  00  
> > 0040: ed  03  00  00  9c  ab  80  05 | 
> 
> In other words, the whole diag_msg struct is ok, diag_msg->id.idiag_sport
> is ok, but diag_msg->id.idiag_dport is NOT ok.
> 
> Could you try this patch, please?
> 
> --- a/socketutils.c
> +++ b/socketutils.c
> @@ -114,7 +114,7 @@ receive_responses(const int fd, const unsigned long inode,
>  		  const char *proto_name,
>  		  bool (* parser) (const char *, const void *, int, const unsigned long))
>  {
> -	static char buf[8192];
> +	static long buf[8192 / sizeof(long)];
>  	struct sockaddr_nl nladdr = {
>  		.nl_family = AF_NETLINK
>  	};

tests pass on alpha with this one change, so i guess that means it's ok ? :)
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20150302/493413d9/attachment.bin>


More information about the Strace-devel mailing list