Re: efivarfs: guid part of filenames are case-insensitive is broken

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/05/2013 05:46 AM, Joseph Yasi wrote:
> Hi,
> 
> With linux 3.8.2 I can no longer mount efivars on my Lenovo Thinkpad T530:
> 
> $ sudo mount -v /sys/firmware/efi/efivars
> mount: Cannot allocate memory

Looking at 47f531e8ba3bc3901a0c493f4252826c41dea1a1
efivarfs: Validate filenames much more aggressively

In efivarfs_valid_name:

+	/* GUID should be right after the first '-' */
+	if (s - 1 != strchr(str, '-'))
+		return false;

But pstore creates entries like dump-type0-1-$timestamp-$GUID.

efivarfs_alloc_dentry fails because it thinks pstore entry names
are invalid.

EFI pstore dump is now enabled by default (as in Fedora), saving
dump sooner or later, so users will likely be affected by this.

Manually creating a test-test-$GUID also reproduces this.


Lingzhu Xiang
--
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


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux