From: js126.lee <js126.lee@samsung.com>
Date: Mon, 5 Oct 2015 07:16:12 +0000 (+0900)
Subject: Correct the ENUM value of OTM according to spec.
X-Git-Tag: 1.0.0~25
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a2525774553f290931b34b8cc6b3f0f5403b5a21;p=contrib%2Fiotivity.git

Correct the ENUM value of OTM according to spec.

- Issue : Compliance issue on OTM type
- Solution : Samsung suggest to revive enum type of OTM, and make URN type to optional and ENUM type to mandatory/
            THis patch is for this solution.
     oic:justwork = 0, oic:randompin = 1, oic:mfgcert = 2

Note : Need to apply this patch, once ENUM type of OTM is revived on Spec.

Change-Id: Id0ae1fcf4175576ff2afd7fde7e473942c391f66
Signed-off-by: js126.lee <js126.lee@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/3627
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Sachin Agrawal <sachin.agrawal@intel.com>
---

diff --git a/android/android_api/base/src/main/java/org/iotivity/base/OxmType.java b/android/android_api/base/src/main/java/org/iotivity/base/OxmType.java
old mode 100644
new mode 100755
index f336076..177105e
--- a/android/android_api/base/src/main/java/org/iotivity/base/OxmType.java
+++ b/android/android_api/base/src/main/java/org/iotivity/base/OxmType.java
@@ -24,11 +24,9 @@ package org.iotivity.base;
 
 public enum OxmType {
     OIC_JUST_WORKS(0),
-    OIC_MODE_SWITCH(1),
-    OIC_RANDOM_DEVICE_PIN(2),
-    OIC_PRE_PROVISIONED_DEVICE_PIN(3),
-    OIC_PRE_PROVISION_STRONG_CREDENTIAL(4),
-    OIC_OXM_COUNT(5);
+    OIC_RANDOM_DEVICE_PIN(1),
+    OIC_MANUFACTURER_CERTIFICATE(2),
+    OIC_OXM_COUNT(3);
 
     private int value;
 
diff --git a/resource/csdk/security/include/internal/srmresourcestrings.h b/resource/csdk/security/include/internal/srmresourcestrings.h
old mode 100644
new mode 100755
index 5fc3922..1ac3fb0
--- a/resource/csdk/security/include/internal/srmresourcestrings.h
+++ b/resource/csdk/security/include/internal/srmresourcestrings.h
@@ -98,10 +98,8 @@ extern const char * WILDCARD_RESOURCE_URI;
 
 //Ownership Transfer Methods
 extern const char * OXM_JUST_WORKS;
-extern const char * OXM_MODE_SWITCH;
 extern const char * OXM_RANDOM_DEVICE_PIN;
-extern const char * OXM_PRE_PROVISIONED_DEVICE_PIN;
-extern const char * OXM_PRE_PROVISIONED_STRONG_CREDENTIAL;
+extern const char * OXM_MANUFACTURER_CERTIFICATE;
 
 extern const char * OIC_SEC_TRUE;
 extern const char * OIC_SEC_FALSE;
diff --git a/resource/csdk/security/include/securevirtualresourcetypes.h b/resource/csdk/security/include/securevirtualresourcetypes.h
old mode 100644
new mode 100755
index 2533a78..83dc60f
--- a/resource/csdk/security/include/securevirtualresourcetypes.h
+++ b/resource/csdk/security/include/securevirtualresourcetypes.h
@@ -240,10 +240,8 @@ typedef enum
 typedef enum
 {
     OIC_JUST_WORKS                          = 0x0,
-    OIC_MODE_SWITCH                         = 0x1,
-    OIC_RANDOM_DEVICE_PIN                   = 0x2,
-    OIC_PRE_PROVISIONED_DEVICE_PIN          = 0x3,
-    OIC_PRE_PROVISION_STRONG_CREDENTIAL     = 0x4,
+    OIC_RANDOM_DEVICE_PIN                   = 0x1,
+    OIC_MANUFACTURER_CERTIFICATE           = 0x2,
     OIC_OXM_COUNT
 }OicSecOxm_t;
 
@@ -448,8 +446,8 @@ struct OicSecSvc
 struct OicSecCrl
 {
     uint16_t CrlId;
-    ByteArray ThisUpdate;   
-    ByteArray CrlData;      
+    ByteArray ThisUpdate;
+    ByteArray CrlData;
 };
 #endif /* __WITH_X509__ */
 
diff --git a/resource/csdk/security/provisioning/sample/oic_svr_db_server_randompin.json b/resource/csdk/security/provisioning/sample/oic_svr_db_server_randompin.json
old mode 100644
new mode 100755
index 49029c9..a79a2b2
--- a/resource/csdk/security/provisioning/sample/oic_svr_db_server_randompin.json
+++ b/resource/csdk/security/provisioning/sample/oic_svr_db_server_randompin.json
@@ -37,7 +37,7 @@
 		"sm":	[3]
 	},
 	"doxm":	{
-		"oxm":	[0,2],
+		"oxm":	[0,1],
 		"oxmsel": 0,
 		"sct": 1,
 		"owned": false,
diff --git a/resource/csdk/security/provisioning/src/ownershiptransfermanager.c b/resource/csdk/security/provisioning/src/ownershiptransfermanager.c
old mode 100644
new mode 100755
index d1150ae..6ac1ff0
--- a/resource/csdk/security/provisioning/src/ownershiptransfermanager.c
+++ b/resource/csdk/security/provisioning/src/ownershiptransfermanager.c
@@ -88,14 +88,10 @@ static const char* GetOxmString(OicSecOxm_t oxmType)
     {
         case OIC_JUST_WORKS:
             return OXM_JUST_WORKS;
-        case OIC_MODE_SWITCH:
-            return OXM_MODE_SWITCH;
         case OIC_RANDOM_DEVICE_PIN:
             return OXM_RANDOM_DEVICE_PIN;
-        case OIC_PRE_PROVISIONED_DEVICE_PIN:
-            return OXM_PRE_PROVISIONED_DEVICE_PIN;
-        case OIC_PRE_PROVISION_STRONG_CREDENTIAL:
-            return OXM_PRE_PROVISIONED_STRONG_CREDENTIAL;
+        case OIC_MANUFACTURER_CERTIFICATE:
+            return OXM_MANUFACTURER_CERTIFICATE;
         default:
             return NULL;
     }
diff --git a/resource/csdk/security/src/srmresourcestrings.c b/resource/csdk/security/src/srmresourcestrings.c
old mode 100644
new mode 100755
index 70ce8a3..be72bf3
--- a/resource/csdk/security/src/srmresourcestrings.c
+++ b/resource/csdk/security/src/srmresourcestrings.c
@@ -96,10 +96,8 @@ const char * WILDCARD_RESOURCE_URI = "*";
 
 //Ownership Transfer Methods
 const char * OXM_JUST_WORKS = "oic.sec.doxm.jw";
-const char * OXM_MODE_SWITCH = "oic.sec.doxm.ms";
 const char * OXM_RANDOM_DEVICE_PIN = "oic.sec.doxm.rdp";
-const char * OXM_PRE_PROVISIONED_DEVICE_PIN = "oic.sec.doxm.ppdp";
-const char * OXM_PRE_PROVISIONED_STRONG_CREDENTIAL = "oic.sec.doxm.ppsc";
+const char * OXM_MANUFACTURER_CERTIFICATE = "oic.sec.doxm.mfgcert";
 
 const char * OIC_SEC_TRUE = "true";
 const char * OIC_SEC_FALSE = "false";