Re: [PATCH V2] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-09-02 Thread Robert Yang
On 08/31/2018 05:35 PM, Mark Wielaard wrote: Hi Robert, On Fri, 2018-08-31 at 10:17 +0800, Robert Yang wrote: Sorry, I can't make sure which ones is wrong, libqb, prelink or elfutils, this happens when cross compiling, and I've built more than 4 hunderds of packages, libqb 1.0.3 i

Re: [PATCH V2] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-30 Thread Robert Yang
On 08/31/2018 03:57 AM, Mark Wielaard wrote: On Wed, Aug 29, 2018 at 04:53:20PM +0800, Robert Yang wrote: The one which actually saves the data is data_list.data.d.d_buf, so check it before free rawdata_base. This can fix a segmentation fault when prelink libqb_1.0.3: prelink: /usr/lib

[PATCH V2] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-29 Thread Robert Yang
elf_end(). Signed-off-by: Robert Yang --- libelf/elf_end.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libelf/elf_end.c b/libelf/elf_end.c index 160f0b8..5280a70 100644 --- a/libelf/elf_end.c +++ b/libelf/elf_end.c @@ -160,14 +160,16 @@ elf_end (Elf *elf

[PATCH V1] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-29 Thread Robert Yang
* V2 - Also check data_list.data.d.d_buf before free scn->data_base, this can fix prelink error libqb_1.0.3 on mips and mips64. * V1 - Initial version Robert Yang (1): libelf/elf_end.c: check data_list.data.d.d_buf before free it libelf/elf_end.c | 6 -- 1 file changed

Re: [PATCH] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-20 Thread Robert Yang
On 08/17/2018 06:10 PM, Mark Wielaard wrote: Hi Robert, [I don't have very good internet connectivity so cannot easily get all the bits and sources to replicate/inspect. So apologies if I am misinterpreting something.] On Fri, Aug 17, 2018 at 04:25:07PM +0800, Robert Yang wrote:

Re: [PATCH] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-17 Thread Robert Yang
On 08/17/2018 03:25 AM, Mark Wielaard wrote: Hi, On Thu, Aug 16, 2018 at 10:34:23AM +0800, Robert Yang wrote: The one which actually saves the data is data_list.data.d.d_buf, so check it before free rawdata_base. This can fix a segmentation fault when prelink libqb_1.0.3: prelink: /usr/lib

[PATCH] libelf/elf_end.c: check data_list.data.d.d_buf before free it

2018-08-15 Thread Robert Yang
elf_end(). Signed-off-by: Robert Yang --- libelf/elf_end.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libelf/elf_end.c b/libelf/elf_end.c index 160f0b8..5388e80 100644 --- a/libelf/elf_end.c +++ b/libelf/elf_end.c @@ -165,9 +165,10 @@ elf_end (Elf *elf