laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-e1d/+/30021 )
Change subject: avoid segfault in 'show interface' of absent icE1usb ......................................................................
avoid segfault in 'show interface' of absent icE1usb
Don't segfault when processing a "show interfaces" for an interface that is configured via VTY but not actually attached/found in the system:
usb.c:693:29: runtime error: member access within null pointer of type 'struct e1_usb_intf_data'
Depends: https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30019 Depends: https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/30020 Change-Id: I2ce990bb57f6ae4edb3a99a4b7bf26a49f362410 --- M src/usb.c 1 file changed, 6 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-e1d refs/changes/21/30021/1
diff --git a/src/usb.c b/src/usb.c index fc461c4..ff624ea 100644 --- a/src/usb.c +++ b/src/usb.c @@ -694,6 +694,12 @@
OSMO_ASSERT(intf->drv == E1_DRIVER_USB);
+ if (!id) { + /* This can happen for statically configured devices (config/vty) which are not + * currently present */ + return snprintf(buf, len, "unknown"); + } + return snprintf(buf, len, "mode=%s, fix=%s, state=%s antenna=%s, tune=%u/%u, freq_est=%u", get_value_string(ice1usb_gpsdo_mode_str, last_st->mode), last_st->valid_fix ? "TRUE" : "FALSE",
-
laforge