diff mbox

[PATCHv12,6/8] linix-generic: return error for unsupported pktio calls

Message ID 1446039773-28325-7-git-send-email-maxim.uvarov@linaro.org
State Superseded
Headers show

Commit Message

Maxim Uvarov Oct. 28, 2015, 1:42 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 1246bff..41613e3 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -716,7 +716,7 @@  int pktin_poll(pktio_entry_t *entry)
 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) {
@@ -731,7 +731,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;
@@ -740,7 +742,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) {
@@ -756,7 +758,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;
@@ -765,7 +768,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) {
@@ -781,13 +784,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;
@@ -812,7 +815,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;