31 Aug
2014
31 Aug
'14
12:42 p.m.
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