add 'ciph_state' member to BTS-side lchan definition

introduce HAVE_TM_GMTOFF_IN_TM

Not all architectures have the tm.tm_gmtoff member.  This fixes cygwin
builds.

make sure we still build on systems that don't have IPPROTO_GRE

nat: Add another test case for TP-SRR clearing and number rewrite

misc: Use other size modifiers to fix compiler warnings

control_if.c:521:2: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘uint64_t’ [-Wformat]
osmo_bsc_bssap.c:473:3: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 7 has type ‘unsigned int’ [-Wformat]
mgcp_main.c:162:4: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘unsigned int’ [-Wformatt]

bsc: Prepare to have multiple MSC connections

We now have a list of MSCs but in the code we will
try to access the MSC with the nr 0.

bsc: Use the right connection for outgoing packets

This is needed for simple UDT messages where we do not have
a SCCP connection.

bsc: Move more things to use osmo_msc_data* directly

bsc: Move the bsc_filter to use the osmo_bsc_sccp_con

bsc: Move away from ->bsc.msc to use the selected MSC

For multiple MSCs we should only have one place where the MSC
is selected and the rest will extract it from somewhere.

bsc: Stop using net->bsc->msc and get the right msc from somewhere else

bsc: Create a osmo_bsc_data and embed osmo_msc_data

We want to have multiple MSCs but we also have some data
that is only present on a per BSC basis. Right now the
MSC data is not allocated with talloc, so we have some
change in the talloc contexts.

bsc: Address warnings about implicit declarations

osmo_bsc_main.c: In function ‘main’:
osmo_bsc_main.c:398:2: warning: implicit declaration of function ‘bts_init’ [-Wimplicit-function-declaration]
osmo_bsc_main.c:399:2: warning: implicit declaration of function ‘libosmo_abis_init’ [-Wimplicit-function-declaration]
osmo_bsc_main.c:418:2: warning: implicit declaration of function ‘bsc_bootstrap_network’ [-Wimplicit-function-declaration]

bsc: Fix compiler warning about wrong return and unused variable

osmo_bsc_api.c: In function ‘bsc_cm_update’:
osmo_bsc_api.c:195:2: warning: ‘return’ with a value, in function returning void [enabled by default]
osmo_bsc_api.c:193:28: warning: unused variable ‘sccp’ [-Wunused-variable]

mgcp: implement a more tolerant parser based on strtok_r()

Instead of building complex manual byte-wise parsers, we simply use two
strtok_r loops:  one iterating over all the lines, the next one
iterating over the invididual space-separated elements in the first line.

The benefit is that we now accept \r, \n or \r\n, or any multiple of
them as line ending.  This works around incompliant MGCP implementations
like that of Zynetix MSC.

Addition: mgcp_analyze_header returns 0 when all out parameters have
been set.

Signed-off-by: Holger Hans Peter Freyther <zecke@selfish.org>

mgcp: make sure all command line options are documented

Signed-off-by: Holger Hans Peter Freyther <zecke@selfish.org>

mgcp: consistently name bsc_mgcp osmo-bsc_mgcp

Signed-off-by: Holger Hans Peter Freyther <zecke@selfish.org>

mgcp_test: Add wrong messages as input and check what happens

Test parsing some invalid messages and see if we start to crash.

mgcp_test: Rename method as it now operates on an array of messages

This code was cleaned up, catch up and rename the method.

misc: Deal with smatch warnings about the bitfields

Use unsigned ints for the bitfield.

misc: Use msgb_free for freeing the messages

Improved generation of SYSTEM INFORMATIONs 2* and 5* for neighbour bands

In addition to SI 2 and SI 5, the SI 2ter and 2bis is generated, if
neighbour cells in other bands exist. Also it is indicated in the rest
octets of SI3, that SI 2ter is used. If no neighbour cell in a different
band exists, the SI 2ter and SI 5ter is omitted.

A special case is P-GSM range (channels 1-124). To be compatible with
older phones, SI 2bis and SI 5bis is used. If the BCCH lays inside the
P-GSM band, only neighbour cells of the P-GSM range are included in
SI 2 and SI 5. If neighbour cells exist in the same band (900), but lay
outside the P-GSM range, the SI 2bis and SI 5bis is used to extend the
list of neighbour cells. The extension is also indicated in SI 2 and
SI 5. If the BCCH lays inside the P-GSM range, but no neighbour cell
exists in the same band outside the P-GSM range, the SI 2bis ans SI 5bis
are omitted.

