gsmmap: Dump SYSTEM INFORMATION messages while processing

Fixed broken gsmmap.

Merge commit '1a99df84e12315c63f2e96a2972864e4c311712d'

gsm/gsm48_ie: Fix frequency list decoding

Fix-by: Andreas.Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fix off-by-one error in OSMO_NUM_DLIB

This bug was introduced in 95f7eb288c4b8b69d61fa8d68957fb21f09e11e5 and
it caused a segfault on 'write terminal'

use the 'L' prefix in debug names (DLINP instead of DINP)

layer23: Add mobile support for sending / receiving voice frame through MNCC

Support GSM FR codec only so far.

Written-by: Andreas Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/prim_tch: Add support for sending frame from l23 (TRAFFIC_REQ/CONF)

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/prim_tch: Add support for sending traffic frames to l23 (TRAFFIC_IND)

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/layer1/l23_api: Add processing of the new audio_mode

This currently just stores it into the l1 sync state struct

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

l23/l1ctl: Add default values for audio_mode

Theses value should simulate the old behavior

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

l1ctl: Add definitions for the audio_mode configuration

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/l1: Add a queue for traffic frame to send to network (TRAFFIC_REQ)

Also hard limit to maximum 4 pending frames (should not happen !), the
upstream is supposed to do its own flow control.

Written-by: Andreas Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

l1ctl: Add definition for TRAFFIC_{REQ,CONF,IND}

Also adapt packet creation length in L1

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

layer23: Adapt to new logging API: fix calls to log_init

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

Merge commit '430be849945688ae107b079db1e216329b1a1f06'

add definition of OSMO_MIN macro, similar to OSMO_MAX

sms: Fix style issues with the code, add spaces after keywords

sms: SMS where cropped (from VTY), concatenation of SMS where not possible

Additionally it wasn't possible to send concatenated sms from the vty.
To send multiple sms, it is necessary to use padding bits and add a user_data_header.
Therefore the gsm_7bit_encode function was splitted to gsm_7bit_encode and gsm_septets2octets.
gsm_septets2octets: this is the old gsm_7bit_encode function + additional padding parameter

Additionally the gsm_7bit_decode function was modified to take account for the user_data_header.
With the new gsm_get_octet_len function you can get the octet length for a given septet length.

I also added several sms tests.

vty: Increase the number of arguments one can have in VTY commands

For some interactive commands (e.g. sending a SMS in OpenBSC) we
are using the "NAME." arg definition. For the usecase of sending SMS
it is not enough to only allow about 13 spaces. Increase the limit.

Original patch is coming from Dennis Wehrle.

don't call a function argoment 'class' for the benefit of c++ users

This had broken lcr compilation...

ipaccess: Reserve an extensions for an external SMSC connection

This could be used to submit SMS to OpenBSC and to get informed
about new SMS coming from OpenBSC.

remove DLRSL and DLNM, we don't need them

They are not used anywhere in our libraries, so they should be defined
by the respective applications

make sure we don't have namespace clashes between libraries and apps

* All loging prefixes in libraries should be DL like DLINP
* All signals / subsystems should be called S_L_* SS_L_*
* All command nodes should be called L_*_NODE

This makes sure existinc code still compiles as expected

mkae the new 'void *dst' member part of a union with the trx pointer

this ensures struct msgb is the same size as before, which will ensure
binary compatibility

vty: reserve nodes for libosmo-abis

This patch reserves E1INP_NODE and IPA_NODE for libosmo-abis
VTY interfaces.

msgb: add ->dst attribute to store routing information

This patch adds a new attribute to msgb objects. This new
attribute will store a pointer to the routing information,
in the case of the A-bis link, this will be used to store
e1inp_sign_link instead of struct gsm_bts_trx.

This deprecates msg->trx, that can be removed once all
application are ported to use msg->dst.

logging: add libosmo-abis logging subsystems

This adds the libosmo-abis logging subsystems to libosmocore,
it uses the new change that harald proposed based on negative
numbers for library logging subsystems.

signal: reserve signal subsystems >= INT_MAX/2 for libraries

The first range (from 0 to INT_MAX/2) of signal subsystems can be
used by client applications while the second range (from INT_MAX)
is reserved for libraries.

This is useful to support signals defined in libraries that
may be used by different client applications.

[mobile] Added missing curly braces, as proposed by Harald

[layer23] Added special return value for invalid MCC/MNC input

This way an MNC of 000 can be entered.

misc: Put git-version-gen into the tarball

autoreconf does not work on released tarballs due the
missing git-version-gen. Add it.

[layer23] Correctly report to restart mobile instance after band change

Mobile instance must be restarted (shutdown / no shutdown) in order to
apply changes to the available ARFCNs.

[layer23] Fixes and improvements of system information decoding

fw/apps: Remove manual gain control with keyboard

 - It's broken by the use of compute_gain
 - Since there is now an AGC loop, manually setting the register
   as no effect.

If someone needs manual gain control for testing, he'll have to
re-implement a proper AGC override.

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

[layer23/mobile] Improvement and fixes of idle mode process

