mirror of https://gerrit.osmocom.org/libosmocore
Sylvain Munaut
71fd42fede
From the mail:
---
appended is another patch for fixing a bug in the calculation of the
frequency lists. This time the patch is for the "Range 256 format".
The problem is that the operand for the "smod" operation might be
negative, in this case the simplified version won't work as expected.
In the patch I introduced a separate function for "smod" which takes
care of the sign. I have not yet checked if the other formats are also
affected, this would be the case if the "smod" operand can be negative.
There might be other solutions to fix the problem without the need
for a separate function, however I have not thought further about it.
A test vector is the following frequency list ("Range 256 format",
first byte is the length):
09 8b 1c 83 8c 15 ef 02 2d 30
The correct ARFCNs are
569 571 576 578 586 608 712 715 719
The uncorrected version would instead return:
444 457 460 464 569 576 578 586 608
This means four ARFCNs are wrong which will cause problems if for
example the frequency list contains the ARFCNs for hopping.
----
Written-by: Dieter Spaar <spaar@mirider.augusta.de>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
|
||
|---|---|---|
| .. | ||
| codec | ||
| gsm | ||
| vty | ||
| Makefile.am | ||
| application.c | ||
| backtrace.c | ||
| bits.c | ||
| bitvec.c | ||
| conv.c | ||
| crc16.c | ||
| gsmtap_util.c | ||
| logging.c | ||
| logging_syslog.c | ||
| msgb.c | ||
| msgfile.c | ||
| panic.c | ||
| plugin.c | ||
| rate_ctr.c | ||
| select.c | ||
| signal.c | ||
| socket.c | ||
| statistics.c | ||
| talloc.c | ||
| timer.c | ||
| utils.c | ||
| write_queue.c | ||