Re: New format Intel microcode...

From: Arjan van de Ven
Date: Mon Mar 26 2007 - 04:53:57 EST


Marcel Holtmann wrote:
that is the whole point. The slash was never meant to be used. It was
designed to take a filename or a pattern that will be later matched by
userspace. However some developers are now trying to abuse this since
the simple firmware helper script matches this directly to a filename
(and directory in this case) on the disk.

Putting a slash in the request_firmware() call now enforces a
subdirectory

I don't see how this follows from the former. Userspace is free to translate the kernel string into anything it wants, even a simple replacement of / with _. So I don't see how this "enforces" a subdirectory. Firmware gets a namespace basically, and a / is a logical namespace separator.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/