[Bug binutils/17512] libbfd/binutils: crashes on fuzzed samples

[clyon at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=17512

Christophe Lyon <clyon at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |clyon at gcc dot gnu.org

--- Comment #230 from Christophe Lyon <clyon at gcc dot gnu.org> ---
(In reply to address@hidden from comment #228)
> The master branch has been updated by Nick Clifton <address@hidden>:
> 
> https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;
> h=4931146e91c3c8208f1e0b8462b8f33e70da506a
> 
> commit 4931146e91c3c8208f1e0b8462b8f33e70da506a
> Author: Nick Clifton <address@hidden>
> Date:   Thu Jul 28 15:37:16 2016 +0100
> 
>     Fix decoding of Windows resources.
>     
>       PR binutils/17512
>       * rescoff.c (read_coff_res_dir): Fix detection of buffer overrun.
>       * resbin.c (bin_to_res_version): Allow for the padded length of a
>       version block to be longer than the recorded length.  Skip padding
>       bytes.

Hi Nick,

Since you committed this patch, I've noticed build failures when binutils are
configured with:
--enable-64-bit-bfd=yes --target arm-wince-pe,
in a i386 chroot.

The error messages are:
../../binutils/resbin.c: In function 'bin_to_res_version':
../../binutils/resbin.c:967:5: error: format '%lu' expects argument of type
'long unsigned int', but argument 2 has type 'rc_uint_type' [-Werror=format=]
     verlen, length);
     ^
../../binutils/resbin.c:967:5: error: format '%lu' expects argument of type
'long unsigned int', but argument 3 has type 'rc_uint_type' [-Werror=format=]

-- 
You are receiving this mail because:
You are on the CC list for the bug.