diff options
| author |   Dan Carpenter <error27@gmail.com> | 2009-04-08 15:44:04 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2009-04-08 15:44:04 -0700 |
| commit | 58f3e0a864c46dadbeadf682e6bbdcab14ba19d3 (patch) | |
| tree | 7630276b8177b034e1cb79f58be73dbb3199527e /drivers/net/niu.c | |
| parent | mv643xx_eth: don't reset the rx coal timer on interface up (diff) | |
| download | linux-dev-58f3e0a864c46dadbeadf682e6bbdcab14ba19d3.tar.xz linux-dev-58f3e0a864c46dadbeadf682e6bbdcab14ba19d3.zip | |
niu: Fix error handling
platform_device_register_simple() returns ERR_PTR(), not NULL, if an error
occurs.
Found by smatch (http://repo.or.cz/w/smatch.git). Compile tested.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/niu.c')
| -rw-r--r-- | drivers/net/niu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/niu.c b/drivers/net/niu.c index 73cac6c78cb6..2cd6109b8586 100644 --- a/drivers/net/niu.c +++ b/drivers/net/niu.c @@ -9542,7 +9542,7 @@ static struct niu_parent * __devinit niu_new_parent(struct niu *np, plat_dev = platform_device_register_simple("niu", niu_parent_index, NULL, 0); - if (!plat_dev) + if (IS_ERR(plat_dev)) return NULL; for (i = 0; attr_name(niu_parent_attributes[i]); i++) { |