aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPau Espin Pedrol <pespin@sysmocom.de>2018-11-05 13:44:12 +0100
committerPau Espin Pedrol <pespin@sysmocom.de>2018-11-05 13:44:14 +0100
commit91650b0b4c680ac6f4b71c00b6dc151bce92851a (patch)
treea16693e37edd90d570218f81c91d15a80fd8a57f
parentf2d2fef39faebcb73570ccdab41bdfef855c7e3b (diff)
bsc: gsm0808: Fix return code in bsc_compl_l3
complete_layer3 returns true if everything succeeded, false otherwise. However, its caller bsc_compl_l3 returns unix style (0 sucess, negative error). This commit has no real effect since only caller of bsc_compl_l3 never checks return code, but will check it in the future. Change-Id: I722696c3f6402288b51d6fcf51f478b3b0c9f0f0
-rw-r--r--src/osmo-bsc/gsm_08_08.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/osmo-bsc/gsm_08_08.c b/src/osmo-bsc/gsm_08_08.c
index 19c259844..7e5b909da 100644
--- a/src/osmo-bsc/gsm_08_08.c
+++ b/src/osmo-bsc/gsm_08_08.c
@@ -382,7 +382,7 @@ int bsc_compl_l3(struct gsm_subscriber_connection *conn, struct msgb *msg, uint1
return -1;
}
- return complete_layer3(conn, msg, msc);
+ return complete_layer3(conn, msg, msc) ? 0 : -1;
}
static int handle_page_resp(struct gsm_subscriber_connection *conn, struct msgb *msg)