keith uploaded patch set #2 to this change.

View Change

The following approvals got outdated and were removed: Verified+1 by Jenkins Builder

sqlite optimisation: check VLR earlier for dest. subscriber

sms_from_result() calls a number of sqlite3_column_XXX functions
to build the SMS in memory.

If the destination subscriber is not attached, the sms
will be free'd immediatly anyway in smsq_take_next_sms()
so let's check if the destination subscriber is present
and attached before we call all the sqlite3 routines.

Change-Id: Ibd07b9d200b48108d705ed4c461cc4ddfa421fd2
---
M src/libmsc/db.c
M src/libmsc/sms_queue.c
M tests/sms_queue/sms_queue_test.c
M tests/sms_queue/sms_queue_test.ok
4 files changed, 26 insertions(+), 39 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/21/41221/2

To view, visit change 41221. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: newpatchset
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Change-Id: Ibd07b9d200b48108d705ed4c461cc4ddfa421fd2
Gerrit-Change-Number: 41221
Gerrit-PatchSet: 2
Gerrit-Owner: keith <keith@rhizomatica.org>
Gerrit-Reviewer: Jenkins Builder