]> granicus.if.org Git - libnl/commitdiff
vxlan: fix error code for missing attribute
authorThomas Haller <thaller@redhat.com>
Sat, 5 Nov 2016 16:32:02 +0000 (17:32 +0100)
committerThomas Haller <thaller@redhat.com>
Sat, 5 Nov 2016 16:35:01 +0000 (17:35 +0100)
Other VXLAN getters wrongly fail with -NLE_AGAIN.
That maybe should be fixed, but would be an API change.

New API should use the correct error code.

lib/route/link/vxlan.c

index 6e65e26f2ed613284b4d0d0520f92d228266d328..9e578ec94baa86d19653f9b99d8ded0cc97d758e 100644 (file)
@@ -1467,7 +1467,7 @@ int rtnl_link_vxlan_get_port(struct rtnl_link *link, uint32_t *port)
                return -NLE_INVAL;
 
        if (!(vxi->ce_mask & VXLAN_ATTR_PORT))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        *port = ntohs(vxi->vxi_port);
 
@@ -1506,7 +1506,7 @@ int rtnl_link_vxlan_get_udp_csum(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_UDP_CSUM))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_udp_csum;
 }
@@ -1543,7 +1543,7 @@ int rtnl_link_vxlan_get_udp_zero_csum6_tx(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_UDP_ZERO_CSUM6_TX))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_udp_zero_csum6_tx;
 }
@@ -1580,7 +1580,7 @@ int rtnl_link_vxlan_get_udp_zero_csum6_rx(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_UDP_ZERO_CSUM6_RX))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_udp_zero_csum6_rx;
 }
@@ -1617,7 +1617,7 @@ int rtnl_link_vxlan_get_remcsum_tx(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_REMCSUM_TX))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_remcsum_tx;
 }
@@ -1654,7 +1654,7 @@ int rtnl_link_vxlan_get_remcsum_rx(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_REMCSUM_RX))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_remcsum_rx;
 }
@@ -1691,7 +1691,7 @@ int rtnl_link_vxlan_get_collect_metadata(struct rtnl_link *link)
        IS_VXLAN_LINK_ASSERT(link);
 
        if (!(vxi->ce_mask & VXLAN_ATTR_COLLECT_METADATA))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        return vxi->vxi_collect_metadata;
 }
@@ -1732,7 +1732,7 @@ int rtnl_link_vxlan_get_label(struct rtnl_link *link, uint32_t *label)
                return -NLE_INVAL;
 
        if (!(vxi->ce_mask & VXLAN_ATTR_LABEL))
-               return -NLE_AGAIN;
+               return -NLE_NOATTR;
 
        *label = ntohl(vxi->vxi_label);