Actions
Bug #4138
closedlogging_vty_add_cmds parameter remove mess
Start date:
07/31/2019
Due date:
% Done:
100%
Spec Reference:
Description
<pespin> amazing mess. Apparently since March 15th 2017, libosmocore API logging_vty_add_cmds() had its parameter removed, but all our osmocom processes still pass a parameter to it nowadays. Even tests/logging/logging_vty_test.c passes a param to it!.. <pespin> how is it possible that compiler doesn't warn about that? I'm really confused <pespin> it was removed in libosmocore c65c5b4ea075ef6cef11fff9442ae0b15c1d6af7 <tnt> pespin: because it's declared as "void logging_vty_add_cmds();" and not "void logging_vty_add_cmds(void);" <LaF0rge> tnt: ACK. <pespin> ah good point <LaF0rge> pespin: so do you think that extra argument in R0 could screw up anything in the function body? <pespin> LaF0rge, that's first thing I saw and I'm thinking about that possibility yes <pespin> LaF0rge, I think we need to fix programs to remove the extra param, then use (void) in libosmocore <LaF0rge> pespin: I somehow doubt it, more thinking about some thread-local-storage issue, see my recent updates to the related sysmocom ticket <pespin> it's been more than 2 years, so I guess that's fine compatibility-wise
$ ag logging_vty_add_cmds libosmocore/tests/logging/logging_vty_test.c 249: logging_vty_add_cmds(&log_info); libosmocore/tests/vty/vty_test.c 508: logging_vty_add_cmds(); libosmocore/src/vty/logging_vty.c 76: * You have to call \ref logging_vty_add_cmds from your application 331: NULL, /* cmdstr is dynamically set in logging_vty_add_cmds(). */ 385: NULL, /* cmdstr is dynamically set in logging_vty_add_cmds(). */ 994:void logging_vty_add_cmds() libosmocore/include/osmocom/vty/logging.h 9:void logging_vty_add_cmds(); osmo-pcu/src/pcu_vty.c 1167: logging_vty_add_cmds(cat); osmo-trx/Transceiver52M/osmo-trx.cpp 587: logging_vty_add_cmds(); osmo-sip-connector/src/main.c 136: logging_vty_add_cmds(&mncc_sip_info); osmo-hlr/src/hlr_vty.c 434: logging_vty_add_cmds(cat); libosmo-sccp/stp/stp_main.c 180: logging_vty_add_cmds(&log_info); libosmo-sccp/tests/vty/ss7_asp_vty_test.c 165: logging_vty_add_cmds(&log_info); osmo-sgsn/src/gprs/sgsn_main.c 387: logging_vty_add_cmds(NULL); osmo-sgsn/src/gprs/gb_proxy_main.c 287: logging_vty_add_cmds(NULL); osmo-sgsn/src/gprs/gtphub_main.c 359: logging_vty_add_cmds(NULL); osmo-iuh/src/hnbgw.c 542: logging_vty_add_cmds(&hnbgw_log_info); osmo-bsc/src/osmo-bsc/bsc_vty.c 5258: logging_vty_add_cmds(NULL); osmocom-bb/src/host/layer23/src/mobile/app_mobile.c 443: logging_vty_add_cmds(NULL); osmocom-bb/src/shared/libosmocore/src/vty/logging_vty.c 154: NULL, /* cmdstr is dynamically set in logging_vty_add_cmds(). */ 578:void logging_vty_add_cmds(const struct log_info *cat) osmocom-bb/src/shared/libosmocore/include/osmocom/vty/logging.h 8:void logging_vty_add_cmds(const struct log_info *cat); osmo-ggsn/ggsn/ggsn.c 1270: logging_vty_add_cmds(NULL); osmo-msc/src/osmo-msc/msc_main.c 547: logging_vty_add_cmds(&log_info); openbsc/openbsc/src/osmo-bsc_nat/bsc_nat.c 1631: logging_vty_add_cmds(NULL); openbsc/openbsc/src/osmo-bsc_nat/bsc_nat_vty.c 1437: logging_vty_add_cmds(NULL); openbsc/openbsc/src/osmo-bsc_mgcp/mgcp_main.c 225: logging_vty_add_cmds(NULL); openbsc/openbsc/src/libbsc/bsc_vty.c 4338: logging_vty_add_cmds(NULL); osmo-mgw/src/osmo-mgw/mgw_main.c 273: logging_vty_add_cmds(NULL); osmo-bts/src/common/vty.c 1625: logging_vty_add_cmds(cat); osmo-bts/src/osmo-bts-litecell15/misc/lc15bts_mgr.c 304: logging_vty_add_cmds(&mgr_log_info); osmo-bts/src/osmo-bts-sysmo/misc/sysmobts_mgr.c 273: logging_vty_add_cmds(&mgr_log_info); osmo-bts/src/osmo-bts-oc2g/misc/oc2gbts_mgr.c 291: logging_vty_add_cmds(&mgr_log_info); osmo-pcap/src/osmo_client_main.c 189: logging_vty_add_cmds(&log_info); osmo-pcap/src/osmo_server_main.c 202: logging_vty_add_cmds(&log_info);
Actions