diff options
| author | Vadim Yanitskiy <axilirator@gmail.com> | 2018-07-16 22:05:38 +0000 |
|---|---|---|
| committer | Vadim Yanitskiy <axilirator@gmail.com> | 2018-07-17 05:09:58 +0700 |
| commit | a0eef8d2e89ddd610bd8d8aa7511d444bd6590ca (patch) | |
| tree | d615c3ba2625950585014b9aaf0a0b53fe80b354 /src/host/layer23/src/mobile/gsm322.c | |
| parent | 70a50a33ccb1d2fcf52a46bcabf5ffd8534c883d (diff) | |
Revert "Move from libc random() to osmo_get_rand_id"
It was decided to migrate to osmo_get_rand_id() and use random()
as a fall-back. But there is a critical difference between both
functions: osmo_get_rand_id() fills an input buffer with random
bytes (0x00 - 0xff), while *random() returns a value in range
between 0 and RAND_MAX.
osmo_get_rand_id() was used in a wrong way, so in some cases we
could get a negative value (how about IMEI starting from '-'?),
what isn't expected in many cases and could lead to unexpected
behaviour and segmentation faults...
This reverts commit 6d49b049ee304f1ea0e4801df61e69713b01f0f8.
Change-Id: I7b2a8a5c63cf64360a824926a2219fd7e419b1bb
Diffstat (limited to 'src/host/layer23/src/mobile/gsm322.c')
| -rw-r--r-- | src/host/layer23/src/mobile/gsm322.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/host/layer23/src/mobile/gsm322.c b/src/host/layer23/src/mobile/gsm322.c index ce25cd57..c3485b6a 100644 --- a/src/host/layer23/src/mobile/gsm322.c +++ b/src/host/layer23/src/mobile/gsm322.c @@ -959,9 +959,7 @@ static int gsm322_sort_list(struct osmocom_ms *ms) entries++; } while(entries) { - if (osmo_get_rand_id((uint8_t *) &move, sizeof(move)) != 0) - move = random(); - move = move % entries; + move = random() % entries; i = 0; llist_for_each_entry(temp, &temp_list, entry) { if (rxlev2dbm(temp->rxlev) > -85) { |