From: Jee Hyeok Kim <jihyeok13.kim@samsung.com>
Date: Fri, 11 Mar 2016 07:57:26 +0000 (+0900)
Subject: Remove broken validation check logic
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4b8e1aabb93fdb1f4bc4dd0df03d9a62f50a75e0;p=contrib%2Fiotivity.git

Remove broken validation check logic

If requestUri contains host string like 'coap+tcp://127.0.0.1:5683',

This function also adds host length to validate

regardless it is not affect packet size.

Change-Id: Id053ba6959181311d36d26a4b1a2853fb2e1af7d
Signed-off-by: Jee Hyeok Kim <jihyeok13.kim@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/5717
Reviewed-by: Ziran Sun <ziran.sun@samsung.com>
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
---

diff --git a/resource/csdk/stack/src/ocstack.c b/resource/csdk/stack/src/ocstack.c
index 65aaa22..a5b7a66 100644
--- a/resource/csdk/stack/src/ocstack.c
+++ b/resource/csdk/stack/src/ocstack.c
@@ -2471,13 +2471,6 @@ OCStackResult OCDoResource(OCDoHandle *handle,
     char *resourceUri = NULL;
     char *resourceType = NULL;
 
-    // This validation is broken, but doesn't cause harm
-    size_t uriLen = strlen(requestUri );
-    if ((result = verifyUriQueryLength(requestUri , uriLen)) != OC_STACK_OK)
-    {
-        goto exit;
-    }
-
     /*
      * Support original behavior with address on resourceUri argument.
      */