This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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] 'add symbol' feature in objcopy

From: Ronald Hoogenboom <hoogenboom30 at zonnet dot nl>
To: Nick Clifton <nickc at redhat dot com>, Ronald Hoogenboom <hoogenboom30 at zonnet dot nl>, fracting at gmail dot com
Cc: binutils at sourceware dot org
Date: Fri, 20 Nov 2015 20:24:52 +0100
Subject: Re: [PATCH] 'add symbol' feature in objcopy
Authentication-results: sourceware.org; auth=none
References: <561D4796 dot 2080400 at atom dot grundel> <CALd+sZQZtJWcgc_U7ofBn2ezq4SToOQ=F+016_--pgA0LbKnSg at mail dot gmail dot com> <5639305F dot 8030406 at atom dot grundel> <564F29D2 dot 3070500 at redhat dot com>

Super! Thanks.
The fsf assignment has been sent, it should be there by now...

Best regards,
Ronald.

On 11/20/2015 03:10 PM, Nick Clifton wrote:

Hi Ronald,

Apparently, the strndup function is not 'portable' enough for mingw-w64.
There seems to be a xstrndup function in libiberty that could be used
instead. I'm sorry I don't have all the possible compilers handy to
verify all possible targets, but this should do.

Thanks - I have gone ahead and checked this patch in along with thefollowing copyright notice.


Cheers
  Nick

binutils/ChangeLog
2015-11-20  Ronald Hoogenboom  <hoogenboom30@zonnet.nl>

    * objcopy.c (parse_symflags): Use xstrndup in place of strndup.
    (copy_main): Likewise.

References:
- Re: [PATCH] 'add symbol' feature in objcopy
  - From: Qian Hong
- Re: [PATCH] 'add symbol' feature in objcopy
  - From: Ronald Hoogenboom
- Re: [PATCH] 'add symbol' feature in objcopy
  - From: Nick Clifton

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