This patch introduces cell re-relection. When camping on a cell, it
scanns neighbour cells. If a 'better' cell is found, the cell is selected.
If the cell is in a different location area, a location upating is
performed under certain conditions.

The 'better' cell depends on various informations that are broadcasted on
the BCCH of a neihbour cell and of course the RX level. Most operators
don't set these informations, so the 'better' cell depend on a better
RX level for the same location area, or a much better RX level (6 dBm)
at a different location area.

There were many issues at the idle mode process that has been fixed.
Expecially when moving, the state machines got stuck, so no more cell search
was possible, or no further calls / location updating was possible.

In order to see the process of cell selection, enter the VTY interface and
enable the network monitor:

enable
monitor network 1  (where '1' is the instance of the MS)

In order to see the current state of the processes, enter:

show ms

[mobile] Corrected output value of PCS power class

[mobile] Adding option to hide most default values in config

In order to shrink config output, a new config option "[no] hide-default"
is added. The config is now easier to read, but does not show all available
options, if they are set to default. Newcommers may want to see all
available options in config. Therefore all options are show by default.

To hide default options, enter:

configure terminal
hide-default
end

[mobile] Adding PCS band to list of maximum ARFCNs to scan in one band

[mobile] Adding security warning, if default IMEI is not changed

[layer23] Adding "neighbour cell" logging category: DNB

[layer23] Detach SIM, if reading fails

This way the mobility management and cell selection process continues as
if no SIM has been inserted.

[layer1] Adding neighbour cell measurement code to layer1.

When listening to BCCH, layer1 may measure the power level of neighbour
cells. A list of neighbour cell frequencies need to be sent to layer1.
After the measurement is done, the results are indicated to layer23.

[layer23] Adding neighbour cell measurement to L1CTL interface.

[rf] Adding rffe_set_gain() and rffe_get_gain() to get/set computed gain

rffe_compute_gain() is the new name for rffe_set_gain(). I needed to change
this, to solve the name collision with the rffe_set_gain() function, which
actually sets the absolute gain.

rffe_get_gain() will now read the absolute gain which has been computed by
rffe_compute_gain() or set by rffe_set_gain().

[layer23] Fixed handling of channel 0 in certain loops

The ARFC counts from 1 to 1023, and then to 0. The index of these loops
count from 1 to 1024. The index 1024 stands for ARFCN 0.

This also reverses commit eb77945e162ae1968242009372e448b7af51ea50.

tpu_window: Fix int16_t overflow in tpu_window calculation

First we add 55500 to an int16_t, then later we subtract it again.
The bug only didn't become apparent as we wrap twice, once adding
then subtracting.

Discovered by Smatch:
firmware/layer1/tpu_window.c +127 l1s_rx_win_ctrl(24) warn: value 55000 can't fit into 32767 'stop'

calypso/uart.c: Fix array bounds checking

Found by Smatch:
calypso/uart.c +433 uart_baudrate(7) error: buffer overflow 'divider' 7 <= 7

calypso/irq: Fix array bounds checking

Found by Smatch:
calypso/irq.c +200 irq_register_handler(5) error: buffer overflow 'irq_handlers' 32 <= 32

settings.h: IMSI is 15 bytes +1 \0 maximum

If we use a larger field to store the IMSI, we can create overflows when
copying the imsi to other structures that are only 16 bytes in size.

Detected by Smatch:
src/host/layer23/src/mobile/subscriber.c +195 gsm_subscr_testcard(39) error: strcpy() 'set->test_imsi' too large for 'subscr->imsi' (20 vs 16)

sysinfo.h: ARFCN can be 0..1024, so we need an array of 1025 entries

Detected by Smatch:
src/host/layer23/src/mobile/gsm48_rr.c +1658 gsm48_new_sysinfo(23) warn: buffer overflow 's->freq' 1024 <= 1024

gsm48_rr: Don't overflow array boundary

Detected by Smatch:
src/host/layer23/src/mobile/gsm48_rr.c +3021 gsm48_rr_render_ma(89) warn: buffer overflow 'cd->freq_seq_lv' 10 <= 10
src/host/layer23/src/mobile/gsm48_rr.c +3023 gsm48_rr_render_ma(91) error: buffer overflow 'cd->freq_seq_lv' 10 <= 10

osmocon: if we ignore a return value, we don't need to assign it to a variable

gsm 04.80: fix typo (256, not 265!)

This was introduced by me in commit
79599ba0eb7a9aeaeb9dfbbf4881f0633724a471

Thanks to Alexander Huemer for detecting it

osmocon: get rid of some Smash warnings

gsm 04.80: ussd_request.text can contain 0xff, needs to be unsigned

Found by Smatch

lapdm: fix memory leak due to unreachable code

detected by Smatch

gsm 08.08: use ANSI function definition with (void)

tlv: Make tlv parser arrays 256 entries wide to prevent overflow on 0xff

If we encounter a tag with 0xFF, we overflow our existing tlv_parse
array definitions.

Warning: this breaks ABI

gprs_cipher_core: Fix potential buffer overflows

detected by Smatch

telnet_interface: if we don't check for the return value, don't use ret