Make sure USSD code only responds *#100# for retrieving own number

strstr() was used with wrong argument order, causing it to always match,
and causing an invalid response to a variety of different SS and USSD
requests.

This has apparently caused havoc among a number of HTC phones which
issue SS requests without user interaction upon boot, and then trip over
our inappropriate response.

fix generation of frequency list (E-GSM vs. P-GSM 900)

bts_ipaccess_nanobts.c: Remove unused variable.

This must have been obsoleted by the move to libosmo-abis.

GCC warning:
bts_ipaccess_nanobts.c: In function ‘ipaccess_drop_oml’:
bts_ipaccess_nanobts.c:509:21: warning: variable ‘line’ set but not used [-Wunused-but-set-variable]

abis_rsl.c: Remove unused variable net.

The old BSC code had code to override the payload type, this has
been removed, remove the variable accessing it.

GCC warning:
abis_rsl.c: In function ‘ipa_rtp_pt_for_lchan’:
abis_rsl.c:1590:22: warning: unused variable ‘net’ [-Wunused-variable]

abis_rsl: The rach information was not used, introduce a signal

Introduce a SS_CCCH for the paging and the rach load. The paging
code could now start using the signal.

GCC warning:
abis_rsl.c: In function ‘rsl_rx_ccch_load’:
abis_rsl.c:1371:11: warning: variable ‘rach_access_count’ set but not used [-Wunused-but-set-variable]
abis_rsl.c:1370:11: warning: variable ‘rach_busy_count’ set but not used [-Wunused-but-set-variable]
abis_rsl.c:1369:11: warning: variable ‘rach_slot_count’ set but not used [-Wunused-but-set-variable]

abis_rsl.c: Remove unused variable ts_number

GCC warning:
abis_rsl.c: In function ‘rsl_rx_chan_rqd’:
abis_rsl.c:1245:10: warning: variable ‘ts_number’ set but not used [-Wunused-but-set-variable]

abis_om2000.c: Remove unused variable to_trx_oml.

GCC warning:
abis_om2000.c: In function ‘abis_om2k_sendmsg’:
abis_om2000.c:804:6: warning: unused variable ‘to_trx_oml’ [-Wunused-variable]

vty: Remove two unimplemented OML functions

attribute get|set <0-255> (.HEX) was never implemented and the
output about the unused attributes clutter the build output, remove
them.

GCC warning:
abis_nm_vty.c: In function ‘oml_attrib_get’:
abis_nm_vty.c:141:25: warning: unused variable ‘oms’ [-Wunused-variable]
abis_nm_vty.c: In function ‘oml_attrib_set’:
abis_nm_vty.c:152:25: warning: unused variable ‘oms’ [-Wunused-variable]

misc: abis_nm.c Use the result of ret.

Use LOGP(DNM, LOGL_ERROR, ...); for errors in the
abis_nm_rx_sw_act_req method.

GCC warning:
abis_nm.c: In function ‘abis_nm_rx_sw_act_req’:
abis_nm.c:412:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]

misc: gsm_data.c address compiler warning

Assume nothing special needs to be done for the Nokia
*site hardware.

GCC warning:
gsm_data.c: In function ‘gsm_set_bts_type’:
gsm_data.c:342:2: warning: enumeration value ‘GSM_BTS_TYPE_NOKIA_SITE’ not handled in switch [-Wswitch]

ARFCNs are in the range of 0..1023, not 0..1024

I'm sure I read somewhere that it actually was 0..1024, as I kept
wondering how stupid it was to use 10bit+1.  However, that source
was incorrect, as GSM TS 05.05 quite clearly states 0..1023

mgcp: add missing prototypes for send_mgcp_reset_*

osmo-bsc: Remove hard-coded check for codec preferecne in ASS CMD

When we get an assignment command from the MSC, we no longer have to
check statically for certain codecs, as we have a vty-configured list of
codecs that is checked just below.

MGCP: Add VTY commands to reset (RSIP) one or all endpoints

bsc_mgcp: optionally connect the MGCP socket to the MGW

This allows the use of the existing "call-agent ip A.B.C.D" command in
libmgcp in order to set a default destination address of the call agent.

This is required as a pre-condition for certain call agents (like the
zynetix MSC) that require a RSIP from the BSC to the MSC as the initial
MGCP message.

MGCP: Add CRCX test cases

The second test case (CRCX_ZYN) fails as the message is not compliant
with the spec.  However, our parer should be more tolerant, just in
case...

