code cleanup.

This commit included code cleanup.
1. coding idiom
2. double free.
3. warning.
4. unlock

second patch included
change the mutex_unlock in NSProviderInterface.

Change-Id: Iff615c577f0cb0e868fa29c1592c523c9bed4d6f
Signed-off-by: jaesick.shin <jaesick.shin@samsung.com>
Reviewed-on: https://gerrit.iotivity.org/gerrit/9679
Tested-by: jenkins-iotivity <jenkins-iotivity@opendaylight.org>
Reviewed-by: Uze Choi <uzchoi@samsung.com>

diff --git a/service/notification/examples/linux/notificationprovider.c b/service/notification/examples/linux/notificationprovider.c
index 6b0cdde..aad18bf 100644 (file)
--- a/service/notification/examples/linux/notificationprovider.c
+++ b/service/notification/examples/linux/notificationprovider.c
@@ -190,11 +190,6 @@ int main()
 \r
             case 4:\r
                 printf("NSRead");\r
-                NSSyncInfo * sync = (NSSyncInfo*) OICMalloc(sizeof(NSSyncInfo));\r
-\r
-                sync->messageId = OICStrdup("dev_001");\r
-                sync->state = 1;\r
-\r
                 break;\r
 \r
             case 5:\r
@@ -210,7 +205,6 @@ int main()
                 printf("Session Code: ");\r
                 gets(REMOTE_SERVER_SESSION);\r
 \r
-\r
                 NSCloudLogin(REMOTE_SERVER_ADDRESS, REMOTE_SERVER_SESSION, CloudLoginoutCallback);\r
                 printf("OCCloudLogin requested");\r
                 break;\r

diff --git a/service/notification/include/NSCommon.h b/service/notification/include/NSCommon.h
index e11fa79..41beb5d 100644 (file)
--- a/service/notification/include/NSCommon.h
+++ b/service/notification/include/NSCommon.h
@@ -44,8 +44,6 @@
 #define NS_ATTRIBUTE_DATETIME "DATE_TIME"
 #define NS_ATTRIBUTE_TTL "TTL"
 
-#include <stdint.h>
-
 /**
  * Result code of notification service
  */