telnet_interface: get rid of 'const' warning

get rid of non-ANSI function declarations missing (void)

Detected by Smatch

don't use signed bit-fields

detected by Smatch

timer: use (void) for functions that take no arguments

This has been detected by http://smatch.sourceforge.net/

Revert various debian related changes

The debian related changes were not meant to be comitted and
were local experiments. The changes should go through Jan and
not this way.

Revert "more changes"
This reverts commit 96ba20cb44e602abb292b53c9e4f92aa59998df1.

Revert "debian: Updates to fix lintian errors"
This reverts commit b670c86edd704d45b572b87111f3224abecc4767.

Revert "debian: Switch to the 3.0(quilt) format to play with launchpad"
This reverts commit 7a91d6071d1ef15e5a05f5feb7f79a06c279c673.

Revert "debian: Add new changelog entry for the shlib:Depends change"
This reverts commit 54b8c4e0edb2f0f0cde7d28c1715810dd5c6a4b1.

Revert "debian: dpkg-buildpackage complains about this here"
This reverts commit 4284307ead92f35e844e452c0ff9c73a61038096.

Revert "debian: Add a new changelog entry"
This reverts commit ae9b220d166dac277a9b9078add20650415d1cb2.

misc: Fix crash in cell_log due missing l1_prim_cb

Commit 3538c38835b9120d7cf062fa533f7657768bf0d3 introduced the
l1_prim_cb but the init of the misc apps were not updated, make
it us the generic callback that should restore the previous behavior

osmo_hexdump: Fix segfault when input is too long.

In snprinftf the size is a size_t (unsigned) in case we want
to write more than we have available, len_remain will be < 0.

This was spotted while removing hexdump from simtrace and comparing
it to our implementation.

int snprintf(char *str, size_t size, const char *format, ...);

more changes

debian: Updates to fix lintian errors

debian: Switch to the 3.0(quilt) format to play with launchpad

debian: Add new changelog entry for the shlib:Depends change

debian: dpkg-buildpackage complains about this here

debian: Add a new changelog entry

GSM 08.08: Fix generation of CIPHER MODE REJECT

The message has a total length of 4 octets, so don't allocate only 3 in the
msgb.

gsm 08.08: add value_strings and gsm0808_msg_name() function

fix against corrupted output in parallel logging

In 825607672215b7a12ea6e201a89cd5209f6d657f it was attempted to fix
a bug previously introduced by logging related changes.  The problem
is that a va_list can be corrupted after it has been used once, so
we need to va_copy before each successive use.

And if we copy it, we also need to use the copy, and not the original ;)

lapdm: make sure we flush all queues whenever entering IDLE state

this fixes a memory leak where the final UA would always remain
in memory after a LAPDm entity has been disconnected.

add msgb_set_talloc_ctx() to set the talloc context for msgb allocations

make sure abis_nm <-> osmocom pchan type conversion always works

LAPDm: Uplink SACCH frames use format B, not format B4

LAPDm: When Rx DATA from L1, L1 does not know the SAPI

We have to determine the SAPI ourselves inside the LAPDm header.

Fix the generation of the log_categories string + LAPDM

... I should do more testing :(

Conflicts:

src/logging.c

logging: make sure to add the internal categories, as intended

add LAPDm code from osmocom-bb into libosmocore

logging: introduce library-internal logging categories

We do this by using a trick: library-internal log categories use
negative subsystem numbers, which are converted into positive
array indexes at the time of logging.

library-internal log categories need to be knwo at compile-time,
while application-specified categories now are of unlimited number,
as they are dynamically allocated.

lapdm: fix use-after-free

We cannot msgb_free() the msgb and then later reference msg->l2h!

import gsm0502_calc_paging_group() from openbsc

add some utility functions for paging related calculation (TS 05.02)

add gsm48_number_of_paging_subchannels() function

(from openbsc's rsl_number_of_paging_subchannels)

add MAX() macro as OSMO_MAX

merge process.[ch] with application.[ch]

Merge commit '2a68c7c8cd2eca65c58c0898d0e1716e9d4c3ecd'

gsm/utils: Adding conversion of "mobile power class" to dBm

Written-by: Andreas Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/l1a: Add a message to safely count the length of a txqueue

(that is by locking)

Written-by: Andreas Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/sercomm: Additional locking needed in drv_pull

Credits to Andreas Eversberg for finding this bug after countless
hours of debug and providing initial patch :)

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/sercomm: Fix up the locking using an 'abstraction' layer

Not _that_ abstract but a long is enough to store a ptr if need
be :p

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/comm/msgb: Protect allocation routine with proper locking

Credits to Andreas Eversberg for finding this bug after countless
hours of debug :)

Written-by: Andreas Eversberg <jolly@eversberg.eu>
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/comm/msgb: Really panic (i.e. lockup) if we run out of msgb

Ideally we should only panic in interrupt context. In user
context, we could wait ...

We could also return NULL and let the calling code deal with it
but it's not ready for that yet.

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>

fw/comm: Remove useless include from msgb.c

Left over from the backlight debug code

Signed-off-by: Sylvain Munaut <tnt@246tNt.com>