From 710f0035b388cb892712b239bac0111bc552403d Mon Sep 17 00:00:00 2001
From: Stefan Sayer <stefan.sayer@googlemail.com>
Date: Fri, 18 Jan 2008 00:32:22 +0000
Subject: [PATCH] some fixes on de-registering with multiple contacts.

git-svn-id: http://svn.berlios.de/svnroot/repos/sems/trunk@604 8eb893ce-cfd4-0310-b710-fb5ebe64c474
---
 apps/registrar_client/ContactInfo.cpp        | 7 +++++--
 apps/registrar_client/SIPRegistrarClient.cpp | 6 ++++++
 2 files changed, 11 insertions(+), 2 deletions(-)

diff --git a/apps/registrar_client/ContactInfo.cpp b/apps/registrar_client/ContactInfo.cpp
index 007e3007..91dbe9fe 100644
--- a/apps/registrar_client/ContactInfo.cpp
+++ b/apps/registrar_client/ContactInfo.cpp
@@ -177,7 +177,10 @@ bool ContactInfo::parse_uri() {
   size_t p1 = 0; 
   int eq = 0; const char* sip_prot = "SIP:";
   uri_user = ""; 	uri_host = ""; uri_port = ""; uri_param = ""; 
-	
+
+  if (uri.empty())
+    return false;
+
   while (pos<uri.length()) {
     char c = uri[pos];
     //    DBG("(1) c = %c, st = %d\n", c, st);
@@ -291,7 +294,7 @@ bool ContactInfo::parse_uri() {
   case uS4: uri_port = uri.substr(p1+1, pos-p1-1); break;
   case uS5: uri_param = uri.substr(p1+1, pos-p1-1); break;
   case uS0:
-  case uS1: { DBG("ERROR while parsing uri"); return false; } break;
+  case uS1: { DBG("ERROR while parsing uri\n"); return false; } break;
   };
   return true;
 }
diff --git a/apps/registrar_client/SIPRegistrarClient.cpp b/apps/registrar_client/SIPRegistrarClient.cpp
index fe9a7540..82122a82 100644
--- a/apps/registrar_client/SIPRegistrarClient.cpp
+++ b/apps/registrar_client/SIPRegistrarClient.cpp
@@ -343,10 +343,14 @@ void SIPRegistration::onSipReply(AmSipReply& reply) {
 
     if (!contacts.length()) {
       DBG("received positive reply to de-Register \n");
+      active = false;
       remove = true;
     } else {
       end = 0;
       while (!found) {
+	if (contacts.length() == end)
+	  break;
+
 	if (!server_contact.parse_contact(contacts, end, end)) {
 	  ERROR("while parsing contact\n");
 	  break;
@@ -386,7 +390,9 @@ void SIPRegistration::onSipReply(AmSipReply& reply) {
 									   req.from_tag,
 									   reply.code, reply.reason));
       }
+      DBG("no matching Contact - deregistered.\n");
       active = false;
+      remove = true;
     }
 		
   } else if (reply.code >= 300) {