On Wed, 3 Apr 2019 at 21:32, Matthew Garrett <[email protected]> wrote: > > 769a8089c1fd2 (x86, efi, kasan: #undef memset/memcpy/memmove per arch) > disables the KASAN version of certain memory calls in the EFI boot stub. > tpm_eventlog.h references memcpy, so must be included after asm/efi.h in > order to allow 769a8089c1fd2 to take effect on its declarations. >
I can't find any reference to memcpy in tpm_eventlog.h or tpm.h which it includes. So can you describe the exact problem? If the issue is due to one of the transitively included headers two levels down, then this is fragile as hell since the same header may be included by linux/efi.h due to, e.g., minor config changes. > Signed-off-by: Matthew Garrett <[email protected]> > --- > drivers/firmware/efi/libstub/tpm.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/firmware/efi/libstub/tpm.c > b/drivers/firmware/efi/libstub/tpm.c > index b6e93e14fcf1..777c1e82495a 100644 > --- a/drivers/firmware/efi/libstub/tpm.c > +++ b/drivers/firmware/efi/libstub/tpm.c > @@ -8,9 +8,11 @@ > * Thiebaud Weksteen <[email protected]> > */ > #include <linux/efi.h> > -#include <linux/tpm_eventlog.h> > #include <asm/efi.h> > > +/* Must be included after asm/efi.h in order to avoid using the wrong memcpy > */ > +#include <linux/tpm_eventlog.h> > + > #include "efistub.h" > > #ifdef CONFIG_RESET_ATTACK_MITIGATION > -- > 2.21.0.392.gf8f6787159e-goog >
