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)

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.

misc: Put git-version-gen into the tarball

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

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

This was introduced by me in commit
79599ba0eb7a9aeaeb9dfbbf4881f0633724a471

Thanks to Alexander Huemer for detecting it

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.

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.

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]

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>

sysinfo: add EXTENDED MEASUREMENT ORDER / MEASUREMENT INFO to SI types

those two are used in the RSL SACCH FILLING messages and delivered
like SI5 / SI6.

RSL: Fix typo

add value_string definitions for RSL message types

introduce GSM primitive definitions

The idea here is to use the osmocom core primitive code ot abstract out
primitives for inter-layer comunication in GSM.

add new 'prim.h' header file for primitives

socket: add OSMO_SOCK_F_[CONNECT|BIND|NON_BLOCK] flags

This extends the socket infrastructure in libosmocore to allow
to create non-blocking sockets.

Basically, it replaces the connect0_bind1 parameter by one
flags parameter.

Merge branch 'master' of git.osmocom.org:libosmocore

add missing #include file to sysinfo.h

(otherwise 'struct value_string' would not be defined)

abis_nm.h: Add include to gsm_utils.h

socket: getaddrinfo(): set AI_PASSIVE if we want to bind

This will tell getaddrinfo() that we want a INADDR_ANY style socket

abis_nm: Forward declare the used enums

This file is using enums in that might or might not be
known to the compiler. Forward declare them.

gsmtap_util: Remove msg_free from the wq call back

write_queue already frees the message after the callback is called.

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

gsm/sysinfo: Fix rsl2sitype array size

0xff is the maximum value ... so there is 256 elements.

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

gsm/sysinfo: don't include netinet/in.h

bitvec: add missing stdint.h

socket: remove unused IPv4 related header includes

socket: Skip ifa's without ifa->ifa_addr

Apparently getifaddrs() returns ifa's without an ifa_addr set.

Merge branch 'master' of git.osmocom.org:libosmocore

Import abis_nm_{chcomb4pchan,pchan4chcomb}() from openbsc

libosmocore: rename configure.in -> configure.ac

rename autoconf input file to align with osmocom-bb host applications,
and because that suffix is preferred nowadays.

libosmocore: add missing AC_LANG_SOURCE for current autoconf

autoconf >=2.68 demands the body of an AC_COMPILE_IFELSE to be wrapped in
an AC_LANG_SOURCE macro, otherwise outputs annoying warnings on the
invocation of `autoreconf -i`. this patch follows that demand.

Import sytem information related definitions + code from openbsc

abis_nm: Mark the structs as extern to avoid compiler warning

warning: array ‘abis_nm_obj_class_names’ assumed to have one element

abis_nm: remove abis_nm_obj_class_name / abis_nm_adm_state_name

and export the underlying raw value_string arrays instead:
abis_nm_obj_class_names / abis_nm_adm_state_names.

This permits the caller to use get_string_value() as well as
get_value_string().

rename abis_nm_adm_name() to abis_nm_adm_state_name()

Merge branch 'master' of git.osmocom.org:libosmocore

abis_nm: import definitions and common code on A-bis OML from OpenBSC

libosmogsm: add ipaccess related header file

socket: use listen() and SO_REUSEADDR, new osmo_sock_init_ofd() function

osmo_sock_init_ofd() is a wrapper around osmo_sock_init() which will
take care of initializing and registering a 'struct osmo_fd' for the
newly-created socket.

remove debug printf from socket.c

gsmtap: deal with apps that call gsmtap_send*() with NULL gsmtap_inst

vty: print actual application name rather than always OpenBSC on connect

GSMTAP/socket code: Check for sys/socket.h and conditionally compile

gsmtap: rework GSMTAP API to be more future-proof

* use write_queue where applicable
* provide functions that work on raw FD and those with osmo_fd
* add support for multiple gsmtap instances (no global variables)

logging: fix corrupted output

Harald reported a problem in the logging:
http://lists.osmocom.org/pipermail/openbsc/2011-May/002896.html

Reverting 81e9636454294ae10ef9bc8bf149dd0248afce76 seems to
fix the problem. However, that workaround looks ugly.

Holger gives us another clue on what was wrong:
http://lists.osmocom.org/pipermail/openbsc/2011-May/002905.html

While digging in the manpage, I found this:

"The functions vprintf(), vfprintf(), vsprintf(), vsnprintf()
are equivalent to the functions printf(), fprintf(), sprintf(),
snprintf(), respectively, except that they are called with a
va_list instead of a variable number of arguments. These functions
do not call the va_end macro. Consequently, the value of ap is
undefined after the call. The application should call va_end(ap)
itself afterwards."

logging: fix corrupted output

Harald reported a problem in the logging:
http://lists.osmocom.org/pipermail/openbsc/2011-May/002896.html

Reverting 81e9636454294ae10ef9bc8bf149dd0248afce76 seems to
fix the problem. However, that workaround looks ugly.

Holger gives us another clue on what was wrong:
http://lists.osmocom.org/pipermail/openbsc/2011-May/002905.html

While digging in the manpage, I found this:

"The functions vprintf(), vfprintf(), vsprintf(), vsnprintf()
are equivalent to the functions printf(), fprintf(), sprintf(),
snprintf(), respectively, except that they are called with a
va_list instead of a variable number of arguments. These functions
do not call the va_end macro. Consequently, the value of ap is
undefined after the call. The application should call va_end(ap)
itself afterwards."

app: Introduce some routines to help with application startup

The plan is to collect structs and routines for application
setup and remove many copies of the boilerplate code we have
right now. This starts with routines to ignore certain signals
and the stderr init code.

Increment the age of the library because a new interface was added.

update debian changelog to 0.3.0

update include paths in debian packaging spec

libosmocore: bump library interface version to '1' for new osmo_ names

msgfile: use namespace prefix osmo_* and use more descriptive names

Summary of changes:

s/msg_entry/osmo_config_entry/g
s/msg_entries/osmo_config_list/g
s/msg_entry_parse/osmo_config_list_parse/g

minor glitch included in this patch while I was at it:

-#include "linuxlist.h"
+#include <osmocom/core/linuxlist.h>

plugin: use namespace prefix osmo_*

Summary of changes:

s/plugin_load_all/osmo_plugin_load_all/g

backtrace: use namespace prefix osmo_*

Summary of changes:

s/backtrace/osmo_backtrace/g

crc: use namespace prefix osmo_*

Summary of changes:

s/crc16_table/osmo_crc16_table/g
s/crc16/osmo_crc16/g
s/crc16_byte/osmo_crc16_byte/g

utils: use namespace prefix osmo_*

Summary of changes:

s/bcd2char/osmo_bcd2char/g
s/char2bcd/osmo_char2bcd/g
s/hexparse/osmo_hexparse/g
s/hexdump/osmo_hexdump/g
s/hexdump_nospc/osmo_hexdump_nospc/g
s/ubit_dump/osmo_ubit_dump/g
s/static_assert/osmo_static_assert/g

statistics: use namespace prefix osmo_counter*

Summary of changes:

s/struct counter/struct osmo_counter/g
s/counter_inc/osmo_counter_inc/g
s/counter_get/osmo_counter_get/g
s/counter_reset/osmo_counter_reset/g
s/counter_alloc/osmo_counter_alloc/g
s/counter_free/osmo_counter_free/g
s/counters_for_each/osmo_counters_for_each/g