[PATCH] Save/restore file offset while reading notes in core file
Keith Seitz
keiths@redhat.com
Wed Mar 31 13:06:03 GMT 2021
On 3/31/21 3:45 AM, Martin Liška wrote:
> On 3/26/21 6:48 PM, Keith Seitz via Binutils wrote:
>> On 3/18/21 9:22 PM, Alan Modra wrote:
>>> On Thu, Mar 18, 2021 at 08:40:26AM -0700, Keith Seitz via Binutils wrote:
>>>> * elfcore.h (_bfd_elf_core_find_build_id): Save and restore
>>>> file offset when calling elf_read_notes.
>>>
>>> Looks OK to me, but you could avoid the ftell with
>>>
>>> /* Make sure ABFD returns to processing the program headers. */
>>> if (bfd_seek (abfd, (file_ptr) (offset + i_ehdr.e_phoff
>>> + (i + 1) * sizeof (x_phdr)),
>>> SEEK_SET) != 0)
>>> goto fail;
>>>
>>> Either way is fine.
>>>
>>
>> After seeing all the hoops that tell can jump through, I've decided
>> to run with your suggestion, and I've pushed this change.
>
> Hello.
>
> May I ask: has the patch been pushed? I can't see it in binutils master.
https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=ff5b3e1458875830595e8c1d1fab6624264b3935
Is that not showing up?
Keith
More information about the Binutils
mailing list