session: Fix RoamingPolicy parsing
authorDaniel Wagner <daniel.wagner@bmw-carit.de>
Fri, 13 May 2011 14:09:54 +0000 (16:09 +0200)
committerDaniel Wagner <daniel.wagner@bmw-carit.de>
Fri, 13 May 2011 14:26:11 +0000 (16:26 +0200)
If the session has an invalid roaming policy and appending a NULL to
the D-Bus message leads to a crash.

src/session.c

index b00f024..b57ef71 100644 (file)
@@ -147,7 +147,7 @@ static const char *roamingpolicy2string(enum connman_session_roaming_policy poli
 {
        switch (policy) {
        case CONNMAN_SESSION_ROAMING_POLICY_UNKNOWN:
-               break;
+               return "unknown";
        case CONNMAN_SESSION_ROAMING_POLICY_DEFAULT:
                return "default";
        case CONNMAN_SESSION_ROAMING_POLICY_ALWAYS: