net: bridge: set error code on failure
authorPan Bian <bianpan2016@163.com>
Sat, 3 Dec 2016 11:33:23 +0000 (19:33 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 5 Dec 2016 18:26:22 +0000 (13:26 -0500)
Function br_sysfs_addbr() does not set error code when the call
kobject_create_and_add() returns a NULL pointer. It may be better to
return "-ENOMEM" when kobject_create_and_add() fails.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188781

Signed-off-by: Pan Bian <bianpan2016@163.com>
Acked-by: Stephen Hemminger <stephen@networkplumber.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_sysfs_br.c

index e120307c6e36ab1173232ee1d4fb1730f5dbae60..f88c4df3f91efc1464a3db29ef961714a2f40506 100644 (file)
@@ -898,6 +898,7 @@ int br_sysfs_addbr(struct net_device *dev)
        if (!br->ifobj) {
                pr_info("%s: can't add kobject (directory) %s/%s\n",
                        __func__, dev->name, SYSFS_BRIDGE_PORT_SUBDIR);
+               err = -ENOMEM;
                goto out3;
        }
        return 0;