diff options
| author | Ben Chan <benchan@chromium.org> | 2017-08-02 21:53:32 -0700 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2017-08-03 12:19:37 +0200 |
| commit | 0bb01ab68d36e63d08f9ed827524a24829c26835 (patch) | |
| tree | e8602d4bbfa6541f50eed373038ef6961ab71884 /src | |
| parent | 6132a4d2ae92e99746f6840bb7330c56c0d8fcbf (diff) | |
sms-part-cdma: add missing break statements in cause_code_to_delivery_state
This patch fixes cause_code_to_delivery_state() by adding two missing
break statements for the case ERROR_CLASS_TEMPORARY and
ERROR_CLASS_PERMANENT in the `switch (error_class)` statement. Without
the break statements, the switch always falls through to the default and
returns MM_SMS_DELIVERY_STATE_UNKNOWN for an `error_class' of value
ERROR_CLASS_TEMPORARY or ERROR_CLASS_PERMANENT.
Diffstat (limited to 'src')
| -rw-r--r-- | src/mm-sms-part-cdma.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mm-sms-part-cdma.c b/src/mm-sms-part-cdma.c index 3aa6b455..8d76bcec 100644 --- a/src/mm-sms-part-cdma.c +++ b/src/mm-sms-part-cdma.c @@ -273,8 +273,10 @@ cause_code_to_delivery_state (guint8 error_class, return MM_SMS_DELIVERY_STATE_COMPLETED_RECEIVED; case ERROR_CLASS_TEMPORARY: delivery_state += 0x300; + break; case ERROR_CLASS_PERMANENT: delivery_state += 0x200; + break; default: return MM_SMS_DELIVERY_STATE_UNKNOWN; } |