diff options
| author | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-12 11:40:06 +0200 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-14 07:05:26 +0200 |
| commit | 15be01d4fac9c761411f23903df5c0d6c2360424 (patch) | |
| tree | 47c2b8eb5c807a74ee7b27003b7aa69cc2b43f1d | |
| parent | baf3ed0c64135af4a21d21ba4d1ca7c6f534ba29 (diff) | |
sms: fix error return when a mandatory parameter missing in dictionary
| -rw-r--r-- | src/mm-sms.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/mm-sms.c b/src/mm-sms.c index f9e9b251..1bedfde8 100644 --- a/src/mm-sms.c +++ b/src/mm-sms.c @@ -1438,7 +1438,7 @@ mm_sms_new_from_properties (MMBaseModem *modem, guint n_parts; MMSmsEncoding encoding; - /* Don't create SMS from properties if either text or number is missing */ + /* Don't create SMS from properties if either (text|data) or number is missing */ if (!mm_sms_properties_get_number (properties) || (!mm_sms_properties_get_text (properties) && !mm_sms_properties_get_data (properties, NULL))) { @@ -1448,6 +1448,7 @@ mm_sms_new_from_properties (MMBaseModem *modem, "Cannot create SMS: mandatory parameter '%s' is missing", (!mm_sms_properties_get_number (properties)? "number" : "text' or 'data")); + return NULL; } split_text = mm_sms_part_util_split_text (mm_sms_properties_get_text (properties), |