From 0bb01ab68d36e63d08f9ed827524a24829c26835 Mon Sep 17 00:00:00 2001 From: Ben Chan Date: Wed, 2 Aug 2017 21:53:32 -0700 Subject: 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. --- src/mm-sms-part-cdma.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src') 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; } -- cgit v1.2.3-70-g09d2