Change in ...osmo-msc[master]: libmsc/paging.c: cosmetic: actually use default branch of switch

laforge gerrit-no-reply at lists.osmocom.org
Sat Jun 15 18:42:35 UTC 2019


laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-msc/+/14469 )

Change subject: libmsc/paging.c: cosmetic: actually use default branch of switch
......................................................................

libmsc/paging.c: cosmetic: actually use default branch of switch

Change-Id: I9b566885f722a28816760532b645f606fdf4faeb
---
M src/libmsc/paging.c
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/src/libmsc/paging.c b/src/libmsc/paging.c
index 94406ad..667acd1 100644
--- a/src/libmsc/paging.c
+++ b/src/libmsc/paging.c
@@ -69,11 +69,9 @@
 	case OSMO_RAT_EUTRAN_SGS:
 		return sgs_iface_tx_paging(vsub, sgs_serv_ind_from_paging_cause(pr->cause));
 	default:
-		break;
+		LOG_PAGING(vsub, pr, LOGL_ERROR, " Cannot page, subscriber not attached\n");
+		return -EINVAL;
 	}
-
-	LOG_PAGING(vsub, pr, LOGL_ERROR, " Cannot page, subscriber not attached\n");
-	return -EINVAL;
 }
 
 struct paging_request *paging_request_start(struct vlr_subscr *vsub, enum paging_cause cause,

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-msc/+/14469
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: I9b566885f722a28816760532b645f606fdf4faeb
Gerrit-Change-Number: 14469
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190615/1899bfa5/attachment.html>


More information about the gerrit-log mailing list