pespin has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/34328?usp=email ) Change subject: ss7: Drop unneeded else branch after early return ...................................................................... ss7: Drop unneeded else branch after early return Change-Id: I845be6bfbf36ec8a38697ca92420483a9d41aa6c --- M src/osmo_ss7.c 1 file changed, 11 insertions(+), 3 deletions(-) Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved diff --git a/src/osmo_ss7.c b/src/osmo_ss7.c index d097fc9..2ba9766 100644 --- a/src/osmo_ss7.c +++ b/src/osmo_ss7.c @@ -1569,10 +1569,9 @@ LOGP(DLSS7, LOGL_NOTICE, "Default IPv6 address %s not supported: %s\n", host, gai_strerror(rc)); return false; - } else { - freeaddrinfo(result); - return true; } + freeaddrinfo(result); + return true; } /* Set default values for local and remote peer hosts if they are not yet set. -- To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/34328?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-sccp Gerrit-Branch: master Gerrit-Change-Id: I845be6bfbf36ec8a38697ca92420483a9d41aa6c Gerrit-Change-Number: 34328 Gerrit-PatchSet: 1 Gerrit-Owner: pespin <pespin(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged