summaryrefslogtreecommitdiffstats
path: root/drivers/scsi/qla4xxx/ql4_init.c
diff options
context:
space:
mode:
authorKevin Hilman <khilman@deeprootsystems.com>2010-05-26 23:30:55 -0700
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2010-05-26 23:34:56 -0700
commit067fb2f648543894ce775082c5636f4c32b99e4f (patch)
treed2725b8748afa732dc017b6c142c830c7802981a /drivers/scsi/qla4xxx/ql4_init.c
parentf2126a99675a930b3ab5258698d82b2a3c69af3f (diff)
downloadtalos-obmc-linux-067fb2f648543894ce775082c5636f4c32b99e4f.tar.gz
talos-obmc-linux-067fb2f648543894ce775082c5636f4c32b99e4f.zip
Input: ads7846 - return error on regulator_get() failure
In probe(), if regulator_get() failed, an error code was not being returned causing the driver to be successfully bound, even though probe failed. This in turn caused the suspend, resume and remove methods to be registered and accessed via the SPI core. Since these functions all access private driver data using pointers that had been freed during the failed probe, this would lead to unpredictable behavior. This patch ensures that probe() returns an error code in this failure case so the driver is not bound. Found using lockdep and noticing the lock used in the suspend/resum path pointed to a bogus lock due to the freed memory. Signed-off-by: Kevin Hilman <khilman@deeprootsystems.com> Acked-by: Mike Frysinger <vapier@gentoo.org> Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
Diffstat (limited to 'drivers/scsi/qla4xxx/ql4_init.c')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud