diff options
| author |   Mohsin Kazmi <sykazmi@cisco.com> | 2022-01-11 11:34:56 +0000 |
|---|---|---|
| committer |   Neale Ranns <neale@graphiant.com> | 2022-01-11 12:47:52 +0000 |
| commit | 30d46e7455b20c46d97ad04f72afce3995e4a2c4 (patch) | |
| tree | cf382caaf9564c44dcf3f028d0ac3f6db06009a7 | |
| parent | ce4083ce48958d9d3956e8317445a5552780af1a (diff) | |
vlib: fix the total len not including first buffer formating
Type: fix
total len not including first buffer is in vlib_buffer_t second cacheline.
It is not reset after the buffer has been consumed. It leads to printing
garbage in packet trace for subsequent use of this buffer. This patch
fixes the issue to only print when VLIB_BUFFER_NEXT_PRESENT flag is set.
Signed-off-by: Mohsin Kazmi <sykazmi@cisco.com>
Change-Id: Ied72308bdb907a5e1ca16d181f2add062807e968
| -rw-r--r-- | src/vlib/buffer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/vlib/buffer.c b/src/vlib/buffer.c index 71f84d377af..7a70a8abdd3 100644 --- a/src/vlib/buffer.c +++ b/src/vlib/buffer.c @@ -107,7 +107,7 @@ format_vlib_buffer_no_chain (u8 * s, va_list * args) "ref-count %u", b->current_data, b->current_length, b->buffer_pool_index, b->ref_count); - if (b->flags & VLIB_BUFFER_TOTAL_LENGTH_VALID) + if (b->flags & VLIB_BUFFER_NEXT_PRESENT) s = format (s, ", totlen-nifb %d", b->total_length_not_including_first_buffer); |