[PATCH 3/5] LC15: fix segfault

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/.

Harald Welte laforge at gnumonks.org
Sun Mar 27 08:40:58 UTC 2016


Hi Max,

regarding the original patch:

* the commit message fails to explain why this fixes a segfault,
  and when that segfault occurs
* if Neels provides some feedback like the one he did, please follow-up
  to that to keep progress on this patch going.

If the OSMO_ASSERT() in the original code was wrong, then please explain
why it was wrong.  This explanation belongs in the commit log message.

Also, as trx_lc15l1_hdl() can now return NULL, did you verify that all
callers of that function can actually deal with a NULL return value?

Regards,
	Harald

-- 
- Harald Welte <laforge at gnumonks.org>           http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
                                                  (ETSI EN 300 175-7 Ch. A6)



More information about the OpenBSC mailing list