This is the mail archive of the elfutils-devel@sourceware.org mailing list for the elfutils 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: [PATCH] gelf_getauxv: Use memcpy instead of pointer dereference to avoid alignment problems.


On Sun, Aug 25, 2013 at 09:40:49PM +0200, Kurt Roeckx wrote:
> It should probably be:
> memcpy(dst, (char *)data_scn->d.d_buf + ndx * sizeof(GElf_auxv_t), sizeof(GElf_axuv_t));

I do think so yes.
On which architecture did you see the alignment problem?
Is it easily repeatable?

BTW. Could you read the CONTRIBUTING file and add a Signed-off-by line
if possible to your patch.

Thanks,

Mark

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