<feed xmlns='http://www.w3.org/2005/Atom'>
<title>user/sven/linux.git/drivers/base/firmware_class.c, branch v2.6.14.7</title>
<subtitle>Linux Kernel
</subtitle>
<id>https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v2.6.14.7</id>
<link rel='self' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/atom?h=v2.6.14.7'/>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/'/>
<updated>2005-09-13T15:22:27Z</updated>
<entry>
<title>[PATCH] drivers/base/*: use kzalloc instead of kmalloc+memset</title>
<updated>2005-09-13T15:22:27Z</updated>
<author>
<name>Jiri Slaby</name>
<email>jirislaby@gmail.com</email>
</author>
<published>2005-09-13T08:25:01Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=4aed0644d684428e811bb6944f032b460a3ab165'/>
<id>urn:sha1:4aed0644d684428e811bb6944f032b460a3ab165</id>
<content type='text'>
Fixes a bunch of memset bugs too.

Signed-off-by: Lion Vollnhals &lt;webmaster@schiggl.de&gt;
Signed-off-by: Jiri Slaby &lt;xslaby@fi.muni.cz&gt;
Cc: Greg KH &lt;greg@kroah.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] modified firmware_class.c to support no hotplug</title>
<updated>2005-09-07T23:57:26Z</updated>
<author>
<name>Abhay Salunke</name>
<email>Abhay_Salunke@dell.com</email>
</author>
<published>2005-09-06T22:17:13Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=6e3eaab02028c4087a92711b20abb9e72cc803a7'/>
<id>urn:sha1:6e3eaab02028c4087a92711b20abb9e72cc803a7</id>
<content type='text'>
Upgrade the request_firmware_nowait function to not start the hotplug
action on a firmware update.

This patch is tested along with dell_rbu driver on i386 and x86-64 systems.

Signed-off-by: Abhay Salunke &lt;Abhay_Salunke@dell.com&gt;
Cc: Greg KH &lt;greg@kroah.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] request_firmware(): avoid race conditions</title>
<updated>2005-06-29T04:20:30Z</updated>
<author>
<name>Stanislaw W. Gruszka</name>
<email>stf_xl@wp.pl</email>
</author>
<published>2005-06-29T03:44:51Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=b92eac01c3c6ad5dab5c74d3bc32ac8e9bf4dae7'/>
<id>urn:sha1:b92eac01c3c6ad5dab5c74d3bc32ac8e9bf4dae7</id>
<content type='text'>
Avoid race occurs when some process have open file descriptor for class
device attributes and already firmware allocated memory are freed.  Don't
allow negative loading timeout.

Signed-off-by: Stanislaw W. Gruszka &lt;stf_xl@wp.pl&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] add TIMEOUT to firmware_class hotplug event</title>
<updated>2005-04-19T04:57:31Z</updated>
<author>
<name>kay.sievers@vrfy.org</name>
<email>kay.sievers@vrfy.org</email>
</author>
<published>2005-04-19T04:57:31Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=6897089c5f7989603ccb9c696050470ba1dbd262'/>
<id>urn:sha1:6897089c5f7989603ccb9c696050470ba1dbd262</id>
<content type='text'>
On Tue, 2005-03-15 at 09:25 +0100, Hannes Reinecke wrote:
&gt; The current implementation of the firmware class breaks a fundamental
&gt; assumption in udevd: that the physical device can be initialised fully
&gt; prior to executing the next event for that device.

Here we add a TIMEOUT value to the hotplug environment of the firmware
requesting event. I will adapt udevd not to wait for anything else, if
it finds a TIMEOUT key.

Signed-off-by: Kay Sievers &lt;kay.sievers@vrfy.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@suse.de&gt;


</content>
</entry>
<entry>
<title>[PATCH] fix unnecessary increment in firmware_class_hotplug() and USB core</title>
<updated>2004-11-29T02:40:36Z</updated>
<author>
<name>Marcel Holtmann</name>
<email>marcel@holtmann.org</email>
</author>
<published>2004-11-29T02:40:36Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=71e560322aa0a22c05ec919b48e0861860b6a21b'/>
<id>urn:sha1:71e560322aa0a22c05ec919b48e0861860b6a21b</id>
<content type='text'>
&gt;   This patch is to fix unnecessary increment of 'i' used to
&gt; specify an element of an arry 'envp[]' in firmware_class_hotplug().
&gt; The 'i' is already incremented in add_hotplug_env_var(), actually.

you are right. The incrementation is wrong, but it doesn't have any
negative effect. However the same applies for the usb_hotplug() function
in drivers/usb/core/usb.c.

Signed-off-by: Keiichiro Tokunaga &lt;tokunaga.keiich@jp.fujitsu.com&gt;
Signed-off-by: Marcel Holtmann &lt;marcel@holtmann.org&gt;
Signed-off-by: Greg Kroah-Hartman &lt;greg@kroah.com&gt;
</content>
</entry>
<entry>
<title>[PATCH] Use add_hotplug_env_var() in firmware loader</title>
<updated>2004-11-07T12:10:32Z</updated>
<author>
<name>Marcel Holtmann</name>
<email>marcel@holtmann.org</email>
</author>
<published>2004-11-07T12:10:32Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=27ec38fdf20de7a9210cc07cfda91d47948aed44'/>
<id>urn:sha1:27ec38fdf20de7a9210cc07cfda91d47948aed44</id>
<content type='text'>
The add_hotplug_env_var() function is available and so use it in the
firmware class code.

Signed-off-by: Marcel Holtmann &lt;marcel@holtmann.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] unexport firmware_class</title>
<updated>2004-10-28T01:30:48Z</updated>
<author>
<name>Christoph Hellwig</name>
<email>hch@lst.de</email>
</author>
<published>2004-10-28T01:30:48Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=289126deda79b60969bf40531e7bbed3bb1f984e'/>
<id>urn:sha1:289126deda79b60969bf40531e7bbed3bb1f984e</id>
<content type='text'>
it's marked static already, and there's no point in exporting it

Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] firmware_class: avoid double free</title>
<updated>2004-10-20T01:35:07Z</updated>
<author>
<name>Duncan Sands</name>
<email>baldrick@free.fr</email>
</author>
<published>2004-10-20T01:35:07Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=12df409bb719d8ede8abae1c46ec67a92e2400d6'/>
<id>urn:sha1:12df409bb719d8ede8abae1c46ec67a92e2400d6</id>
<content type='text'>
The error exit path in request_firmware frees the allocated struct firmware
*firmware, which is good.  What is not so good is that the value of
firmware has already been copied out to the caller as *firmware_p.  The
risk is that the caller will pass this to release_firmware, a double free. 
This is exactly what will happen if the caller copied the example code

         if(request_firmware(&amp;fw_entry, $FIRMWARE, device) == 0)
                copy_fw_to_device(fw_entry-&gt;data, fw_entry-&gt;size);
         release(fw_entry);

from the firmware documentation.

Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
<entry>
<title>merge</title>
<updated>2004-10-19T01:03:51Z</updated>
<author>
<name>Greg Kroah-Hartman</name>
<email>greg@kroah.com</email>
</author>
<published>2004-10-19T01:03:51Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=bffe01870598b7a0a77073e25ee94e026bc98e6b'/>
<id>urn:sha1:bffe01870598b7a0a77073e25ee94e026bc98e6b</id>
<content type='text'>
</content>
</entry>
<entry>
<title>[PATCH] security issue in firmware system</title>
<updated>2004-10-18T01:22:09Z</updated>
<author>
<name>Oliver Neukum</name>
<email>oliver@neukum.org</email>
</author>
<published>2004-10-18T01:22:09Z</published>
<link rel='alternate' type='text/html' href='https://git.stealer.net/cgit.cgi/user/sven/linux.git/commit/?id=eb51a678c95f0c23dbf6f7b2934c60e11a33531f'/>
<id>urn:sha1:eb51a678c95f0c23dbf6f7b2934c60e11a33531f</id>
<content type='text'>
The firmware loader has a security issue.  Firmware on some devices can
write to all memory through DMA.  Therefore the ability to feed firmware
to the kernel is equivalent to writing to /dev/kmem.  CAP_SYS_RAWIO is
needed to protect itself.

[ Editors note: the firmware file is 0644, and owned by root, so this
  "security issue" is really only an issue for people who use
  capabilities explicitly, rather than the regular Unix permissions.
  This patch makes it do the same checks we do for /dev/mem etc.  ]

Signed-Off-By: Oliver Neukum &lt;oliver@neukum.name&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Adrian Bunk &lt;bunk@stusta.de&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@osdl.org&gt;
</content>
</entry>
</feed>
