diff options
| author |   Paul Atkins <patkins@graphiant.com> | 2021-11-23 10:51:07 +0000 |
|---|---|---|
| committer |   Neale Ranns <neale@graphiant.com> | 2021-11-29 16:21:27 +0000 |
| commit | b779ca9324a6bd2dd605e578b3ca9a4bb90bfa03 (patch) | |
| tree | 9c9a9c5c914dceaab6aea71376852e09c4fb7fc9 /src/vnet/ipfix-export/flow_api.c | |
| parent | 19f6e260acf1342a773b9d47ef9111c3dab3897b (diff) | |
ipfix-export: don't check the result of pool_get
The code to check the exp is set after the call to pool_get()
is marked as unreachable in coverity. This is becasue if it
fails in pool_get then the it panics. Remove the unreachable code.
Type: fix
Signed-off-by: Paul Atkins <patkins@graphiant.com>
Change-Id: Iabb51b6b6788b6c12ef37c28dde2118c798de831
Diffstat (limited to 'src/vnet/ipfix-export/flow_api.c')
| -rw-r--r-- | src/vnet/ipfix-export/flow_api.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/vnet/ipfix-export/flow_api.c b/src/vnet/ipfix-export/flow_api.c index 4a9b525ac67..0b287335bbf 100644 --- a/src/vnet/ipfix-export/flow_api.c +++ b/src/vnet/ipfix-export/flow_api.c @@ -107,8 +107,6 @@ vl_api_set_ipfix_exporter_t_internal ( if (pool_elts (frm->exporters) >= IPFIX_EXPORTERS_MAX) return VNET_API_ERROR_INVALID_VALUE; pool_get (frm->exporters, exp); - if (!exp) - return VNET_API_ERROR_INVALID_VALUE; } } else |