Fix firmware loader uevent buffer NULL pointer dereference
commite8780dac8cd1f84f7c31c68bb87c18b6141afca7
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 9 Jul 2015 18:20:01 +0000 (9 11:20 -0700)
committerSasha Levin <sasha.levin@oracle.com>
Tue, 4 Aug 2015 18:43:38 +0000 (4 14:43 -0400)
treecf3460667111ea488e4b2ebba7390d29bfae429c
parent3e35337a4ba284ce6a338ee0de8c0fbb2932bc2c
Fix firmware loader uevent buffer NULL pointer dereference

[ Upstream commit 6f957724b94cb19f5c1c97efd01dd4df8ced323c ]

The firmware class uevent function accessed the "fw_priv->buf" buffer
without the proper locking and testing for NULL.  This is an old bug
(looks like it goes back to 2012 and commit 1244691c73b2: "firmware
loader: introduce firmware_buf"), but for some reason it's triggering
only now in 4.2-rc1.

Shuah Khan is trying to bisect what it is that causes this to trigger
more easily, but in the meantime let's just fix the bug since others are
hitting it too (at least Ingo reports having seen it as well).

Reported-and-tested-by: Shuah Khan <shuahkh@osg.samsung.com>
Acked-by: Ming Lei <ming.lei@canonical.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
drivers/base/firmware_class.c