[PATCH] Add subscriber delete command

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 Aug 31 10:42:56 UTC 2014


On Sun, Aug 24, 2014 at 02:02:16PM -0500, Ruben Pollan wrote:

Hi!

> This patch comes from some needs of rhizmoatica. It looks good in our tests, but 
> today it's my first time looking at openbsc code and I might miss something.

sorry, I intendted to reply right-away and then had to rush and
dropped the ball. To make sure your feature continue to work you
should definately add an end-to-end test.

There is one technical issue. The subscriber record is reference
counted. So if you free the memory while someobody else is using
it you will crash.

Third, for machine to machine interaction we have the control
interface which already has a command to delete the subscriber.
Please have a look at

 src/libmsc/ctrl_commands.c:set_subscriber_delete


cheers
	holger





More information about the OpenBSC mailing list