diff options
| author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-10-09 16:07:37 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-10-09 16:08:20 +0200 |
| commit | 84bcce6b4f8cbfa16d634d3e77cdd489216d1d2f (patch) | |
| tree | f7016a4c435a2bc41b6f60f26a21490acb5d80c8 | |
| parent | 6ac70a41ee4cc9f3f286fb6b8f397215590fc2ac (diff) | |
gprs-ns/test: Re-add return value check for gprs_nsvc_reset
The return value check has been removed in the commit 'Use
gprs_nsvc_reset instead of gprs_ns_tx_reset', because gprs_nsvc_reset
had no return value then. Since a compatible return value has been
added by 'Let gprs_nsvc_reset return a value', the former patch is
partly reverted by this commit.
The printf is now an OSMO_ASSERT. The commit message was taken
from Jacob.
Sponsored-by: On-Waves ehf
| -rw-r--r-- | tests/gb/gprs_ns_test.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tests/gb/gprs_ns_test.c b/tests/gb/gprs_ns_test.c index 2185ff8a..500b40da 100644 --- a/tests/gb/gprs_ns_test.c +++ b/tests/gb/gprs_ns_test.c @@ -592,6 +592,7 @@ static void test_bss_reset_ack() struct sockaddr_in peer[4] = {{0},}; struct gprs_nsvc *nsvc; struct sockaddr_in *nse[4]; + int rc; peer[0].sin_family = AF_INET; peer[0].sin_port = htons(1111); @@ -641,7 +642,8 @@ static void test_bss_reset_ack() printf("--- Setup VC 2 SGSN -> BSS (hits NSEI 2) ---\n\n"); nsvc = gprs_nsvc_by_nsvci(nsi, 0x2001); - gprs_nsvc_reset(nsvc, NS_CAUSE_OM_INTERVENTION); + rc = gprs_nsvc_reset(nsvc, NS_CAUSE_OM_INTERVENTION); + OSMO_ASSERT(rc < 0); printf("--- Setup VC 1 SGSN -> BSS (hits NSEI 1) ---\n\n"); |