RE: [PATCH] 'add symbol' feature in objcopy

["Tejas Belagod" <tejas dot belagod at foss dot arm dot com>]

> -----Original Message-----
> From: Tristan Gingold [mailto:gingold@adacore.com]
> Sent: 20 November 2015 15:38
> To: Tejas Belagod
> Cc: binutils@sourceware.org
> Subject: Re: [PATCH] 'add symbol' feature in objcopy
> 
> 
> > On 20 Nov 2015, at 16:29, Tejas Belagod <tejas.belagod@foss.arm.com>
> wrote:
> >
> > Hi Tristian,
> >
> > Is this patch OK to backport to 2.26?
> 
> Sure.
> 
> Tristan.
> >

Thanks. Here is the backport built and tested arm-none-eabi for mingw and
linux.

I don't have commit rights to binutils. Please could someone who does,
commit this?

binutils/ChangeLog

2015-11-23  Tejas Belagod  <tejas.belagod@arm.com>

	Backport from mainline:
	2015-11-20  Ronald Hoogenboom  <hoogenboom30@zonnet.nl>

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

Thanks,
Tejas.


> > Thanks,
> > Tejas.
> >
> >> -----Original Message-----
> >> From: Tejas Belagod [mailto:tejas.belagod@foss.arm.com]
> >> Sent: 20 November 2015 14:44
> >> To: 'Nick Clifton'; 'Ronald Hoogenboom'
> >> Cc: binutils@sourceware.org
> >> Subject: RE: [PATCH] 'add symbol' feature in objcopy
> >>
> >> Nick,
> >>
> >> Is this OK to backport to 2.26?
> >>
> >> Thanks,
> >> Tejas.
> >>
> >>>
> >>> -----Original Message-----
> >>> From: binutils-owner@sourceware.org [mailto:binutils-
> >>> owner@sourceware.org] On Behalf Of Nick Clifton
> >>> Sent: 20 November 2015 14:10
> >>> To: Ronald Hoogenboom; fracting@gmail.com
> >>> Cc: binutils@sourceware.org
> >>> Subject: Re: [PATCH] 'add symbol' feature in objcopy
> >>>
> >>> 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 the
> >>> following 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.
> >>>
> >>>
> >
> >

Attachment: objcopy-226.txt
Description: Text document