[PATCH] libdwfl: Read no more than required to parse dynamic sections

2022-11-28 Thread gavin
From: Gavin Li Since size checking has been moved to dwfl_elf_phdr_memory_callback(), there is no longer a need for dwfl_segment_report_module() to enforce the same. Reading beyond the end of the dynamic section actually causes issues when passing the data to elfXX_xlatetom() because it is

Re: [PATCH] libdwfl: Read no more than required to parse dynamic sections

2022-11-29 Thread Gavin Li
heck be > changed to: > > if (dyn_data_size != 0 && dyn_data_size < dyn_filesz) > dyn_filesz = dyn_data_size; > I think for the purposes of reading small segments (like PT_DYNAMIC and PT_NOTE), we should ignore *buffer_available altogether. Best, Gavin

Re: [PATCH] libdwfl: Read no more than required to parse dynamic sections

2022-12-01 Thread Gavin Li
te patch. Best, Gavin On Wed, Nov 30, 2022 at 3:14 PM Mark Wielaard wrote: > > Hi Gavin, > > On Tue, Nov 29, 2022 at 01:48:42PM -0800, Gavin Li wrote: > > I think for the purposes of reading small segments (like PT_DYNAMIC > > and PT_NOTE), we should ignore *buffer_availabl