Attention is currently required from: laforge, fixeria. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30613 )
Change subject: layer23: fix rx_l1_sim_conf(): msg->l2h is NULL, use msg->l1h ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File src/host/layer23/src/common/l1ctl.c:
https://gerrit.osmocom.org/c/osmocom-bb/+/30613/comment/13dee7dc_ef434020 PS1, Line 742: LOGP(DL1C, LOGL_INFO, "SIM %s\n", osmo_hexdump(data, len)); Independent of this patch, I think it's really worth having a special case inside osmo_hexdump to print "(null)" if the pointer is NULL. What do you think? do you want to submit it?