Changeset 4955 for pjproject/trunk/pjsip/src/pjsua-lib/pjsua_acc.c

Timestamp:

Oct 27, 2014 7:36:08 AM (10 years ago)

Author:

nanang

Message:

Fix #1801:

• put error check in re-registration attempt in pjsua_acc_modify(),
• updated pjsua_acc_modify() docs about its behavior regarding unregistration and re-registration.

File:

• pjproject/trunk/pjsip/src/pjsua-lib/pjsua_acc.c (modified) (2 diffs)

pjproject/trunk/pjsip/src/pjsua-lib/pjsua_acc.c

@@ -1334 +1334 @@
     /* Unregister first */
     if (unreg_first) {
-        pjsua_acc_set_registration(acc->index, PJ_FALSE);
+        status = pjsua_acc_set_registration(acc->index, PJ_FALSE);
+        if (status != PJ_SUCCESS) {
+            pjsua_perror(THIS_FILE, "Ignored failure in unregistering the "
+                         "old account setting in modifying account", status);
+            /* Not really sure if we should return error */
+            status = PJ_SUCCESS;
+        }
         if (acc->regc != NULL) {
             pjsip_regc_destroy(acc->regc);
@@ -1351 +1357 @@
     if (update_reg) {
         /* If accounts has registration enabled, start registration */
-        if (acc->cfg.reg_uri.slen)
-            pjsua_acc_set_registration(acc->index, PJ_TRUE);
+        if (acc->cfg.reg_uri.slen) {
+            status = pjsua_acc_set_registration(acc->index, PJ_TRUE);
+            if (status != PJ_SUCCESS) {
+                pjsua_perror(THIS_FILE, "Failed to register with new account "
+                             "setting in modifying account", status);
+                goto on_return;
+            }
+        }
     }
 
     /* Update MWI subscription */
     if (update_mwi) {
-        pjsua_start_mwi(acc_id, PJ_TRUE);
+        status = pjsua_start_mwi(acc_id, PJ_TRUE);
+        if (status != PJ_SUCCESS) {
+            pjsua_perror(THIS_FILE, "Failed in starting MWI subscription for "
+                         "new account setting in modifying account", status);
+        }
     }