Coverity issues in lapd_core/gsm mncc

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/OpenBSC@lists.osmocom.org/.

Holger Hans Peter Freyther holger at freyther.de
Sun Jul 7 17:44:32 UTC 2013


On Sat, Jul 06, 2013 at 08:43:52PM +0200, Andreas Eversberg wrote:

> this is because openbsc does not do re-establishment in case of a
> data link failure. osmocombb uses resume for assignment (which
> causes the same function as re-establishment), but for this case,
> lapdm.c will provide a zero-length msg.

Hi,

so maybe it simplifies the code just to assume/assert that the msgb
is present in the primitive? As you freed the msgb, this means that
OsmocomBB currently leaks this msgb during the handling of assignment?

thanks
	holger




More information about the OpenBSC mailing list