On Fri, Jul 22, 2022 at 02:43:13AM -0500, Glenn Washburn wrote: > According to the LUKS specification the uuid header field is of data type > "char[]", which is defined as "a string stored as null terminated sequence > of 8-bit characters". So enforce this by adding a null byte as the last byte > of the uuid. The LUKS2 specification defers to the LUKS1 specification in > this regard. > > Signed-off-by: Glenn Washburn <developm...@efficientek.com>
In general Reviewed-by: Daniel Kiper <daniel.ki...@oracle.com>... Though I want to ask you to add a sentence to the commit message saying why current code works now. And a second sentence should say the part of existing code making it work will be removed by subsequent patch. Otherwise it is not really clear why this patch is needed... Or maybe we should merge patch #1 and #2... However, I do not have strong opinion here. Daniel _______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel