On Thu, Apr 10, 2025 at 09:45:15AM +0200, Jan Beulich wrote: > With us now reading the full combined optional and NT headers, the > subsequent reading of (and seeking to) NT header fields is wrong. Since > PE32 and PE32+ NT headers are different anyway (beyond the image base > oddity extending across both headers), switch to using a union. This > allows to fetch the image base more directly then. > > Additionally add checking to map_section(), which would have caught at > least the wrong (zero) image size that we previously used. > > Fixes: f7f42accbbbb ("x86/efi: Use generic PE/COFF structures") > Reported-by: Roger Pau Monné <roger....@citrix.com> > Signed-off-by: Jan Beulich <jbeul...@suse.com>
Reviewed-by: Roger Pau Monné <roger....@citrix.com> Thanks, Roger.