On 10/28/2017 10:23 AM, SF Markus Elfring wrote:
@@ -227,10 +227,8 @@ static int vxcan_newlink(struct net *net, struct
net_device *dev,
netif_carrier_off(peer);
err = rtnl_configure_link(peer, ifmp);
- if (err < 0) {
- unregister_netdevice(peer);
- return err;
-
>> @@ -227,10 +227,8 @@ static int vxcan_newlink(struct net *net, struct
>> net_device *dev,
>> netif_carrier_off(peer);
>> err = rtnl_configure_link(peer, ifmp);
>> - if (err < 0) {
>> - unregister_netdevice(peer);
>> - return err;
>> - }
>> + if (err)
>> +
Hi Markus,
On 10/27/2017 10:30 PM, SF Markus Elfring wrote:
From: Markus Elfring
Date: Fri, 27 Oct 2017 22:22:24 +0200
* Add a jump target so that a bit of exception handling can be better
reused at the end of this function.
* Adjust two condition checks.
This issue was detected by using
From: Markus Elfring
Date: Fri, 27 Oct 2017 22:22:24 +0200
* Add a jump target so that a bit of exception handling can be better
reused at the end of this function.
* Adjust two condition checks.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
dr
