> > + utf16_strlen(efi_name))) { > > + /* > > + * Check if an old format, > > + * which doesn't support holding > > + * multiple logs, remains. > > + */ > > + if (sscanf(name, "dump-type%u-%u-%lu", > > + &type_old, &part_old, &time_old) != > > + 3) > > This doesn't look right. This should probably mirror the sprintf() at the top of the function using a new string, convert it to 16-bit > Unicodeand then compare it like we do for the 4-variable version above (please ignore the fact that this driver incorrectly calls these > strings utf16 everywhere -- that needs to be fixed). I will fix it as well. Seiji -- 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