Review at https://gerrit.osmocom.org/5183
sms db: don't attempt to query pending SMS for unset MSISDN
When the subscriber has no MSISDN, we might construct an invalid SQL statement
such as
... AND dest_addr= AND ...
Instead, don't even query for empty MSISDNs.
Related: OS#2706
Change-Id: I7d6169d774b2da04b3051957e364fe620feed51e
---
M src/libmsc/db.c
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/83/5183/1
diff --git a/src/libmsc/db.c b/src/libmsc/db.c
index 4e4477b..ca27b6a 100644
--- a/src/libmsc/db.c
+++ b/src/libmsc/db.c
@@ -840,6 +840,10 @@
if (!vsub->lu_complete)
return NULL;
+ /* A subscriber having no phone number cannot possibly receive SMS. */
+ if (*vsub->msisdn == '\0')
+ return NULL;
+
result = dbi_conn_queryf(conn,
"SELECT * FROM SMS"
" WHERE sent IS NULL"
--
To view, visit https://gerrit.osmocom.org/5183
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I7d6169d774b2da04b3051957e364fe620feed51e
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <[email protected]>
