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