diff mbox

[API-NEXT,PATCHv20,7/9] linux-generic: return error for unsupported pktio calls

Message ID 1455109705-5871-8-git-send-email-maxim.uvarov@linaro.org
State Accepted
Commit 3b9d7a41920c093f087a68883c9f36ceb5c298ae
Headers show

Commit Message

Maxim Uvarov Feb. 10, 2016, 1:08 p.m. UTC 
For some pktios like loop and ipc functions like mtu, promisc,
and mac addr are not applicable. Instead of crash on deference
null pointer just return error if functions are not defined.
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 platform/linux-generic/odp_packet_io.c | 24 ++++++++++++++++--------
 1 file changed, 16 insertions(+), 8 deletions(-)
diff mbox

Patch

diff --git a/platform/linux-generic/odp_packet_io.c b/platform/linux-generic/odp_packet_io.c
index 27b6553..97fea77 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -718,7 +718,7 @@  int pktin_poll(pktio_entry_t *entry, int num_queue, int index[])
 int odp_pktio_mtu(odp_pktio_t id)
 {
 	pktio_entry_t *entry;
-	int ret;
+	int ret = -1;
 
 	entry = get_pktio_entry(id);
 	if (entry == NULL) {
@@ -733,7 +733,9 @@  int odp_pktio_mtu(odp_pktio_t id)
 		ODP_DBG("already freed pktio\n");
 		return -1;
 	}
-	ret = entry->s.ops->mtu_get(entry);
+
+	if (entry->s.ops->mtu_get)
+		ret = entry->s.ops->mtu_get(entry);
 
 	unlock_entry(entry);
 	return ret;
@@ -742,7 +744,7 @@  int odp_pktio_mtu(odp_pktio_t id)
 int odp_pktio_promisc_mode_set(odp_pktio_t id, odp_bool_t enable)
 {
 	pktio_entry_t *entry;
-	int ret;
+	int ret = -1;
 
 	entry = get_pktio_entry(id);
 	if (entry == NULL) {
@@ -762,7 +764,8 @@  int odp_pktio_promisc_mode_set(odp_pktio_t id, odp_bool_t enable)
 		return -1;
 	}
 
-	ret = entry->s.ops->promisc_mode_set(entry, enable);
+	if (entry->s.ops->promisc_mode_set)
+		ret = entry->s.ops->promisc_mode_set(entry, enable);
 
 	unlock_entry(entry);
 	return ret;
@@ -771,7 +774,7 @@  int odp_pktio_promisc_mode_set(odp_pktio_t id, odp_bool_t enable)
 int odp_pktio_promisc_mode(odp_pktio_t id)
 {
 	pktio_entry_t *entry;
-	int ret;
+	int ret = -1;
 
 	entry = get_pktio_entry(id);
 	if (entry == NULL) {
@@ -787,13 +790,13 @@  int odp_pktio_promisc_mode(odp_pktio_t id)
 		return -1;
 	}
 
-	ret = entry->s.ops->promisc_mode_get(entry);
+	if (entry->s.ops->promisc_mode_get)
+		ret = entry->s.ops->promisc_mode_get(entry);
 	unlock_entry(entry);
 
 	return ret;
 }
 
-
 int odp_pktio_mac_addr(odp_pktio_t id, void *mac_addr, int addr_size)
 {
 	pktio_entry_t *entry;
@@ -818,7 +821,12 @@  int odp_pktio_mac_addr(odp_pktio_t id, void *mac_addr, int addr_size)
 		return -1;
 	}
 
-	ret = entry->s.ops->mac_get(entry, mac_addr);
+	if (entry->s.ops->mac_get) {
+		ret = entry->s.ops->mac_get(entry, mac_addr);
+	} else {
+		ODP_DBG("pktio does not support mac addr get\n");
+		ret = -1;
+	}
 	unlock_entry(entry);
 
 	return ret;