This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-commitlog@lists.osmocom.org/.
gitosis at osmocom.org gitosis at osmocom.orgThis is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "An utility library for Open Source Mobile Communications".
The branch, neels/timer_cb2 has been created
at f230123b2b39cf265cf1b42d2ec63c25dbfed5a4 (commit)
- Log -----------------------------------------------------------------
http://cgit.osmocom.org/libosmocore/commit/?id=f230123b2b39cf265cf1b42d2ec63c25dbfed5a4
commit f230123b2b39cf265cf1b42d2ec63c25dbfed5a4
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Sat Nov 18 23:07:12 2017 +0100
add fsm timer_cb2 with T arg
Change-Id: Ie2fce9a62bc0d3e96f1cde2255505f33f1a7e0a2
http://cgit.osmocom.org/libosmocore/commit/?id=9488d4d54054538ea1d2d805f236dd21a668bbd3
commit 9488d4d54054538ea1d2d805f236dd21a668bbd3
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Fri Nov 17 00:15:44 2017 +0100
sercomm_test: sanitize: fix msgb mem leak
This should fix the last current remaining sanitizer build failure in
libosmocore regression tests.
Helps fix sanitizer build on debian 9.
Change-Id: I4d6dd7f4348675bc77d4df5a7a0ce41f12d4a043
http://cgit.osmocom.org/libosmocore/commit/?id=66ee5a28b78e2c02186d51e527bdfd58853f082d
commit 66ee5a28b78e2c02186d51e527bdfd58853f082d
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 23:34:33 2017 +0100
tlv_put: guard against NULL val and 0 len
For example encode_auth_info() from gsup.c calls
msgb_tlv_put(msg, iei, 0, NULL)
to put a tag and len with content data following later.
However, this would cause a memcpy() from a NULL pointer, in tlv_put(). Allow
passing NULL and len = 0 for cases like the above:
If val is NULL, use memset(0) instead of memcpy().
If len is zero, do not copy nor memset anything.
Hence make tlv_put() behave in a well-defined and valid way for any and all
input args; no negative fallout is possible from this patch.
Add proper API doc comment.
Fixes a sanitizer build failure in gsup_test:
../../../../src/libosmocore/include/osmocom/gsm/tlv.h:99:2: runtime error: null pointer passed as argument 2, which is declared to never be null
Helps fix sanitizer build on debian 9.
Change-Id: I13dce9cd1228817890d3e81edeeb660c893c1d64
http://cgit.osmocom.org/libosmocore/commit/?id=7a4e2acb94f9223bd2b13dffbbea20b38b3f37cb
commit 7a4e2acb94f9223bd2b13dffbbea20b38b3f37cb
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 22:31:57 2017 +0100
bssgp_fc_test: sanitize: free fc struct when done
Helps fix sanitizer build on debian 9.
Change-Id: I0ef95ee8185a4789f0732b9420243dda5104d181
http://cgit.osmocom.org/libosmocore/commit/?id=0a08423fe76e949830c3e57c5dc6a0706dd99c8d
commit 0a08423fe76e949830c3e57c5dc6a0706dd99c8d
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 22:32:36 2017 +0100
gprs_bssgp: bssgp_fc_in(): fix mem leak on queue overflow
All successful and all error code paths of bssgp_fc_in() free the msgb, except
the code path calling fc_enqueue() when the msg is dropped (due to queue being
full, or failure to allocate).
Callers could theoretically catch the -ENOSPC return value and discard the
msgb. However, in other code paths, a callback's return value is returned,
which is expected to free the msgb, so such callback would have to never return
-ENOSPC when it freed the msgb. Much simpler semantics would be to free the
msgb in every code path, no matter which kind of error occurred.
Who is currently calling bssgp_fc_in and how do they handle the return value?
- bssgp_fc_test.c ignores the return value (and hits a mem leak aka sanitizer
build failure if the queue is full).
- fc_timer_cb() ignores the return value.
- bssgp_tx_dl_ud() returns the bssgp_fc_in() rc.
- which is returned by a cascade of functions leading up to being returned,
for example, by gprs_llgmm_reset(), which is usually called with ignored
return code.
At this point it is already fairly clear that bssgp_fc_in() should always free
the msgb, since the callers don't seem to distinguish even between error or
success, let alone between -ENOSPC or other errors.
bssgp_fc_test: assert that no msgbs remain unfreed after the tests.
Adjust expected results.
Helps fix sanitizer build on debian 9.
Change-Id: I00c62a104baeaad6a85883c380259c469aebf0df
http://cgit.osmocom.org/libosmocore/commit/?id=cdad4f6385494fe75b7e6226d29042ac2fe9f86e
commit cdad4f6385494fe75b7e6226d29042ac2fe9f86e
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 22:32:36 2017 +0100
bssgp_fc_test: sanitizer: free msgb context when done
Print remaining msgbs when done, then free the entire tall_msgb_context. To be
able to do that, call msgb_talloc_ctx_init() and use its return value.
A subsequent patch will fix a known mem leak and add assertions for 0b in 1
blocks remaining in the tall_msgb_context.
Helps fix sanitizer build on debian 9.
Change-Id: I67d347ab2642b0bfc27b21b44231a7f3146ff641
http://cgit.osmocom.org/libosmocore/commit/?id=b8fa9fd6cfb9e7d5d7dc010657b2850e9ecbd69a
commit b8fa9fd6cfb9e7d5d7dc010657b2850e9ecbd69a
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 22:55:02 2017 +0100
bssgp_fc_test: clarify by outputting ok / failure messages
The test fills up the queue / sends too large PDUs on purpose. Make that
obvious by outputting returned errors in the expected output.
Cosmetic:
- fc_in()'s return value is ignored, hence don't return anything.
- add comment.
Change-Id: I57d6fce2515a65f6dd037e75af5397079215cb46
http://cgit.osmocom.org/libosmocore/commit/?id=fd7aa1bbbf83ef8ca7871430d2b82db7a4c02035
commit fd7aa1bbbf83ef8ca7871430d2b82db7a4c02035
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 22:29:38 2017 +0100
bssgp_fc_test: remove rounding to deciseconds
Ever since this test was changed to use osmo_gettimeofday_override, the times
it sees are exact every time and don't need rounding to pass the expected
output.
Change-Id: I4a9a5d31fc02eb55caf7ba9c141426d8115bb740
http://cgit.osmocom.org/libosmocore/commit/?id=b15d6aea96dfdd6e529c7fdd53434d27453f60ef
commit b15d6aea96dfdd6e529c7fdd53434d27453f60ef
Author: Neels Hofmeyr <neels at hofmeyr.de>
Date: Thu Nov 16 18:31:57 2017 +0100
rate_ctr: fix mem leak for mangled desc
Using the NULL context creates mem leaks that bother sanitizer builds.
Allocate as talloc "child" of the rate_ctr_group, so that the mangled desc (if
any) gets freed when the rate_ctr group is freed.
Remove the comment concerning osmo-msc: one way to fix if the talloc becomes
unexpected from this patch is to adjust the talloc expectations, another
(preferred) way is to have no invalid rate counter names in osmo-msc.
Change-Id: Ief9abfeb78b7706200bcc6aaa5dcb04fbeaa9b5b
-----------------------------------------------------------------------
hooks/post-receive
--
An utility library for Open Source Mobile Communications