On Thu, 04 Feb, at 10:34:33AM, Peter Jones wrote: > v2 correctly checks the guid for validation *as well as* attribute > whitelisting. > v3 moves the sorting of variable_validate here and adds a comment about > it. > > Signed-off-by: Peter Jones <pjones@xxxxxxxxxx> > Tested-by: Lee, Chun-Yi <jlee@xxxxxxxx> > Acked-by: Matthew Garrett <mjg59@xxxxxxxxxx> > --- > drivers/firmware/efi/efivars.c | 5 ++-- > drivers/firmware/efi/vars.c | 52 +++++++++++++++++++++++++++--------------- > include/linux/efi.h | 3 ++- > 3 files changed, 38 insertions(+), 22 deletions(-) I've expanded on the commit message ever so slightly so that a casual observer can figure out why this change was made. Something like this? --- efi: Make our variable validation list include the guid This list of variables is defined to be part of the global namespace in the UEFI spec, so this just further ensures we're validating the variables we think we are. Including the guid for entries will become more important in future patches when we decide whether or not to allow deletion of variables based on presence in this list. -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
