A fix for a bug reported by Omkar and Vijay:

The check for required uri lenght should consider that it includes the uri and query.

Change-Id: I1ca3e4709de02cb4f5217aba5133b8aa2bc4ee14

diff --git a/csdk/stack/src/ocstack.c b/csdk/stack/src/ocstack.c
index c8cb91e..99111ef 100644 (file)
--- a/csdk/stack/src/ocstack.c
+++ b/csdk/stack/src/ocstack.c
@@ -362,7 +362,7 @@ OCStackResult OCDoResource(OCDoHandle *handle, OCMethod method, const char *requ
             goto exit;
     }
 
-    if(strlen(requiredUri) > MAX_URI_LENGTH)
+    if(strlen(requiredUri) > MAX_URI_LENGTH + MAX_QUERY_LENGTH)
     {
         result = OC_STACK_INVALID_PARAM;
         goto exit;