svn commit: trunk/busybox/archival/libunarchive

[Peter Kjellerstedt]

> -----Original Message-----
> From: busybox-cvs-bounces at busybox.net 
> [mailto:busybox-cvs-bounces at busybox.net] On Behalf Of 
> landley at busybox.net
> Sent: Saturday, November 12, 2005 03:54
> To: busybox-cvs at busybox.net
> Subject: svn commit: trunk/busybox/archival/libunarchive
> 
> Author: landley
> Date: 2005-11-11 18:54:06 -0800 (Fri, 11 Nov 2005)
> New Revision: 12224
> 
> Log:
> I noticed that "tar tvjf file.tbz" was segfaulting.  This fixed it.
> 
> 
> Modified:
>    trunk/busybox/archival/libunarchive/header_verbose_list.c
> 
> Changeset:
> Modified: trunk/busybox/archival/libunarchive/header_verbose_list.c
> ===================================================================
> --- trunk/busybox/archival/libunarchive/header_verbose_list.c	
> 2005-11-12 01:15:04 UTC (rev 12223)
> +++ trunk/busybox/archival/libunarchive/header_verbose_list.c	
> 2005-11-12 02:54:06 UTC (rev 12224)
> @@ -6,7 +6,7 @@
>  
>  extern void header_verbose_list(const file_header_t *file_header)
>  {
> -	struct tm *mtime = localtime(&file_header->mtime);
> +	struct tm *mtime = localtime(&(file_header->mtime));
>  
>  	printf("%s %d/%d%10u %4u-%02u-%02u %02u:%02u:%02u %s",
>  		bb_mode_string(file_header->mode),

How can this possibly make any difference at all?  The -> operator
already has a higher precedence than the & operator, so the 
parenthesises should change nothing.  Or am I missing something?

//Peter