projects
/
openwrt
/
staging
/
blogic.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
32f7f89
)
fsi: sbefifo: Fix checker warning about late NULL check
author
Benjamin Herrenschmidt
<
[email protected]
>
Thu, 12 Jul 2018 01:53:37 +0000
(11:53 +1000)
committer
Benjamin Herrenschmidt
<
[email protected]
>
Thu, 12 Jul 2018 01:53:37 +0000
(11:53 +1000)
"dev" is dereferences before it's checked.
Reported-by: Dan Carpenter <
[email protected]
>
Signed-off-by: Benjamin Herrenschmidt <
[email protected]
>
drivers/fsi/fsi-sbefifo.c
patch
|
blob
|
history
diff --git
a/drivers/fsi/fsi-sbefifo.c
b/drivers/fsi/fsi-sbefifo.c
index a34ff997ca9d42f833556fa0cff3f0b1c6d5311c..6b31cc24fb0d01fe15b6e6618db5c463d79e377e 100644
(file)
--- a/
drivers/fsi/fsi-sbefifo.c
+++ b/
drivers/fsi/fsi-sbefifo.c
@@
-706,13
+706,16
@@
static int __sbefifo_submit(struct sbefifo *sbefifo,
int sbefifo_submit(struct device *dev, const __be32 *command, size_t cmd_len,
__be32 *response, size_t *resp_len)
{
- struct sbefifo *sbefifo
= dev_get_drvdata(dev)
;
+ struct sbefifo *sbefifo;
struct iov_iter resp_iter;
struct kvec resp_iov;
size_t rbytes;
int rc;
- if (!dev || !sbefifo)
+ if (!dev)
+ return -ENODEV;
+ sbefifo = dev_get_drvdata(dev);
+ if (!sbefifo)
return -ENODEV;
if (WARN_ON_ONCE(sbefifo->magic != SBEFIFO_MAGIC))
return -ENODEV;