diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-04-23 14:01:04 +0200 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2010-05-02 11:04:23 -0400 |
| commit | 2471b894068ec59ab3012e788401b345ef459e49 (patch) | |
| tree | 09a540d9c6f5b0c7e2bd80ad12f3b6864e7d142d /init/do_mounts.c | |
| parent | daa142d1773dd3a986f02a8a4da929608d24daaa (diff) | |
[SCSI] pm8001: potential null dereference in pm8001_dev_gone_notify()
In the original code we dereferenced "pm8001_dev" before checking if it
was null. This patch moves the dereference inside the condition.
This was found by a static checker (smatch). I looked, but I couldn't
tell if "pm8001_dev" dev was ever actually null. The approach in this
patch seemed like the safest response.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Jack Wang <jack_wang@usish.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'init/do_mounts.c')
0 files changed, 0 insertions, 0 deletions