@@ -725,7 +725,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) {
@@ -740,7 +740,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;
@@ -749,7 +751,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) {
@@ -769,7 +771,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;
@@ -778,7 +781,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) {
@@ -794,13 +797,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;
@@ -825,7 +828,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;
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(-)