aboutsummaryrefslogtreecommitdiffstats
path: root/src/vpp-api/python/vpp_papi/vpp_format.py
diff options
context:
space:
mode:
authorOle Troan <ot@cisco.com>2020-06-02 14:38:53 +0200
committerAndrew Yourtchenko <ayourtch@gmail.com>2020-08-11 13:51:59 +0000
commit710fe104620683755de71e2bb5d97c438987df74 (patch)
tree9df125d39f3ec03ab10f7e650d44dee3130f63b0 /src/vpp-api/python/vpp_papi/vpp_format.py
parentfdc678081ca5f0971b8bcbf312c1e83017365c33 (diff)
papi: allow unknown address family
In unions all representations of the union are decoded. Which means trying to decode something that isn't an address might have invalid address family types. Type: fix Ticket: VPP-1884 Signed-off-by: Ole Troan <ot@cisco.com> Change-Id: Id3381ef8cc885952c1eb488ebc70e276eaceb366
Diffstat (limited to 'src/vpp-api/python/vpp_papi/vpp_format.py')
-rw-r--r--src/vpp-api/python/vpp_papi/vpp_format.py29
1 files changed, 19 insertions, 10 deletions
diff --git a/src/vpp-api/python/vpp_papi/vpp_format.py b/src/vpp-api/python/vpp_papi/vpp_format.py
index 4c26463bbb5..261683d5816 100644
--- a/src/vpp-api/python/vpp_papi/vpp_format.py
+++ b/src/vpp-api/python/vpp_papi/vpp_format.py
@@ -57,6 +57,7 @@ def format_vl_api_prefix_t(args):
return {'address': format_vl_api_address_t(p),
'len': int(length)}
+
def format_vl_api_address_with_prefix_t(args):
if isinstance(args, (ipaddress.IPv4Interface, ipaddress.IPv6Interface)):
return {'address': format_vl_api_address_t(
@@ -75,6 +76,7 @@ def format_vl_api_ip6_prefix_t(args):
return {'address': inet_pton(AF_INET6, p),
'len': int(length)}
+
def format_vl_api_ip6_address_with_prefix_t(args):
if isinstance(args, ipaddress.IPv6Interface):
return {'address': args.network_address.packed,
@@ -83,6 +85,7 @@ def format_vl_api_ip6_address_with_prefix_t(args):
return {'address': inet_pton(AF_INET6, p),
'len': int(length)}
+
def format_vl_api_ip4_prefix_t(args):
if isinstance(args, ipaddress.IPv4Network):
return {'address': args.network_address.packed,
@@ -91,6 +94,7 @@ def format_vl_api_ip4_prefix_t(args):
return {'address': inet_pton(AF_INET, p),
'len': int(length)}
+
def format_vl_api_ip4_address_with_prefix_t(args):
if isinstance(args, ipaddress.IPv4Interface):
return {'address': args.network_address.packed,
@@ -144,13 +148,13 @@ conversion_table = {
'vl_api_address_with_prefix_t':
{
'IPv4Interface': lambda o: {'address':
- {'af': ADDRESS_IP4, 'un':
- {'ip4': o.packed}},
- 'len': o.network.prefixlen},
+ {'af': ADDRESS_IP4, 'un':
+ {'ip4': o.packed}},
+ 'len': o.network.prefixlen},
'IPv6Interface': lambda o: {'address':
- {'af': ADDRESS_IP6, 'un':
- {'ip6': o.packed}},
- 'len': o.network.prefixlen},
+ {'af': ADDRESS_IP6, 'un':
+ {'ip6': o.packed}},
+ 'len': o.network.prefixlen},
'str': lambda s: format_vl_api_address_with_prefix_t(s)
},
'vl_api_ip4_address_with_prefix_t':
@@ -162,7 +166,7 @@ conversion_table = {
'vl_api_ip6_address_with_prefix_t':
{
'IPv6Interface': lambda o: {'address': o.packed,
- 'len': o.network.prefixlen},
+ 'len': o.network.prefixlen},
'str': lambda s: format_vl_api_ip6_address_with_prefix_t(s)
},
'vl_api_mac_address_t':
@@ -183,14 +187,15 @@ def unformat_api_address_t(o):
return ipaddress.IPv6Address(o.un.ip6)
if o.af == 0:
return ipaddress.IPv4Address(o.un.ip4)
- raise ValueError('Unknown address family {}'.format(o))
+ return None
+
def unformat_api_prefix_t(o):
if o.address.af == 1:
return ipaddress.IPv6Network((o.address.un.ip6, o.len), False)
if o.address.af == 0:
return ipaddress.IPv4Network((o.address.un.ip4, o.len), False)
- raise ValueError('Unknown address family {}'.format(o))
+ return None
if isinstance(o.address, ipaddress.IPv4Address):
return ipaddress.IPv4Network((o.address, o.len), False)
@@ -198,19 +203,23 @@ def unformat_api_prefix_t(o):
return ipaddress.IPv6Network((o.address, o.len), False)
raise ValueError('Unknown instance {}', format(o))
+
def unformat_api_address_with_prefix_t(o):
if o.address.af == 1:
return ipaddress.IPv6Interface((o.address.un.ip6, o.len))
if o.address.af == 0:
return ipaddress.IPv4Interface((o.address.un.ip4, o.len))
- raise ValueError('Unknown address family {}'.format(o))
+ return None
+
def unformat_api_ip4_address_with_prefix_t(o):
return ipaddress.IPv4Interface((o.address, o.len))
+
def unformat_api_ip6_address_with_prefix_t(o):
return ipaddress.IPv6Interface((o.address, o.len))
+
conversion_unpacker_table = {
'vl_api_ip6_address_t': lambda o: ipaddress.IPv6Address(o),
'vl_api_ip6_prefix_t': lambda o: ipaddress.IPv6Network((o.address, o.len)),