pespin has submitted this change. ( https://gerrit.osmocom.org/c/osmo-remsim/+/39022?usp=email )
Change subject: Fix uninitialized return value ...................................................................... Fix uninitialized return value Caught by gcc 14.2.1: /osmo-remsim/src/client/user_simtrace2.c:153:16: warning: ‘rc’ may be used uninitialized [-Wmaybe-uninitialized] 153 | return rc; | ^~ /home/pespin/dev/sysmocom/git/osmo-remsim/src/client/user_simtrace2.c:128:13: note: ‘rc’ was declared here 128 | int rc; | ^~ Change-Id: If90892ff6e00be38cea6f6d3161bf2386351bb21 --- M src/client/user_simtrace2.c 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/client/user_simtrace2.c b/src/client/user_simtrace2.c index 3bc32e2..65d4cde 100644 --- a/src/client/user_simtrace2.c +++ b/src/client/user_simtrace2.c @@ -143,6 +143,7 @@ break; case SIMTRACE_MSGT_BD_CEMU_CONFIG: /* firmware confirms configuration change; ignore */ + rc = 0; break; default: LOGCI(ci, LOGL_ERROR, "unknown simtrace msg type 0x%02x\n", sh->msg_type); -- To view, visit https://gerrit.osmocom.org/c/osmo-remsim/+/39022?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: osmo-remsim Gerrit-Branch: master Gerrit-Change-Id: If90892ff6e00be38cea6f6d3161bf2386351bb21 Gerrit-Change-Number: 39022 Gerrit-PatchSet: 2 Gerrit-Owner: pespin <pes...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <lafo...@osmocom.org> Gerrit-Reviewer: osmith <osm...@sysmocom.de> Gerrit-Reviewer: pespin <pes...@sysmocom.de>