further generalize mgcp testing code

use a data-structure driven approach

mgcp_test: remove duplicate code

Move the bulk of RR processing from MSC into BSC

RR Messages like STATUS, GPRS SUSPEND, HANDOVER COMPLETE/FAIL, ...
should be processed on the BSC side of things, not on the MSC side.

This is among other things required in preparation of intra-BSC
hand-over support in osmo-bsc.

Move processing of CLASSMARK CHANGE behind bsc_api

This prevents osmo-bsc from sending RR messages to the MSC and rather
process them inside the BSC and turn them into BSSAP CM UPDATE.

osmo-bsc: correctly parse the TMSI in BSSAP paging from MSC

the TMSI is a uint32_t big-endian value and not a gsm 04.08 mobile
identity like the IMSI.

nat: Add VTY command for the TP-DA rewriting.

nat: Fix the no sms-clear-tp-srr command

nat: Internationalize the TP-DA if it starts with a 00

Internationalize the number like we do it for Call Control. Update
the test result to match this new behavior.

nat: Prepare to rewrite the TP-DA number of a SMS submit.

Introduce number rewriting of SMS-SUBMIT. Introduce a new list,
move code around to help with finding a new number, somehow the
number encoding for TP-DA is borked, 03.40 references 04.11 but
the length appears to be strlen(number) without taken the type
field into account.

nat: Print the sizes and the offending data in case of a failure

rsl: This is provided by libosmo-abis, remove it here

mgcp: Fix warning about unused variable

mgcp_network.c:335:22: warning: variable ‘cfg’ set but not used [-Wunused-but-set-variable]

mgcp: Provide documentation for the method, check sscanf return

bts: Convert the rsl_link to be a struct ipabis_link

lchan: Close a possible still open RTP Socket

In case of a reset (loss of the BTS) close down all remaining
RTP Proxy sockets. In case of a lchan_free shout if the rtp
proxy is still open. I would prefer if the proxy code sits inside
the gsm subscriber connection.

ho: Remove rtp_proxy.h include, no proxy code in here

nat: Add the bsc_data.c file to the EXTRA_DIST

abis: gsm_data_shared.h includes libosmo-abis header, add abis to cflags

gsm_data_shared.h includes e1_input.h of libosmo-abis, add the
LIBOSMOABIS_CFLAGS to Makefile.am, remove AM_LDFLAGS at the same
time as we only build .a archives.

mncc: Include size and offsets of struct gsm_mncc in the hello

Increase the version number and send the sizeof of the gsm_mncc
and the gsm_data_frame structure. Include the offsets of some
members as well to maybe identify ABI problems.

The reason we go through this is that we want to benefit from
the native ABI and don't want to force packed structs.

channel: Review lchan_free calls and the state the channel is in

In case of a memory allocation failure in rsl_rx_chan_rqd we would
have left the channel in the LCHAN_S_ACT_REQ state. Move the state
change below the allocation.

lchan: Rename release_reason to release_mode

Use the name of GSM 08.58 for the release mode.

channel: Fix a typo in the lchan state documentation

channel: The error timer and the lchan_deact_tmr are both running

In case a BTS does not send the RF Channel Release ACK and we are
closing the channel because of an error two timers are running to
set the state back to none.

Make lchan_deact_tmr_cb and rsl_rx_rf_chan_rel_ack behave the same
in regard to changing the state of the lchan. For the other direction
we are save, the error path will set the state to NONE and the timeout
will call lchan_free to set the channel type back to NONE, only then
the lchan may be allocated again.

The channel release procedure requires some more tweaking, some part
was started in the zecke/28c3 branch and needs to be tested/integrated
with the goal of having one common release path.

Revert "libbsc: Log if a channel is freed with lchan->state != LCHAN_S_NONE"

In case of a failed channel we still want the channel to not be
re-allocated right away but keep it closed/unused for (T3109 + T311).

rsl_rx_rf_chan_rel_ack has a check to not set the channel to S_NONE in
case the channel is in the error state. Add the camp Harald added a timer
to set the channel back to the none state in case the RF Channel Release
is not acked.

This reverts commit fc462dd59ee4772f06c9f2477b36b3f166a8e74b.

bsc: Fix "show lchan" for partially provided information

show lchan should be capable of showing all allocated lchans,
all of a given bts, a given trx, a given ts. This feature was
broken when I added the ability to show a more simple summary.

