[PATCH] libosmocore[master]: gsm/oap: more robust: decode presence as clearly either 1 or 0

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Fri Dec 9 19:30:23 UTC 2016


Hello Harald Welte, Jenkins Builder,

I'd like you to reexamine a change.  Please visit

    https://gerrit.osmocom.org/1378

to look at the new patch set (#3).

gsm/oap: more robust: decode presence as clearly either 1 or 0

Prepares for the unit test, which can now simply check for encode/decode
identity.

Change-Id: I410ff120d868e8d55df9cb19ba6eefd99749cdf0
---
M src/gsm/oap.c
1 file changed, 4 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/78/1378/3

diff --git a/src/gsm/oap.c b/src/gsm/oap.c
index 979bde6..1494a6a 100644
--- a/src/gsm/oap.c
+++ b/src/gsm/oap.c
@@ -92,7 +92,7 @@
 				return -GMM_CAUSE_PROTO_ERR_UNSPEC;
 			}
 			memcpy(oap_msg->autn, value, value_len);
-			oap_msg->autn_present = value_len;
+			oap_msg->autn_present = value_len ? 1 : 0;
 			break;
 
 		case OAP_RAND_IE:
@@ -103,7 +103,7 @@
 				return -GMM_CAUSE_PROTO_ERR_UNSPEC;
 			}
 			memcpy(oap_msg->rand, value, value_len);
-			oap_msg->rand_present = value_len;
+			oap_msg->rand_present = value_len ? 1 : 0;
 			break;
 
 		case OAP_XRES_IE:
@@ -114,7 +114,7 @@
 				return -GMM_CAUSE_PROTO_ERR_UNSPEC;
 			}
 			memcpy(oap_msg->xres, value, value_len);
-			oap_msg->xres_present = value_len;
+			oap_msg->xres_present = value_len ? 1 : 0;
 			break;
 
 		case OAP_AUTS_IE:
@@ -125,7 +125,7 @@
 				return -GMM_CAUSE_PROTO_ERR_UNSPEC;
 			}
 			memcpy(oap_msg->auts, value, value_len);
-			oap_msg->auts_present = value_len;
+			oap_msg->auts_present = value_len ? 1 : 0;
 			break;
 
 		case OAP_CAUSE_IE:

-- 
To view, visit https://gerrit.osmocom.org/1378
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I410ff120d868e8d55df9cb19ba6eefd99749cdf0
Gerrit-PatchSet: 3
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list