diff mbox

[ovs-dev] netdev: Initialize netdev's features before getting them

Message ID 20161012210839.9320-1-xu.binbin1@zte.com.cn
State Rejected
Headers show

Commit Message

Xu Binbin Oct. 12, 2016, 9:08 p.m. UTC 
When OVS&DPDK is used, DPDK doesn't support features 'advertised',
'supported' and 'peer'. If a physical port added to bridge, features
descirbed above can't be assigned, and the values are random.

Signed-off-by: Binbin Xu <xu.binbin1@zte.com.cn>
---
 lib/netdev.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
diff mbox

Patch

diff --git a/lib/netdev.c b/lib/netdev.c
index 6c4c657..f40adfd 100644
--- a/lib/netdev.c
+++ b/lib/netdev.c
@@ -957,14 +957,14 @@  netdev_get_features(const struct netdev *netdev,
         peer = &dummy[3];
     }
 
+    *current = *advertised = *supported = *peer = 0;
+
     get_features = netdev->netdev_class->get_features;
     error = get_features
                     ? get_features(netdev, current, advertised, supported,
                                    peer)
                     : EOPNOTSUPP;
-    if (error) {
-        *current = *advertised = *supported = *peer = 0;
-    }
+
     return error;
 }