Restore the initial behavior by splitting out the for loops
for the bts/trx/ts and check if we have parsed all parameters
and then call and return the subroutine.

nat: Add the line break after the '.

nat: Implement clearing of TP-SRR flags from TPDUs

Match IMSI and destination address against a set of entries, if it
is matching the header will be modified and no sender report will be
requested. Change the test case to request the sender report and then
verify that this bit is reset to 0.

nat: Reduce copy and paste in the test, create a verify_msg

nat: Use the link_id from the original message in the new DTAP message

SMS went from SAPI=3 to SAPI=0 and nobody notices on the NAT->MSC direction.

nat: Compare the DTAP L3 size with the size of the msgb - dtap header

nat: Fix comment and mask for looking at TP-MTI

nat: SMS rewrite matching, empty list means match everything

nat: SMS comment updates

nat: Move SMS creation into a new method, no functional change

nat: Access data_ptr after we have checked the length

nat: Move the finding of the number to a new method

tests: Use atlocal/atlocal.in to enable/disable the NAT test

Kill the hacking, use atlocal to remember if the NAT test should
be enabled and then skip the test (exit with 77).

Revert "tests: Make the test setup more abusive to fix the build"

This reverts commit 41b42c72294d181217bae59e37ddf50c490df17f.

tests: Make the test setup more abusive to fix the build

Paolo says I will need to use atconfig/atlocal, need to read up on
it first though.

gprs: Honor GSM 04.64 8.4.2 Receipt of unacknowledged information

GSM 04.64 8.4.2 asks to ignore UI frames if the DLCI is not known,
or if the "(V(UR)- 32) <= N(U) < V(UR)". E.g. if we want to have
V(UR) == 511 and this frame is dropped, we would ignore N(U)'s
0 to 510. Calculate the delta.

The code is based on Jonathan Santos's "LLC UI window" fix but the
issue was discovered independly.

gprs_sgsn.h: Include timer.h for the timer struct

tests: Introduce running tests with GNU autotest in OpenBSC

The quality of the tests is of different value but it is good to
get started and improve from here.

mgcp: Print message when the test is finished

gsm0408: Print message when the test is done

db: Fix the test code

* Initialize logging
* use a dummy net before calling subcr_put as the keep in ram
  option is going to be set.

channel: Make the test not crash again.

* Initialize logging
* The callback expects NULL or a subscriber connection

nat: Log test output stdout using printf in prep for GNU autotest

bsc: Assume assignment_complete/assignment_fail is set

The osmo-nitb application sometimes crashes because the BSC API
is doing an assignment underneath which is not handled by the code,
add dumy handlers to not crash, the right thing to do is to change
MNCC to have an assignment that can succeed/fail.

The keyword to look for is MNCC_LCHAN_MODIFY and mncc_sock should
wait for an ack/nack but right now the call just continues.

RSL: Nokia BTS reports TA in quarter-bits, not full bits.

We need to compensate accordingly...

RSL: BS-11 reports TA in quarter-bits, not full bits.

We need to compensate accordingly...

Add NITZ (timezone) support as part of MM INFO

The UTC offset from the operating system will be used by default to
calculate the NITZ in MM INFO.  However,  a "timezone" vty command is
added at the BTS level, allowing BTS-specific overrides, e.g. in case
BTSs are distributed accross multiple timezones.

fix two mistakes in AM_LDFLAGS

BSSGP: Add functions required for true BSS-side BSSGP implementation

The BSS-side of BSSGP requires quite a number of additional functions
for sending unidirectional messages that a SGSN never sends.

This is a first step into completing the BSSGP implementation and making
it ready to be used from osmo-bts and other BTS-side GPRS
implementations.

BSSGP: add more doxygen comments

GPRS: add new bssgp_create_cell_id(), opposite of bssgp_parse_cell_id()

GPRS NS: Add some doxygen API documentation

bsc_vty: Fix usage of deprecated osmo_osmo_hexdump_nospc

The typo was fixed upstream

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

osmo_nitb: introduce --no-dbcounter command line option

this allows us to disable writing the current rate counters to the
sqlite database every 60 seconds.

gsm0408: Print a message when the LU is timing out.

mncc: Introduce a hello packet that is sent to the client.

Send a hello packet down to the client with the version number
of the MNCC interface. The hello structure might be extended to
include the endianes, size of each structure, etc.

sgsn: Make sure each parameter has some sort of documentation

gb_proxy: Make sure each parameter has some sort of documentation