pdump: initialize snaplen if enabled by device id
Checks
Commit Message
If packet dump was enabled via pdump_enable_by_deviceid
the packet snapshot length was not being set.
Bugzilla ID: 840
Fixes: 10f726efe26c ("pdump: support pcapng and filtering")
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
lib/pdump/rte_pdump.c | 3 +++
1 file changed, 3 insertions(+)
Comments
On Wed, 27 Oct 2021 11:08:47 -0700
Stephen Hemminger <stephen@networkplumber.org> wrote:
> If packet dump was enabled via pdump_enable_by_deviceid
> the packet snapshot length was not being set.
>
> Bugzilla ID: 840
> Fixes: 10f726efe26c ("pdump: support pcapng and filtering")
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
> lib/pdump/rte_pdump.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/lib/pdump/rte_pdump.c b/lib/pdump/rte_pdump.c
> index 71602685d544..3086b2a1688b 100644
> --- a/lib/pdump/rte_pdump.c
> +++ b/lib/pdump/rte_pdump.c
> @@ -631,6 +631,9 @@ pdump_enable_by_deviceid(const char *device_id, uint16_t queue,
> if (ret < 0)
> return ret;
>
> + if (snaplen == 0)
> + snaplen = UINT32_MAX;
> +
> return pdump_prepare_client_request(device_id, queue, flags, snaplen,
> ENABLE, ring, mp, prm);
> }
There appears to be bug in the CI system reporting bogus failure on
this patch.
27/10/2021 20:08, Stephen Hemminger:
> If packet dump was enabled via pdump_enable_by_deviceid
> the packet snapshot length was not being set.
>
> Bugzilla ID: 840
> Fixes: 10f726efe26c ("pdump: support pcapng and filtering")
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Applied, thanks.
@@ -631,6 +631,9 @@ pdump_enable_by_deviceid(const char *device_id, uint16_t queue,
if (ret < 0)
return ret;
+ if (snaplen == 0)
+ snaplen = UINT32_MAX;
+
return pdump_prepare_client_request(device_id, queue, flags, snaplen,
ENABLE, ring, mp, prm);
}