[farsight2/master] Don't force base ip/port, its optional

Olivier Crête olivier.crete at collabora.co.uk
Tue Feb 10 05:38:13 PST 2009


---
 tests/check/transmitter/nice.c                 |   12 ++++++++++--
 transmitters/nice/fs-nice-stream-transmitter.c |   23 -----------------------
 2 files changed, 10 insertions(+), 25 deletions(-)

diff --git a/tests/check/transmitter/nice.c b/tests/check/transmitter/nice.c
index 5593e65..9c45ed7 100644
--- a/tests/check/transmitter/nice.c
+++ b/tests/check/transmitter/nice.c
@@ -73,8 +73,16 @@ _new_local_candidate (FsStreamTransmitter *st, FsCandidate *candidate,
   ts_fail_if (candidate->foundation == NULL,
       "Candidate doenst have a foundation");
   ts_fail_if (candidate->component_id == 0, "Component id is 0");
-  ts_fail_if (candidate->base_ip == NULL, "Candidate doesnt have a base ip");
-  ts_fail_if (candidate->base_port == 0, "Candidate doesnt have a base port");
+  if (candidate->type == FS_CANDIDATE_TYPE_HOST)
+  {
+    ts_fail_if (candidate->base_ip != NULL, "Host candidate has a base ip");
+    ts_fail_if (candidate->base_port != 0, "Host candidate has a base port");
+  }
+  else
+  {
+    ts_fail_if (candidate->base_ip == NULL, "Candidate doesnt have a base ip");
+    ts_fail_if (candidate->base_port == 0, "Candidate doesnt have a base port");
+  }
   ts_fail_if (candidate->username == NULL, "Candidate doenst have a username");
   ts_fail_if (candidate->password == NULL, "Candidate doenst have a password");
 
diff --git a/transmitters/nice/fs-nice-stream-transmitter.c b/transmitters/nice/fs-nice-stream-transmitter.c
index cb3c66d..2d46969 100644
--- a/transmitters/nice/fs-nice-stream-transmitter.c
+++ b/transmitters/nice/fs-nice-stream-transmitter.c
@@ -697,29 +697,6 @@ fs_nice_stream_transmitter_set_remote_candidates (
       return FALSE;
     }
 
-
-    if (self->priv->compatibility_mode == NICE_COMPATIBILITY_DRAFT19)
-    {
-      if (candidate->type == FS_CANDIDATE_TYPE_HOST &&
-          (candidate->base_ip || candidate->base_port))
-      {
-        g_set_error (error, FS_ERROR, FS_ERROR_INVALID_ARGUMENTS,
-            "Must not include the base addr/port in host candidates");
-        return FALSE;
-      }
-
-      if ((candidate->type == FS_CANDIDATE_TYPE_SRFLX ||
-              candidate->type == FS_CANDIDATE_TYPE_PRFLX ||
-              candidate->type == FS_CANDIDATE_TYPE_RELAY ) &&
-          (!candidate->base_ip || !candidate->base_port))
-      {
-        g_set_error (error, FS_ERROR, FS_ERROR_INVALID_ARGUMENTS,
-            "Must include the base addr/port in server/peer reflexive and"
-            " relay candidates");
-        return FALSE;
-      }
-    }
-
     if (!candidate->username)
     {
       g_set_error (error, FS_ERROR, FS_ERROR_INVALID_ARGUMENTS,
-- 
1.5.6.5



More information about the farsight-commits mailing list