This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] peXXigen.c, _bfd_XXi_swap_aux_in, wrong size used in memcpy.


On Thursday 03 March 2011 18:09:04, Michael Snyder wrote:
>   2011-03-03  Michael Snyder  <msnyder@msnyder-server.eng.vmware.com>
> 
>         * peXXigen.c (_bfd_XXi_swap_aux_in): Use E_FILNMNEN instead of
>         FILENMLEN, otherwise will overwrite array.

Doesn't pe.h define them both the same?

> 
> Index: peXXigen.c
> ===================================================================
> RCS file: /cvs/src/src/bfd/peXXigen.c,v
> retrieving revision 1.69
> diff -u -p -u -p -r1.69 peXXigen.c
> --- peXXigen.c  21 Dec 2010 15:24:38 -0000      1.69
> +++ peXXigen.c  3 Mar 2011 18:03:44 -0000
> @@ -249,7 +249,7 @@ _bfd_XXi_swap_aux_in (bfd * abfd,
>           in->x_file.x_n.x_offset = H_GET_32 (abfd, ext->x_file.x_n.x_offset);
>         }
>        else
> -       memcpy (in->x_file.x_fname, ext->x_file.x_fname, FILNMLEN);
> +       memcpy (in->x_file.x_fname, ext->x_file.x_fname, E_FILNMLEN);
>        return;
>  
>      case C_STAT:
> @@ -323,7 +323,7 @@ _bfd_XXi_swap_aux_out (bfd *  abfd,
>           H_PUT_32 (abfd, in->x_file.x_n.x_offset, ext->x_file.x_n.x_offset);
>         }
>        else
> -       memcpy (ext->x_file.x_fname, in->x_file.x_fname, FILNMLEN);
> +       memcpy (ext->x_file.x_fname, in->x_file.x_fname, E_FILNMLEN);

If FILNMLEN can really be different from E_FILNMLEN, I'd've expected
something else needs doing here?

-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]