diff --git a/service/notification/src/common/NSUtil.c b/service/notification/src/common/NSUtil.c
index 5b6f41b..541cdd8 100755 (executable)
--- a/service/notification/src/common/NSUtil.c
+++ b/service/notification/src/common/NSUtil.c
@@ -163,7 +163,7 @@ NSSyncInfo* NSDuplicateSync(NSSyncInfo * copyMsg)
 {
     NSSyncInfo * newMsg = NULL;
 
-    if(copyMsg == NULL)
+    if(!copyMsg)
     {
         NS_LOG(ERROR, "Copy Msg is NULL");
         return NULL;
@@ -171,6 +171,12 @@ NSSyncInfo* NSDuplicateSync(NSSyncInfo * copyMsg)
 
     newMsg = (NSSyncInfo *)OICMalloc(sizeof(NSSyncInfo));
 
+    if(!newMsg)
+    {
+        NS_LOG(ERROR, "newMsg is NULL");
+        return NULL;
+    }
+
     newMsg->messageId = copyMsg->messageId;
     OICStrcpy(newMsg->providerId, UUID_STRING_SIZE, copyMsg->providerId);
     newMsg->state = copyMsg->state;
@@ -204,6 +210,13 @@ NSConsumer* NSDuplicateConsumer(NSConsumer * copyMsg)
     }
 
     newMsg = (NSConsumer *)OICMalloc(sizeof(NSConsumer));
+
+    if(!newMsg)
+    {
+        NS_LOG(ERROR, "newMsg is NULL");
+        return NULL;
+    }
+
     (newMsg->consumerId)[0] = '\0';
 
     OICStrcpy(newMsg->consumerId, UUID_STRING_SIZE, copyMsg->consumerId);
@@ -233,7 +246,7 @@ NSSyncInfo * NSGetSyncInfo(OCPayload * payload)
 {
     NS_LOG(DEBUG, "NSGetSyncInfo - IN");
     char * providerId = NULL;
-    int64_t state;
+    int64_t state = 0;
 
     if(!payload)
     {
@@ -362,6 +375,12 @@ NSMediaContents * NSDuplicateMediaContents(NSMediaContents * copyObj)
 
     NSMediaContents * newObj = (NSMediaContents *)OICMalloc(sizeof(NSMediaContents));
 
+    if(!newObj)
+    {
+        NS_LOG(ERROR, "contents newObj is NULL");
+        return NULL;
+    }
+
     if(copyObj->iconImage)
     {
         newObj->iconImage = OICStrdup(copyObj->iconImage);
@@ -374,7 +393,7 @@ NSResult NSFreeMediaContents(NSMediaContents * obj)
 {
     if(!obj)
     {
-        return NS_OK;
+        return NS_FAIL;
     }
 
     NSFreeMalloc(&(obj->iconImage));
@@ -386,6 +405,13 @@ NSResult NSFreeMediaContents(NSMediaContents * obj)
 NSMessage * NSInitializeMessage()
 {
     NSMessage * msg = (NSMessage *)OICMalloc(sizeof(NSMessage));
+
+    if(!msg)
+    {
+        NS_LOG(ERROR, "Msg is NULL");
+        return NULL;
+    }
+
     msg->messageId = OICGetCurrentTime(TIME_IN_MS);
     (msg->providerId)[0] = '\0';
     msg->type = 0;

diff --git a/service/notification/src/provider/NSProviderInterface.c b/service/notification/src/provider/NSProviderInterface.c
index f60ef7b..e2c7061 100644 (file)
--- a/service/notification/src/provider/NSProviderInterface.c
+++ b/service/notification/src/provider/NSProviderInterface.c
@@ -119,6 +119,7 @@ NSResult NSProviderEnableRemoteService(char *serverAddress)
     if(!initProvider)\r
     {\r
         NS_LOG(DEBUG, "Provider service has not been started yet");\r
+        pthread_mutex_unlock(&nsInitMutex);\r
         return NS_FAIL;\r
     }\r
 \r

diff --git a/service/notification/src/provider/NSProviderNotification.c b/service/notification/src/provider/NSProviderNotification.c
index 339357f..a149efd 100644 (file)
--- a/service/notification/src/provider/NSProviderNotification.c
+++ b/service/notification/src/provider/NSProviderNotification.c
@@ -100,7 +100,7 @@ NSResult NSSendNotification(NSMessage *msg)
         return NS_ERROR;
     }
 
-    OCRepPayload* payload;
+    OCRepPayload* payload = NULL;
 
     if (NSSetMessagePayload(msg, &payload) != NS_OK)
     {
@@ -123,10 +123,14 @@ NSResult NSSendNotification(NSMessage *msg)
         if (subData->isWhite)
         {
             if(subData->messageObId != 0)
+            {
                 obArray[obCount++] = subData->messageObId;
+            }
 
             if(subData->remote_messageObId != 0)
+            {
                 obArray[obCount++] = subData->remote_messageObId;
+            }
         }
         it = it->next;
     }
@@ -193,10 +197,14 @@ NSResult NSSendSync(NSSyncInfo *sync)
         if (subData->isWhite)
         {
             if(subData->syncObId != 0)
+            {
                 obArray[obCount++] = subData->syncObId;
+            }
 
             if(subData->remote_syncObId != 0)
+            {
                 obArray[obCount++] = subData->remote_syncObId;
+            }
         }
         it = it->next;
     }

diff --git a/service/notification/src/provider/NSProviderResource.c b/service/notification/src/provider/NSProviderResource.c
index 8da8796..a0fa445 100644 (file)
--- a/service/notification/src/provider/NSProviderResource.c
+++ b/service/notification/src/provider/NSProviderResource.c
@@ -19,6 +19,7 @@
 //-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 
 #include "NSProviderResource.h"
+#include <string.h>
 
 NSNotificationResource NotificationResource;
 NSMessageResource NotificationMessageResource;
@@ -27,6 +28,8 @@ NSSyncResource NotificationSyncResource;
 OCStackApplicationResult NSHandlePublishCb(void *ctx, OCDoHandle handle,
     OCClientResponse *clientResponse)
 {
+    (void) handle;
+
     if (ctx != (void *)DEFAULT_CONTEXT_VALUE)
     {
         NS_LOG(DEBUG, "Invalid publish callback received");

diff --git a/service/notification/src/provider/NSProviderScheduler.c b/service/notification/src/provider/NSProviderScheduler.c
index 64032be..a576fa9 100755 (executable)
--- a/service/notification/src/provider/NSProviderScheduler.c
+++ b/service/notification/src/provider/NSProviderScheduler.c
@@ -143,8 +143,10 @@ bool NSStopScheduler()
 void NSPushQueue(NSSchedulerType schedulerType, NSTaskType taskType, void* data)\r
 {\r
 \r
-    if(NSIsRunning[schedulerType] == false)\r
+    if(!NSIsRunning[schedulerType])\r
+    {\r
         return;\r
+    }\r
 \r
     pthread_mutex_lock(&NSMutex[schedulerType]);\r
 \r
@@ -155,7 +157,6 @@ void NSPushQueue(NSSchedulerType schedulerType, NSTaskType taskType, void* data)
     if (NSHeadMsg[schedulerType] == NULL)\r
     {\r
         NSHeadMsg[schedulerType] = (NSTask*) OICMalloc(sizeof(NSTask));\r
-        memset(NSHeadMsg[schedulerType], 0, sizeof(NSTask));\r
         NSHeadMsg[schedulerType]->taskType = taskType;\r
         NSHeadMsg[schedulerType]->taskData = data;\r
         NSHeadMsg[schedulerType]->nextTask = NULL;\r
@@ -164,7 +165,6 @@ void NSPushQueue(NSSchedulerType schedulerType, NSTaskType taskType, void* data)
     else\r
     {\r
         NSTask* newNode = (NSTask*) OICMalloc(sizeof(NSTask));\r
-        memset(newNode, 0, sizeof(NSTask));\r
         newNode->taskType = taskType;\r
         newNode->taskData = data;\r
         newNode->nextTask = NULL;\r

diff --git a/service/notification/src/provider/NSProviderSubscription.c b/service/notification/src/provider/NSProviderSubscription.c
index bb8d742..3c2c267 100644 (file)
--- a/service/notification/src/provider/NSProviderSubscription.c
+++ b/service/notification/src/provider/NSProviderSubscription.c
@@ -60,7 +60,7 @@ NSResult NSSendAccessPolicyResponse(OCEntityHandlerRequest *entityHandlerRequest
     NS_LOG(DEBUG, "NSSendAccessPolicyResponse - IN");\r
 \r
     // put notification resource\r
-    OCResourceHandle notificationResourceHandle;\r
+    OCResourceHandle notificationResourceHandle = NULL;\r
     if (NSPutNotificationResource(NSGetSubscriptionAccepter(), &notificationResourceHandle)\r
             != NS_OK)\r
     {\r
@@ -244,7 +244,7 @@ NSResult NSSendResponse(const char * id, bool accepted)
         return NS_ERROR;\r
     }\r
 \r
-    OCResourceHandle rHandle;\r
+    OCResourceHandle rHandle = NULL;\r
     if (NSPutMessageResource(NULL, &rHandle) != NS_OK)\r
     {\r
         NS_LOG(ERROR, "Fail to put notification resource");\r

diff --git a/service/notification/src/provider/NSProviderSystem.c b/service/notification/src/provider/NSProviderSystem.c
index 2a6c6ee..b9b2bf6 100644 (file)
--- a/service/notification/src/provider/NSProviderSystem.c
+++ b/service/notification/src/provider/NSProviderSystem.c
@@ -71,7 +71,7 @@ void NSInitProviderInfo()
     providerInfo = (NSProviderInfo *) OICMalloc(sizeof(NSProviderInfo));\r
     const char * generatedUuid = (char *)OCGetServerInstanceIDString();\r
     NS_LOG_V(DEBUG, "Generate Provider ID: %s", generatedUuid);\r
-    OICStrcpy(providerInfo->providerId, strlen(generatedUuid), generatedUuid);\r
+    OICStrcpy(providerInfo->providerId, UUID_STRING_SIZE, generatedUuid);\r
 \r
     providerInfo->providerName = NULL;\r
 }\r

diff --git a/service/notification/src/provider/cache/linux/NSProviderMemoryCache.c b/service/notification/src/provider/cache/linux/NSProviderMemoryCache.c
index bf761e5..66d88f9 100755 (executable)
--- a/service/notification/src/provider/cache/linux/NSProviderMemoryCache.c
+++ b/service/notification/src/provider/cache/linux/NSProviderMemoryCache.c
@@ -19,6 +19,7 @@
 //-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=\r
 \r
 #include "NSProviderMemoryCache.h"\r
+#include <string.h>\r
 \r
 NSCacheList * NSStorageCreate()\r
 {\r
@@ -204,7 +205,9 @@ NSResult NSStorageWrite(NSCacheList * list, NSCacheElement * newObj)
 \r
         NSCacheMsgData * msgData = (NSCacheMsgData *) newObj->data;\r
 \r
+        pthread_mutex_unlock(&NSCacheMutex);\r
         NSCacheElement * it = NSStorageRead(list, msgData->id);\r
+        pthread_mutex_lock(&NSCacheMutex);\r
         if (it)\r
         {\r
             NSCacheMsgData * itData = (NSCacheMsgData *) it->data;\r
@@ -248,7 +251,9 @@ NSResult NSStorageDelete(NSCacheList * list, const char * delId)
         if (del == list->head) // first object\r
         {\r
             if (del == list->tail) // first object (one object)\r
+            {\r
                 list->tail = del->next;\r
+            }\r
 \r
             list->head = del->next;\r
 \r
@@ -265,7 +270,9 @@ NSResult NSStorageDelete(NSCacheList * list, const char * delId)
         if (NSProviderCompareIdCacheData(type, del->data, delId))\r
         {\r
             if (del == list->tail) // delete object same to last object\r
+            {\r
                 list->tail = prev;\r
+            }\r
 \r
             prev->next = del->next;\r
             NSProviderDeleteCacheData(type, del->data);\r
@@ -391,7 +398,10 @@ bool NSIsSameObId(NSCacheSubData * data, OCObservationId id)
 {\r
     if (id == data->messageObId || id == data->syncObId || id == data->remote_messageObId ||\r
                 id == data->remote_syncObId)\r
+    {\r
         return true;\r
+    }\r
+\r
     return false;\r
 }\r
 \r
@@ -416,7 +426,9 @@ NSResult NSProviderDeleteSubDataFromObId(NSCacheList * list, OCObservationId id)
             if (del == list->head) // first object\r
             {\r
                 if (del == list->tail) // first object (one object)\r
+                {\r
                     list->tail = del->next;\r
+                }\r
 \r
                 list->head = del->next;\r
 \r
@@ -433,7 +445,9 @@ NSResult NSProviderDeleteSubDataFromObId(NSCacheList * list, OCObservationId id)
                 if (NSIsSameObId(curr, id))\r
                 {\r
                     if (del == list->tail) // delete object same to last object\r
+                    {\r
                         list->tail = prev;\r
+                    }\r
 \r
                     prev->next = del->next;\r
                     NSProviderDeleteCacheData(type, del->data);\r

diff --git a/service/notification/src/provider/cache/linux/NSProviderMemoryCache.h b/service/notification/src/provider/cache/linux/NSProviderMemoryCache.h
index 218ce92..43cca9c 100755 (executable)
--- a/service/notification/src/provider/cache/linux/NSProviderMemoryCache.h
+++ b/service/notification/src/provider/cache/linux/NSProviderMemoryCache.h
@@ -23,7 +23,6 @@
 \r
 #include <pthread.h>\r
 #include <stdbool.h>\r
-#include <string.h>\r
 \r
 #include "NSCommon.h"\r
 #include "NSConstants.h"\r