Commit b9db942b authored by Stephen Hemminger's avatar Stephen Hemminger

[PATCH] remove dev_get from wanrouter

The call to dev_get() in wanrouter_device_new_if is racy and redundant and should
be removed.  The later 'register_netdev()' does the same test internally and will
return the appropriate error if the name already exists.

This patch is against 2.6.0-test6.
Resend of earlier patch because it was ignored, or missed.
parent 6e634db3
......@@ -726,8 +726,6 @@ static int wanrouter_device_new_if(struct wan_device *wandev,
if (dev->name == NULL) {
err = -EINVAL;
} else if (dev_get(dev->name)) {
err = -EEXIST; /* name already exists */
} else {
#ifdef WANDEBUG
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment