From: Inga Stotland Date: Mon, 18 Apr 2016 22:41:52 +0000 (-0700) Subject: [IOT-963] Fix maximum URI length X-Git-Tag: accepted/tizen/common/20160603.150732 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Ftags%2Faccepted%2Ftizen%2Fcommon%2F20160603.150732;p=platform%2Fupstream%2Fiotivity.git [IOT-963] Fix maximum URI length According to core spec, maximum URI length could be up to 256 Change-Id: Ic9037fcca8a6ef49b0695ac5d3515f84aa11c773 Signed-off-by: Inga Stotland Reviewed-on: https://gerrit.iotivity.org/gerrit/7821 Tested-by: jenkins-iotivity Reviewed-by: Habib Virji --- diff --git a/resource/csdk/stack/include/ocstackconfig.h b/resource/csdk/stack/include/ocstackconfig.h index 1bab0fc..8680c7f 100644 --- a/resource/csdk/stack/include/ocstackconfig.h +++ b/resource/csdk/stack/include/ocstackconfig.h @@ -33,7 +33,7 @@ * Maximum length of the URI supported by client/server while processing * REST requests/responses. */ -#define MAX_URI_LENGTH (64) +#define MAX_URI_LENGTH (256) /** * Maximum length of the query supported by client/server while processing diff --git a/resource/csdk/stack/test/stacktests.cpp b/resource/csdk/stack/test/stacktests.cpp index 037edcd..a5b9fe4 100644 --- a/resource/csdk/stack/test/stacktests.cpp +++ b/resource/csdk/stack/test/stacktests.cpp @@ -453,7 +453,7 @@ TEST(StackResource, CreateResourceBadUri) OIC_LOG(INFO, TAG, "Starting CreateResourceBadUri test"); InitStack(OC_SERVER); - const char *uri65 = "ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKL"; + const char *uri257 = "ABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVWXYZABCDEFGHIJKLMNOPQRSTUVW"; OCResourceHandle handle; @@ -476,7 +476,7 @@ TEST(StackResource, CreateResourceBadUri) EXPECT_EQ(OC_STACK_INVALID_URI, OCCreateResource(&handle, "core.led", "core.rw", - uri65, //"/a/led", + uri257, //"/a/led", 0, 0, OC_DISCOVERABLE|OC_OBSERVABLE));