niu: Fix error handling
authorDan Carpenter <error27@gmail.com>
Wed, 8 Apr 2009 22:44:04 +0000 (15:44 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 8 Apr 2009 22:44:04 +0000 (15:44 -0700)
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>
drivers/net/niu.c

index 73cac6c..2cd6109 100644 (file)
@@ -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++) {