[Bug binutils/14302] ar mishandles 4GB files

2012-06-29 Thread fgouget at codeweavers dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

--- Comment #6 from Francois Gouget  2012-06-30 
00:54:00 UTC ---
Thanks for tweaking the patch so it could be committed.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/14302] ar mishandles 4GB files

2012-06-29 Thread cvs-commit at gcc dot gnu.org
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

--- Comment #4 from cvs-commit at gcc dot gnu.org  2012-06-29 12:59:54 UTC ---
CVSROOT:/cvs/src
Module name:src
Changes by:ni...@sourceware.org2012-06-29 12:59:49

Modified files:
binutils   : ChangeLog ar.c bucomm.c 

Log message:
PR binutils/14302
* bucomm.c (print_arelt_descr): Correctly report the archive size
field (for 'ar tv').
* ar.c (print_contents): Use correct types for archive element
sizes (for 'ar p').
(extract_file): Likewise (for 'ar x').

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/ChangeLog.diff?cvsroot=src&r1=1.1918&r2=1.1919
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/ar.c.diff?cvsroot=src&r1=1.84&r2=1.85
http://sourceware.org/cgi-bin/cvsweb.cgi/src/binutils/bucomm.c.diff?cvsroot=src&r1=1.45&r2=1.46

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/14302] ar mishandles 4GB files

2012-06-29 Thread nickc at redhat dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 CC||nickc at redhat dot com
 Resolution||FIXED

--- Comment #5 from Nick Clifton  2012-06-29 13:02:52 
UTC ---
Hi Francois,

  Thanks for the bug report and patch, and please accept my apologies in
getting round to reviewing it.  The patch itself is fine, although there are a
couple of places where a comment needs to be updated, and a couple of places
where there are now redundant type cases.  I have taken care of both of these
things and applied your patch.

Cheers
  Nick

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/14302] ar mishandles 4GB files

2012-06-27 Thread fgouget at codeweavers dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

Francois Gouget  changed:

   What|Removed |Added

   Attachment #6489|0   |1
is obsolete||

--- Comment #3 from Francois Gouget  2012-06-27 
09:33:48 UTC ---
Created attachment 6490
  --> http://sourceware.org/bugzilla/attachment.cgi?id=6490
A fix for 'ar tv', 'ar p' and 'ar x'

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/14302] ar mishandles 4GB files

2012-06-27 Thread fgouget at codeweavers dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

--- Comment #2 from Francois Gouget  2012-06-27 
09:30:08 UTC ---
Note that 'ar p' is also impacted:

$ ~/binutils-cvs/binutils/ar p ar4G.ar >file4G
$ ls -l file4G
-rw-r--r-- 1 fgouget fgouget 104857600 Jun 27 11:29 file4G

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug binutils/14302] ar mishandles 4GB files

2012-06-27 Thread fgouget at codeweavers dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=14302

Francois Gouget  changed:

   What|Removed |Added

   Attachment #6488|0   |1
is obsolete||

--- Comment #1 from Francois Gouget  2012-06-27 
09:26:26 UTC ---
Created attachment 6489
  --> http://sourceware.org/bugzilla/attachment.cgi?id=6489
A fix for 'ar tv' and 'ar x'

Updated the commit changelog to reference this bug and be more explicit about
the ar commands being fixed.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils