Re: [PATCH] libelf: elf_compress doesn't handle multiple elf_newdata chunks correctly

2024-10-18 Thread Mark Wielaard
Hi Aaron, On Fri, Oct 18, 2024 at 12:08:11AM -0400, Aaron Merey wrote: > LGTM. The new testcases pass on the trybots. And when I revert the > elf_compress.c changes and run the new tests they fail as expected, so > the elf_compress.c changes do in fact fix PR32102. Thanks, pushed. Cheers, Mark

Re: [PATCH] libelf: elf_compress doesn't handle multiple elf_newdata chunks correctly

2024-10-17 Thread Aaron Merey
On Thu, Oct 17, 2024 at 11:59 AM Mark Wielaard wrote: > > elf_compress would compress all (new) data chunks, but didn't reset > the section data_list. This would cause extra data to be returned > after decompression or create bad compressed data. Add a new testcase > for this and explicitly zap th