diff options
| author |   Vratko Polak <vrpolak@cisco.com> | 2019-04-17 20:02:56 +0200 |
|---|---|---|
| committer |   Paul Vinciguerra <pvinci@vinciconsulting.com> | 2019-04-17 18:48:54 +0000 |
| commit | 4a72406360270a4ff0197c9c8cca43366f515aa8 (patch) | |
| tree | 7bea27431c6bb5c02313aa0456959996479c33fc /src/vnet/interface_api.c | |
| parent | 317b8e08367c769b90463613231b9fcfad486098 (diff) | |
Make sw_interface_dump more compatible with 2.2.0
+ As old PAPI clients are likely to put zero as the value
for sw_if_index, the behavior should not perform
index filtering at least when name filtering is enabled (valid).
+ interface.api version set to 2.3.0,
as the new behavior is backward compatible
(at least for PAPI with name filter enabled),
but not forward compatible.
+ Minor whitespace cleanup.
Change-Id: I315a0eae4004f9d9b6c5f9ecf0f179e669729118
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
Diffstat (limited to 'src/vnet/interface_api.c')
| -rw-r--r-- | src/vnet/interface_api.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/vnet/interface_api.c b/src/vnet/interface_api.c index 993d12c0617..8b0cd8e4a33 100644 --- a/src/vnet/interface_api.c +++ b/src/vnet/interface_api.c @@ -308,7 +308,7 @@ vl_api_sw_interface_dump_t_handler (vl_api_sw_interface_dump_t * mp) u8 *filter = 0, *name = 0; sw_if_index = ntohl (mp->sw_if_index); - if (sw_if_index != ~0) + if (!mp->name_filter_valid && sw_if_index != ~0) { /* is it a valid sw_if_index/ */ if (vec_len (im->sw_interfaces) <= sw_if_index) @@ -316,8 +316,6 @@ vl_api_sw_interface_dump_t_handler (vl_api_sw_interface_dump_t * mp) swif = vec_elt_at_index (im->sw_interfaces, sw_if_index); - /* If we have a sw_if_index, ignore the name filter. */ - mp->name_filter_valid = 0; vec_reset_length (name); name = format (name, "%U%c", format_vnet_sw_interface_name, am->vnet_main, |