summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <neels@hofmeyr.de>2017-08-26 22:08:36 +0200
committerHarald Welte <laforge@gnumonks.org>2017-08-29 12:46:44 +0000
commitd157bbb46f5fcf305b0e4710030ecd795432d444 (patch)
tree24c14374b39ade09ad77f4cc9380e6450f060159
parent516910754e51f5a6066f129a0b3c7e214d771b77 (diff)
osmo-auc-gen: umts: properly request --sqn
The user supplied an --sqn to generate a vector for, but milenage_gen_vec() performs a nontrivial SQN increment before generating the vector. To end up with the user supplied SQN, we need to reverse this increment beforehand. Do this after all cmdline args have been parsed, in case one of them modifies the IND-bitlen parameter, which affects the SQN calculations. Related: OS#2464 OS#2465 Change-Id: Ic51a8f6333fee9c02b4073ca360991d0aa69c74f
-rw-r--r--utils/osmo-auc-gen.c22
1 files changed, 17 insertions, 5 deletions
diff --git a/utils/osmo-auc-gen.c b/utils/osmo-auc-gen.c
index fade6ec5..87960aed 100644
--- a/utils/osmo-auc-gen.c
+++ b/utils/osmo-auc-gen.c
@@ -95,9 +95,11 @@ int main(int argc, char **argv)
struct osmo_auth_vector _vec;
struct osmo_auth_vector *vec = &_vec;
uint8_t _rand[16], _auts[14];
+ uint64_t sqn;
int rc, option_index;
int rand_is_set = 0;
int auts_is_set = 0;
+ int sqn_is_set = 0;
int fmt_triplets_dat = 0;
printf("osmo-auc-gen (C) 2011-2012 by Harald Welte\n");
@@ -196,11 +198,8 @@ int main(int argc, char **argv)
fprintf(stderr, "Only UMTS has SQN\n");
exit(2);
}
- test_aud.u.umts.sqn = strtoull(optarg, 0, 10);
- /* Before calculating the UMTS auth vector,
- * osmo_auth_gen_vec() increments the SQN. SQN-1 here
- * to end up with the SQN the user requested. */
- test_aud.u.umts.sqn--;
+ sqn = strtoull(optarg, 0, 10);
+ sqn_is_set = 1;
break;
case 'r':
rc = osmo_hexparse(optarg, _rand, sizeof(_rand));
@@ -246,6 +245,19 @@ int main(int argc, char **argv)
memset(vec, 0, sizeof(*vec));
+ if (test_aud.type == OSMO_AUTH_TYPE_UMTS) {
+ uint64_t seq_1 = 1LL << test_aud.u.umts.ind_bitlen;
+ uint64_t ind_mask = seq_1 - 1;
+
+ if (sqn_is_set) {
+ /* Before calculating the UMTS auth vector, osmo_auth_gen_vec() increments SEQ.
+ * To end up with the SQN passed in by the user, we need to pass in SEQ-1, and
+ * indicate which IND slot to target. */
+ test_aud.u.umts.sqn = sqn - seq_1;
+ test_aud.u.umts.ind = sqn & ind_mask;
+ }
+ }
+
if (!auts_is_set)
rc = osmo_auth_gen_vec(vec, &test_aud, _rand);
else