tree 828f5ae1c6311a7e7cbd44c6763c795edf9c65d4
parent de392254ff05a5304ef0bbd351314d74b2ffd1e3
author Holger Hans Peter Freyther <holger@moiji-mobile.com> 1459534863 +0200
committer Holger Hans Peter Freyther <holger@moiji-mobile.com> 1459970393 +0200

subscr: Make db_create_subscriber fail on duplicates

The issue of db_create_subscriber updating an already existing subscr
is that the same subscriber will then have two entries in the active
subscribers list. In general this will break assumptions that a subscr
can be compared by comparing the pointer.

In the case of the VTY this was not an issue as the created subscr
was immediately destroyed again but it is better to avoid this problem.

Change the VTY command to find the subscriber and then call sync to
have the updated time set. The side-effect is we will now have two
queries for the subscriber. Once through subscr_get_by_imsi and once
through db_create_subscriber.

Change the db_create_subscriber to fail if a subscriber already exists,
and add a testcase for this behavior and do not updated the 'updated'
timestamp of an already existing subscriber.

Add a testcase for this behavior.

Related: OS Issue #1657