Re: [PATCH] debuginfod: Support queries for ELF/DWARF sections

2022-10-26 Thread Aaron Merey via Elfutils-devel
Hi Mark, On Wed, Oct 26, 2022 at 11:06 AM Mark Wielaard wrote: > On Mon, 2022-10-24 at 14:38 -0400, Frank Ch. Eigler via Elfutils-devel > wrote: > > - not sure I understand why the code worries about dots in or not in > > section names. Why not just pass them verbatim throughout the code > >

Re: [PATCH] debuginfod: Support queries for ELF/DWARF sections

2022-10-26 Thread Aaron Merey via Elfutils-devel
Hi Frank, On Mon, Oct 24, 2022 at 2:38 PM Frank Ch. Eigler wrote: > - use of write(2) to put files onto disk is not quite right; write(2) can > be partial, so you need a loop (or a macro wrapping a loop) Fixed. > - not sure I understand why the code worries about dots in or not in > section

Re: [PATCH] debuginfod: Support queries for ELF/DWARF sections

2022-10-26 Thread Frank Ch. Eigler via Elfutils-devel
Hi - > Is/should the section name be URL-encoded? Yes! > I would drop the maybe_debuginfo_section heuristics. There are some > sections like .strtab/.symtab that are probably in the debug file, but > might be in the executable. I would assume that a named section can > normally be found in the d

Re: [PATCH] debuginfod: Support queries for ELF/DWARF sections

2022-10-26 Thread Mark Wielaard
Hi, On Mon, 2022-10-24 at 14:38 -0400, Frank Ch. Eigler via Elfutils-devel wrote: > - use of write(2) to put files onto disk is not quite right; write(2) > can > be partial, so you need a loop (or a macro wrapping a loop) Since debuginfod-client.c already includes system.h it can use: static i

Re: [PATCH] debuginfod: Support queries for ELF/DWARF sections

2022-10-26 Thread Mark Wielaard
Hi, On Mon, 2022-10-24 at 14:38 -0400, Frank Ch. Eigler via Elfutils-devel wrote: > - not sure I understand why the code worries about dots in or not in > section names. Why not just pass them verbatim throughout the code > base, and not worry about whether or not there's a dot? Does the >