api: fix trace_plugin_msg_ids segv in trace dump 10/38710/2
authorMatthew Smith <mgsmith@netgate.com>
Wed, 19 Apr 2023 20:02:25 +0000 (20:02 +0000)
committerDave Barach <vpp@barachs.net>
Thu, 20 Apr 2023 21:11:41 +0000 (21:11 +0000)
commit5b32d3a13bc66b884bbf25f1409174ed34a8adc7
tree90dbc2c2bf0284d723c12898d9a482624443b126
parentbc4dc16a5ce891649211200f0e346c76cdef4c56
api: fix trace_plugin_msg_ids segv in trace dump

With 'api-trace { on }' in startup.conf, running 'api trace dump' in
vppctl was causing VPP to seg fault. vl_msg_print_trace() was calling
m->endian_handler() without checking whether its null.

Checking if its non-null prevents a crash, but the trace dump prints
the message IDs for trace_plugin_msg_ids in network byte order. There is
an auto-generated endian function for that message. Set it on the call
to vl_msg_api_config() for trace_plugin_msg_ids so the IDs will be
printed in host byte order in trace dump output.

Type: fix
Fixes: fe45f8f5

Signed-off-by: Matthew Smith <mgsmith@netgate.com>
Change-Id: I0ab463985e9a983155feba13ac4eb99ab883ace6
src/vlibmemory/memclnt_api.c
src/vlibmemory/vlib_api_cli.c