Imported Upstream version 1.1.0
[platform/upstream/iotivity.git] / resource / csdk / connectivity / src / cablockwisetransfer.c
index 6fc3443..f214067 100644 (file)
@@ -43,7 +43,7 @@
 #include "camutex.h"
 #include "logger.h"
 
-#define TAG "CA_BWT"
+#define TAG "OIC_CA_BWT"
 
 #define BLOCKWISE_OPTION_BUFFER    (sizeof(unsigned int))
 #define BLOCK_NUMBER_IDX           4
@@ -53,7 +53,9 @@
 #define BLOCK_SIZE(arg) (1 << ((arg) + 4))
 
 // context for block-wise transfer
-static CABlockWiseContext_t g_context = { 0 };
+static CABlockWiseContext_t g_context = { .sendThreadFunc = NULL,
+                                          .receivedThreadFunc = NULL,
+                                          .dataList = NULL };
 
 static bool CACheckPayloadLength(const CAData_t *sendData)
 {
@@ -62,7 +64,7 @@ static bool CACheckPayloadLength(const CAData_t *sendData)
 
     // check if message has to be transfered to a block
     size_t maxBlockSize = BLOCK_SIZE(CA_DEFAULT_BLOCK_SIZE);
-    OIC_LOG_V(DEBUG, TAG, "payloadLen=%d, maxBlockSize=%d", payloadLen, maxBlockSize);
+    OIC_LOG_V(DEBUG, TAG, "payloadLen=%zu, maxBlockSize=%zu", payloadLen, maxBlockSize);
 
     if (payloadLen <= maxBlockSize)
     {
@@ -75,7 +77,7 @@ static bool CACheckPayloadLength(const CAData_t *sendData)
 CAResult_t CAInitializeBlockWiseTransfer(CASendThreadFunc sendThreadFunc,
                                          CAReceiveThreadFunc receivedThreadFunc)
 {
-    OIC_LOG(DEBUG, TAG, "initialize");
+    OIC_LOG(DEBUG, TAG, "CAInitializeBlockWiseTransfer");
 
     // set block-wise transfer context
     if (!g_context.sendThreadFunc)
@@ -96,6 +98,8 @@ CAResult_t CAInitializeBlockWiseTransfer(CASendThreadFunc sendThreadFunc,
     CAResult_t res = CAInitBlockWiseMutexVariables();
     if (CA_STATUS_OK != res)
     {
+        u_arraylist_free(&g_context.dataList);
+        g_context.dataList = NULL;
         OIC_LOG(ERROR, TAG, "init has failed");
     }
 
@@ -104,7 +108,7 @@ CAResult_t CAInitializeBlockWiseTransfer(CASendThreadFunc sendThreadFunc,
 
 CAResult_t CATerminateBlockWiseTransfer()
 {
-    OIC_LOG(DEBUG, TAG, "terminate");
+    OIC_LOG(DEBUG, TAG, "CATerminateBlockWiseTransfer");
 
     if (g_context.dataList)
     {
@@ -162,13 +166,36 @@ CAResult_t CASendBlockWiseData(const CAData_t *sendData)
     VERIFY_NON_NULL(sendData, TAG, "sendData");
 
     // check if message type is CA_MSG_RESET
-    if (sendData->responseInfo)
+    if (sendData->requestInfo)
+    {
+        if (CA_MSG_RESET == sendData->requestInfo->info.type)
+        {
+            OIC_LOG(DEBUG, TAG, "reset message can't be sent to the block");
+            return CA_NOT_SUPPORTED;
+        }
+
+        /*
+         * Other uses of the Block options in conjunction with multicast
+         * messages are for further study.
+         */
+        if (sendData->requestInfo->isMulticast)
+        {
+            OIC_LOG(DEBUG, TAG, "multicast message can't be sent to the block");
+            return CA_NOT_SUPPORTED;
+        }
+    }
+    else if (sendData->responseInfo)
     {
         if (CA_MSG_RESET == sendData->responseInfo->info.type)
         {
             OIC_LOG(DEBUG, TAG, "reset message can't be sent to the block");
             return CA_NOT_SUPPORTED;
         }
+        if (sendData->responseInfo->isMulticast)
+        {
+            OIC_LOG(DEBUG, TAG, "multicast message can't be sent to the block");
+            return CA_NOT_SUPPORTED;
+        }
     }
 
     // #1. check if it is already included in block data list
@@ -211,7 +238,7 @@ CAResult_t CASendBlockWiseData(const CAData_t *sendData)
         // #4. send block message
         OIC_LOG(DEBUG, TAG, "send first block msg");
         res = CAAddSendThreadQueue(currData->sentData,
-                                   (const CABlockDataID_t *)&currData->blockDataId);
+                                   (const CABlockDataID_t *) &currData->blockDataId);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
@@ -287,20 +314,31 @@ CAResult_t CAReceiveBlockWiseData(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
     {
         OIC_LOG(DEBUG, TAG, "code is CA_EMPTY..");
 
-        // get token from block-wise transfer list when CA_EMPTY(RST/ACK) is received
-        CAResult_t res = CAGetTokenFromBlockDataList(pdu, endpoint, receivedData->responseInfo);
-        if (CA_STATUS_OK != res)
+        if (!receivedData->responseInfo->info.token)
         {
-            OIC_LOG(ERROR, TAG, "fail to get token");
-            return res;
+            // get token from block-wise transfer list when CA_EMPTY(RST/ACK) is received
+            CAResult_t res = CAGetTokenFromBlockDataList(pdu, endpoint,
+                                                         receivedData->responseInfo);
+            if (CA_STATUS_OK != res)
+            {
+                OIC_LOG(ERROR, TAG, "fail to get token");
+                return res;
+            }
         }
 
         CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
                 receivedData->responseInfo->info.token,
                 receivedData->responseInfo->info.tokenLength,
                 endpoint->port);
-        if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+        if (NULL == blockDataID || blockDataID->idLength < 1)
         {
+            // if retransmission is timeout, callback msg will be send without token.
+            if (NULL == blockDataID && !receivedData->responseInfo->info.token)
+            {
+                OIC_LOG(INFO, TAG, "retransmission was stopped");
+                return CA_REQUEST_TIMEOUT;
+            }
+
             OIC_LOG(ERROR, TAG, "blockId is null");
             CADestroyBlockID(blockDataID);
             return CA_STATUS_FAILED;
@@ -312,7 +350,7 @@ CAResult_t CAReceiveBlockWiseData(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
     }
 
     // check if block option is set and get block data
-    coap_block_t block = {0, 0, 0};
+    coap_block_t block = { 0, 0, 0 };
 
     // get block1 option
     int isBlock1 = coap_get_block(pdu, COAP_OPTION_BLOCK1, &block);
@@ -338,63 +376,53 @@ CAResult_t CAReceiveBlockWiseData(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
         }
     }
 
-    // check if there is error code
+    // if there is no block option in pdu, check if there is error code.
     if (!isBlock1 && !isBlock2)
     {
+        CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
+                (CAToken_t)pdu->hdr->coap_hdr_udp_t.token,
+                pdu->hdr->coap_hdr_udp_t.token_length,
+                endpoint->port);
+        if (NULL == blockDataID || blockDataID->idLength < 1)
+        {
+            OIC_LOG(ERROR, TAG, "blockId is null");
+            CADestroyBlockID(blockDataID);
+            return CA_STATUS_FAILED;
+        }
+
         uint32_t code = CA_RESPONSE_CODE(pdu->hdr->coap_hdr_udp_t.code);
         if (CA_REQUEST_ENTITY_INCOMPLETE == code)
         {
-            CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
-                    (CAToken_t)pdu->hdr->coap_hdr_udp_t.token,
-                     pdu->hdr->coap_hdr_udp_t.token_length,
-                     endpoint->port);
-
-            if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+            CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
+            if (!data)
             {
-                OIC_LOG(ERROR, TAG, "blockId is null");
+                OIC_LOG(ERROR, TAG, "getting has failed");
                 CADestroyBlockID(blockDataID);
                 return CA_STATUS_FAILED;
             }
 
-            CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
-            if (!data)
+            coap_block_t *block = CAGetBlockOption(blockDataID, data->type);
+            if (!block)
             {
-                OIC_LOG(ERROR, TAG, "getting has failed");
+                OIC_LOG(ERROR, TAG, "block is null");
                 CADestroyBlockID(blockDataID);
                 return CA_STATUS_FAILED;
             }
 
+            CAResult_t res = CA_STATUS_OK;
             if (COAP_OPTION_BLOCK2 == data->type)
             {
-                coap_block_t *block2 = CAGetBlockOption(blockDataID, COAP_OPTION_BLOCK2);
-                if (!block2)
-                {
-                    OIC_LOG(ERROR, TAG, "block is null");
-                    CADestroyBlockID(blockDataID);
-                    return CA_STATUS_FAILED;
-                }
-
-                CAResult_t res = CASetNextBlockOption2(pdu, endpoint, receivedData, *block2,
-                                                       dataLen);
+                res = CASetNextBlockOption2(pdu, endpoint, receivedData, *block, dataLen);
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "setting has failed");
                     CADestroyBlockID(blockDataID);
-                    return res;
+                    return CA_STATUS_FAILED;
                 }
             }
             else if (COAP_OPTION_BLOCK1 == data->type)
             {
-                coap_block_t *block1 = CAGetBlockOption(blockDataID, COAP_OPTION_BLOCK1);
-                if (!block1)
-                {
-                    OIC_LOG(ERROR, TAG, "block is null");
-                    CADestroyBlockID(blockDataID);
-                    return CA_STATUS_FAILED;
-                }
-
-                CAResult_t res = CASetNextBlockOption1(pdu, endpoint, receivedData, *block1,
-                                                       dataLen);
+                res = CASetNextBlockOption1(pdu, endpoint, receivedData, *block, dataLen);
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "setting has failed");
@@ -402,6 +430,7 @@ CAResult_t CAReceiveBlockWiseData(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
                     return res;
                 }
             }
+            CADestroyBlockID(blockDataID);
         }
         else
         {
@@ -412,24 +441,12 @@ CAResult_t CAReceiveBlockWiseData(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
             // and sent data remain in block data list, remove block data
             if (receivedData->responseInfo)
             {
-                CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
-                        (CAToken_t)pdu->hdr->coap_hdr_udp_t.token,
-                        pdu->hdr->coap_hdr_udp_t.token_length,
-                        endpoint->port);
-                if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
-                {
-                    OIC_LOG(ERROR, TAG, "blockId is null");
-                    CADestroyBlockID(blockDataID);
-                    return CA_STATUS_FAILED;
-                }
-
                 CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
             }
+            CADestroyBlockID(blockDataID);
             return CA_NOT_SUPPORTED;
         }
     }
-
     return CA_STATUS_OK;
 }
 
@@ -455,7 +472,7 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
             }
             break;
 
-        case CA_OPTION2_CON:
+        case CA_OPTION2_REQUEST:
             // add data to send thread
             data = CAGetDataSetFromBlockDataList(blockID);
             if (!data)
@@ -483,11 +500,10 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
 
             break;
 
-        case CA_OPTION1_ACK:
-        case CA_OPTION2_ACK:
-        case CA_SENT_PREVIOUS_NON_MSG:
-            res = CASendBlockMessage(pdu, CA_MSG_CONFIRM, blockWiseStatus,
-                                     blockID);
+        case CA_OPTION1_RESPONSE:
+        case CA_OPTION2_RESPONSE:
+        case CA_OPTION1_REQUEST_BLOCK:
+            res = CASendBlockMessage(pdu, pdu->hdr->coap_hdr_udp_t.type, blockID);
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "send has failed");
@@ -513,7 +529,7 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
             }
             break;
 
-        case CA_OPTION1_NO_ACK_LAST_BLOCK:
+        case CA_OPTION1_REQUEST_LAST_BLOCK:
             // process last block and send upper layer
             res = CAReceiveLastBlock(blockID, receivedData);
             if (CA_STATUS_OK != res)
@@ -521,53 +537,22 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
                 OIC_LOG(ERROR, TAG, "receive has failed");
                 return res;
             }
-
-            if (CA_MSG_NONCONFIRM == pdu->hdr->coap_hdr_udp_t.type)
-            {
-                // remove data from list
-                res = CARemoveBlockDataFromList(blockID);
-                if (CA_STATUS_OK != res)
-                {
-                    OIC_LOG(ERROR, TAG, "remove has failed");
-                    return res;
-                }
-            }
-            break;
-
-        case CA_OPTION1_NO_ACK_BLOCK:
-            if (CA_MSG_CONFIRM == pdu->hdr->coap_hdr_udp_t.type)
-            {
-                // add data to send thread
-                res = CASendBlockMessage(pdu, CA_MSG_ACKNOWLEDGE, blockWiseStatus,
-                                         blockID);
-                if (CA_STATUS_OK != res)
-                {
-                    OIC_LOG(ERROR, TAG, "send has failed");
-                    return res;
-                }
-            }
             break;
 
         case CA_BLOCK_INCOMPLETE:
-            if (CA_MSG_CONFIRM == pdu->hdr->coap_hdr_udp_t.type ||
-                    CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type)
+            // add data to send thread
+            res = CASendErrorMessage(pdu, blockWiseStatus, CA_REQUEST_ENTITY_INCOMPLETE, blockID);
+            if (CA_STATUS_OK != res)
             {
-                // add data to send thread
-                res = CASendErrorMessage(pdu, blockWiseStatus,
-                                         CA_REQUEST_ENTITY_INCOMPLETE,
-                                         blockID);
-                if (CA_STATUS_OK != res)
-                {
-                    OIC_LOG(ERROR, TAG, "send has failed");
-                    return res;
-                }
+                OIC_LOG(ERROR, TAG, "send has failed");
+                return res;
             }
             break;
 
         case CA_BLOCK_TOO_LARGE:
-            if (CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type)
+            if (receivedData->requestInfo)
             {
-                res = CASendBlockMessage(pdu, CA_MSG_CONFIRM, blockWiseStatus,
+                res = CASendErrorMessage(pdu, blockWiseStatus, CA_REQUEST_ENTITY_TOO_LARGE,
                                          blockID);
                 if (CA_STATUS_OK != res)
                 {
@@ -575,11 +560,9 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
                     return res;
                 }
             }
-            else if (CA_MSG_CONFIRM == pdu->hdr->coap_hdr_udp_t.type)
+            else if (receivedData->responseInfo)
             {
-                res = CASendErrorMessage(pdu, blockWiseStatus,
-                                         CA_REQUEST_ENTITY_TOO_LARGE,
-                                         blockID);
+                res = CASendBlockMessage(pdu, pdu->hdr->coap_hdr_udp_t.type, blockID);
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "send has failed");
@@ -594,7 +577,7 @@ CAResult_t CAProcessNextStep(const coap_pdu_t *pdu, const CAData_t *receivedData
 }
 
 CAResult_t CASendBlockMessage(const coap_pdu_t *pdu, CAMessageType_t msgType,
-                              uint8_t status, const CABlockDataID_t *blockID)
+                              const CABlockDataID_t *blockID)
 {
     VERIFY_NON_NULL(pdu, TAG, "pdu");
     VERIFY_NON_NULL(pdu->hdr, TAG, "pdu->hdr");
@@ -607,34 +590,38 @@ CAResult_t CASendBlockMessage(const coap_pdu_t *pdu, CAMessageType_t msgType,
         return CA_STATUS_FAILED;
     }
 
-    if (CA_MSG_CONFIRM == msgType)
+    CAMessageType_t sentMsgType = CA_MSG_NONCONFIRM;
+    switch (msgType)
     {
-        OIC_LOG(DEBUG, TAG, "need new msgID");
-        if (data->requestInfo)
-        {
-            data->requestInfo->info.messageId = 0;
-        }
+        case CA_MSG_CONFIRM:
+            sentMsgType = CA_MSG_ACKNOWLEDGE;
+            break;
+        case CA_MSG_ACKNOWLEDGE:
+            sentMsgType = CA_MSG_CONFIRM;
+            break;
+        default:
+            sentMsgType = CA_MSG_NONCONFIRM;
+            break;
+    }
 
+    uint32_t code = pdu->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
+    {
         if (data->responseInfo)
         {
-            data->responseInfo->info.messageId = 0;
+            OIC_LOG(DEBUG, TAG, "set response info");
+            data->responseInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
+            data->responseInfo->info.type = sentMsgType;
+            data->responseInfo->result = CA_CONTINUE;
         }
     }
-    else if (CA_MSG_ACKNOWLEDGE == msgType)
+    else
     {
-        if (data->responseInfo)
+        OIC_LOG(DEBUG, TAG, "need new msgID");
+        if (data->requestInfo)
         {
-            OIC_LOG(DEBUG, TAG, "set ACK message");
-            data->responseInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
-            data->responseInfo->info.type = CA_MSG_ACKNOWLEDGE;
-            if (CA_OPTION1_NO_ACK_LAST_BLOCK == status)
-            {
-                data->responseInfo->result = CA_CHANGED;
-            }
-            else if (CA_OPTION1_NO_ACK_BLOCK == status)
-            {
-                data->responseInfo->result = CA_CONTINUE;
-            }
+            data->requestInfo->info.messageId = 0;
+            data->requestInfo->info.type = sentMsgType;
         }
     }
 
@@ -649,8 +636,7 @@ CAResult_t CASendBlockMessage(const coap_pdu_t *pdu, CAMessageType_t msgType,
 }
 
 CAResult_t CASendErrorMessage(const coap_pdu_t *pdu, uint8_t status,
-                              CAResponseResult_t responseResult,
-                              const CABlockDataID_t *blockID)
+                              CAResponseResult_t responseResult, const CABlockDataID_t *blockID)
 {
     VERIFY_NON_NULL(pdu, TAG, "pdu");
     VERIFY_NON_NULL(pdu->hdr, TAG, "pdu->hdr");
@@ -664,32 +650,57 @@ CAResult_t CASendErrorMessage(const coap_pdu_t *pdu, uint8_t status,
         return CA_STATUS_FAILED;
     }
 
+    CAMessageType_t sentMsgType = CA_MSG_NONCONFIRM;
+    switch (pdu->hdr->coap_hdr_udp_t.type)
+    {
+        case CA_MSG_CONFIRM:
+            sentMsgType = CA_MSG_ACKNOWLEDGE;
+            break;
+        case CA_MSG_ACKNOWLEDGE:
+            sentMsgType = CA_MSG_CONFIRM;
+            break;
+    }
+
     CAData_t *cloneData = NULL;
-    if (data->sentData && data->sentData->responseInfo)
+    if (data->sentData)
     {
-        data->sentData->responseInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
-        data->sentData->responseInfo->info.type = CA_MSG_ACKNOWLEDGE;
-        data->sentData->responseInfo->result = responseResult;
         cloneData = CACloneCAData(data->sentData);
         if (!cloneData)
         {
             OIC_LOG(ERROR, TAG, "clone has failed");
             return CA_MEMORY_ALLOC_FAILED;
         }
-        OIC_LOG(DEBUG, TAG, "set ACK message");
-    }
-    else if (data->sentData)
-    {
-        cloneData = CACreateNewDataSet(pdu, data->sentData->remoteEndpoint);
-        if(!cloneData)
+
+        if (cloneData->responseInfo)
         {
-            OIC_LOG(ERROR, TAG, PCF("CACreateNewDataSet failed"));
-            return CA_MEMORY_ALLOC_FAILED;
+            cloneData->responseInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
+            cloneData->responseInfo->info.type = sentMsgType;
+            cloneData->responseInfo->result = responseResult;
         }
+        else
+        {
+            CAInfo_t responseData = { .tokenLength = pdu->hdr->coap_hdr_udp_t.token_length };
+            responseData.token = (CAToken_t) OICMalloc(responseData.tokenLength);
+            if (!responseData.token)
+            {
+                OIC_LOG(ERROR, TAG, "out of memory");
+                return CA_MEMORY_ALLOC_FAILED;
+            }
+            memcpy(responseData.token, pdu->hdr->coap_hdr_udp_t.token, responseData.tokenLength);
 
-        cloneData->responseInfo->info.type = CA_MSG_CONFIRM;
-        cloneData->responseInfo->result = responseResult;
-        OIC_LOG(DEBUG, TAG, "set CON message");
+            cloneData->responseInfo = (CAResponseInfo_t*) OICCalloc(1, sizeof(CAResponseInfo_t));
+            if (!cloneData->responseInfo)
+            {
+                OIC_LOG(ERROR, TAG, "out of memory");
+                OICFree(responseData.token);
+                return CA_MEMORY_ALLOC_FAILED;
+            }
+
+            cloneData->responseInfo->info = responseData;
+            cloneData->responseInfo->info.type = sentMsgType;
+            cloneData->responseInfo->result = responseResult;
+        }
+        OIC_LOG(DEBUG, TAG, "set response message to send error code");
     }
     else
     {
@@ -697,6 +708,13 @@ CAResult_t CASendErrorMessage(const coap_pdu_t *pdu, uint8_t status,
         return CA_MEMORY_ALLOC_FAILED;
     }
 
+    // if there is a requestInfo, remove it to send response message
+    if (cloneData->requestInfo)
+    {
+        CADestroyRequestInfoInternal(cloneData->requestInfo);
+        cloneData->requestInfo = NULL;
+    }
+
     // add data to send thread
     if (g_context.sendThreadFunc)
     {
@@ -723,8 +741,7 @@ CAResult_t CASendErrorMessage(const coap_pdu_t *pdu, uint8_t status,
     return CA_STATUS_OK;
 }
 
-CAResult_t CAReceiveLastBlock(const CABlockDataID_t *blockID,
-                              const CAData_t *receivedData)
+CAResult_t CAReceiveLastBlock(const CABlockDataID_t *blockID, const CAData_t *receivedData)
 {
     VERIFY_NON_NULL(blockID, TAG, "blockID");
     VERIFY_NON_NULL(receivedData, TAG, "receivedData");
@@ -739,8 +756,7 @@ CAResult_t CAReceiveLastBlock(const CABlockDataID_t *blockID,
 
     // update payload
     size_t fullPayloadLen = 0;
-    CAPayload_t fullPayload = CAGetPayloadFromBlockDataList(blockID,
-                                                            &fullPayloadLen);
+    CAPayload_t fullPayload = CAGetPayloadFromBlockDataList(blockID, &fullPayloadLen);
     if (fullPayload)
     {
         CAResult_t res = CAUpdatePayloadToCAData(cloneData, fullPayload, fullPayloadLen);
@@ -781,117 +797,59 @@ CAResult_t CASetNextBlockOption1(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
             (CAToken_t)pdu->hdr->coap_hdr_udp_t.token,
             pdu->hdr->coap_hdr_udp_t.token_length,
             endpoint->port);
-
-    if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+    if (NULL == blockDataID || blockDataID->idLength < 1)
     {
         OIC_LOG(ERROR, TAG, "blockId is null");
         CADestroyBlockID(blockDataID);
         return CA_STATUS_FAILED;
     }
 
-    // BlockData data is created if it not existed
-    if (!CAIsBlockDataInList(blockDataID))
+    // Get BlockData data. If does not exist, create a new data
+    CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
+    if (!data)
     {
-        OIC_LOG(DEBUG, TAG, "no message in list");
+        OIC_LOG(DEBUG, TAG, "block data doesn't exist in list. create new one");
 
-        CAData_t *data = CACreateNewDataSet(pdu, endpoint);
-        if (!data)
+        CAData_t *cadata = CACreateNewDataSet(pdu, endpoint);
+        if (!cadata)
         {
             OIC_LOG(ERROR, TAG, "data is null");
             CADestroyBlockID(blockDataID);
             return CA_STATUS_FAILED;
         }
 
-        CABlockData_t *currData = CACreateNewBlockData(data);
-        if (!currData)
+        data = CACreateNewBlockData(cadata);
+        if (!data)
         {
-            OIC_LOG(ERROR, TAG, "currData is null");
-            CADestroyDataSet(data);
+            OIC_LOG(ERROR, TAG, "failed to create a new block data");
+            CADestroyDataSet(cadata);
             CADestroyBlockID(blockDataID);
             return CA_STATUS_FAILED;
         }
-        CADestroyDataSet(data);
+        CADestroyDataSet(cadata);
     }
 
     // update BLOCK OPTION1 type
-    CAResult_t res = CAUpdateBlockOptionType(blockDataID,
-                                             COAP_OPTION_BLOCK1);
+    CAResult_t res = CAUpdateBlockOptionType(blockDataID, COAP_OPTION_BLOCK1);
     if (CA_STATUS_OK != res)
     {
         OIC_LOG(ERROR, TAG, "update has failed");
-        CARemoveBlockDataFromList(blockDataID);
-        CADestroyBlockID(blockDataID);
-        return res;
-    }
-
-    CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
-    if (!data)
-    {
-        OIC_LOG(ERROR, TAG, "getting has failed");
-        CADestroyBlockID(blockDataID);
-        return CA_STATUS_FAILED;
+        goto exit;
     }
 
     uint8_t blockWiseStatus = CA_BLOCK_UNKNOWN;
-    // received type from remote device
-    if (CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type)
-    {
-        uint32_t code = CA_RESPONSE_CODE(pdu->hdr->coap_hdr_udp_t.code);
-        if (0 == block.m &&
-                (CA_REQUEST_ENTITY_INCOMPLETE != code && CA_REQUEST_ENTITY_TOO_LARGE != code))
-        {
-            int isBlock2 = coap_get_block(pdu, COAP_OPTION_BLOCK2, &block);
-            if (isBlock2)
-            {
-                OIC_LOG(INFO, TAG, "received data is combining block1 and block2");
-                // initialize block number for response message
-                data->block1.num = 0;
-                CADestroyBlockID(blockDataID);
-                return CA_STATUS_OK;
-            }
-            else
-            {
-                OIC_LOG(INFO, TAG, "received data is not bulk data");
-                CAReceiveLastBlock(blockDataID, receivedData);
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return CA_STATUS_OK;
-            }
-        }
-
-        blockWiseStatus = CA_OPTION1_ACK;
-        res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK1, blockWiseStatus);
-        if (CA_STATUS_OK != res)
-        {
-            OIC_LOG(ERROR, TAG, "update has failed");
-            CADestroyBlockID(blockDataID);
-            return res;
-        }
-
-        res = CAUpdateBlockData(data, block, COAP_OPTION_BLOCK1);
-        if (CA_STATUS_OK != res)
-        {
-            OIC_LOG(ERROR, TAG, "update has failed");
-            CARemoveBlockDataFromList(blockDataID);
-            CADestroyBlockID(blockDataID);
-            return res;
-        }
-    }
-    else // CON or NON message
+    uint32_t code = pdu->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
     {
+        // received message type is request
         OIC_LOG_V(INFO, TAG, "num:%d, M:%d", block.num, block.m);
 
         // check the size option
-        bool isSizeOption = CAIsPayloadLengthInPduWithBlockSizeOption(pdu,
-                                                                      COAP_OPTION_SIZE1,
+        bool isSizeOption = CAIsPayloadLengthInPduWithBlockSizeOption(pdu, COAP_OPTION_SIZE1,
                                                                       &(data->payloadLength));
 
-        // check if received payload is exact
-        if (CA_MSG_CONFIRM == pdu->hdr->coap_hdr_udp_t.type)
-        {
-            blockWiseStatus = CACheckBlockErrorType(data, &block, receivedData,
-                                                    COAP_OPTION_BLOCK1, dataLen);
-        }
+        blockWiseStatus = CACheckBlockErrorType(data, &block, receivedData,
+                                                COAP_OPTION_BLOCK1, dataLen);
 
         if (CA_BLOCK_RECEIVED_ALREADY != blockWiseStatus)
         {
@@ -901,18 +859,15 @@ CAResult_t CASetNextBlockOption1(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "update has failed");
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return res;
+                goto exit;
             }
 
-            res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK1, blockWiseStatus);
+            res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK1,
+                                           blockWiseStatus);
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "update has failed");
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return res;
+                goto exit;
             }
 
             // update block data
@@ -920,37 +875,82 @@ CAResult_t CASetNextBlockOption1(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "update has failed");
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return res;
+                goto exit;
             }
         }
 
         // check the blcok-wise transfer status for next step
         if (CA_BLOCK_UNKNOWN == blockWiseStatus || CA_BLOCK_RECEIVED_ALREADY == blockWiseStatus)
         {
-            if (0 == block.m) // Last block is received
+            OIC_LOG_V(DEBUG, TAG, "M bit is %d", block.m);
+
+            if (0 == block.m)
             {
-                OIC_LOG(DEBUG, TAG, "M bit is 0");
-                blockWiseStatus = CA_OPTION1_NO_ACK_LAST_BLOCK;
+                // Last block is received
+                blockWiseStatus = CA_OPTION1_REQUEST_LAST_BLOCK;
+            }
+            else
+            {
+                blockWiseStatus = CA_OPTION1_REQUEST_BLOCK;
+            }
+        }
+    }
+    else
+    {
+        // received message type is response
+        uint32_t code = CA_RESPONSE_CODE(pdu->hdr->coap_hdr_udp_t.code);
+        if (0 == block.m && (CA_REQUEST_ENTITY_INCOMPLETE != code
+                && CA_REQUEST_ENTITY_TOO_LARGE != code))
+        {
+            int isBlock2 = coap_get_block(pdu, COAP_OPTION_BLOCK2, &block);
+            if (isBlock2)
+            {
+                OIC_LOG(INFO, TAG, "received data is combining block1 and block2");
+                // initialize block number for response message
+                data->block1.num = 0;
+                CADestroyBlockID(blockDataID);
+                return CA_STATUS_OK;
             }
             else
             {
-                OIC_LOG(DEBUG, TAG, "M bit is 1");
-                blockWiseStatus = CA_OPTION1_NO_ACK_BLOCK;
+                OIC_LOG(INFO, TAG, "received data is not bulk data");
+                CAReceiveLastBlock(blockDataID, receivedData);
+                res = CA_STATUS_OK;
+                goto exit;
             }
         }
+
+        blockWiseStatus = CA_OPTION1_RESPONSE;
+        res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK1, blockWiseStatus);
+        if (CA_STATUS_OK != res)
+        {
+            OIC_LOG(ERROR, TAG, "update has failed");
+            CADestroyBlockID(blockDataID);
+            return res;
+        }
+
+        res = CAUpdateBlockData(data, block, COAP_OPTION_BLOCK1);
+        if (CA_STATUS_OK != res)
+        {
+            OIC_LOG(ERROR, TAG, "update has failed");
+            goto exit;
+        }
     }
 
     res = CAProcessNextStep(pdu, receivedData, blockWiseStatus, blockDataID);
     if (CA_STATUS_OK != res)
     {
         OIC_LOG(ERROR, TAG, "setting has failed");
-        CARemoveBlockDataFromList(blockDataID);
+        goto exit;
     }
 
     CADestroyBlockID(blockDataID);
     return res;
+
+exit:
+    CARemoveBlockDataFromList(blockDataID);
+    CADestroyBlockID(blockDataID);
+    return res;
 }
 
 // TODO make pdu const after libcoap is updated to support that.
@@ -970,56 +970,44 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
             (CAToken_t)pdu->hdr->coap_hdr_udp_t.token,
             pdu->hdr->coap_hdr_udp_t.token_length,
             endpoint->port);
-
-    if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+    if (NULL == blockDataID || blockDataID->idLength < 1)
     {
         OIC_LOG(ERROR, TAG, "blockId is null");
         CADestroyBlockID(blockDataID);
         return CA_STATUS_FAILED;
     }
 
-    // BlockData data is created if it not existed
-    if (!CAIsBlockDataInList(blockDataID))
+    // Get BlockData data. If does not exist, create a new data
+    CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
+    if (!data)
     {
-        OIC_LOG(DEBUG, TAG, "no msg in list.");
+        OIC_LOG(DEBUG, TAG, "block data doesn't exist in list. create new one");
 
-        CAData_t *data = CACreateNewDataSet(pdu, endpoint);
-        if (!data)
+        CAData_t *cadata = CACreateNewDataSet(pdu, endpoint);
+        if (!cadata)
         {
             OIC_LOG(ERROR, TAG, "data is null");
             CADestroyBlockID(blockDataID);
             return CA_STATUS_FAILED;
         }
 
-        CABlockData_t *currData = CACreateNewBlockData(data);
-        if (!currData)
+        data = CACreateNewBlockData(cadata);
+        if (!data)
         {
-            OIC_LOG(ERROR, TAG, "data is null");
-            CADestroyDataSet(data);
+            OIC_LOG(ERROR, TAG, "failed to create a new block data");
+            CADestroyDataSet(cadata);
             CADestroyBlockID(blockDataID);
             return CA_STATUS_FAILED;
         }
-        CADestroyDataSet(data);
+        CADestroyDataSet(cadata);
     }
 
     // set Block Option Type
-    CAResult_t res = CAUpdateBlockOptionType(blockDataID,
-                                             COAP_OPTION_BLOCK2);
+    CAResult_t res = CAUpdateBlockOptionType(blockDataID, COAP_OPTION_BLOCK2);
     if (CA_STATUS_OK != res)
     {
         OIC_LOG(ERROR, TAG, "update has failed");
-        CARemoveBlockDataFromList(blockDataID);
-        CADestroyBlockID(blockDataID);
-        return res;
-    }
-
-    CABlockData_t *data = CAGetBlockDataFromBlockDataList(blockDataID);
-    if (!data)
-    {
-        OIC_LOG(ERROR, TAG, "getting has failed");
-        CARemoveBlockDataFromList(blockDataID);
-        CADestroyBlockID(blockDataID);
-        return CA_STATUS_FAILED;
+        goto exit;
     }
 
     uint8_t blockWiseStatus = CA_BLOCK_UNKNOWN;
@@ -1031,9 +1019,7 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "update has failed");
-            CARemoveBlockDataFromList(blockDataID);
-            CADestroyBlockID(blockDataID);
-            return res;
+            goto exit;
         }
 
         // first block data have to notify to Application
@@ -1041,29 +1027,48 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "update has failed");
-            CARemoveBlockDataFromList(blockDataID);
-            CADestroyBlockID(blockDataID);
-            return res;
+            goto exit;
         }
         blockWiseStatus = CA_OPTION2_FIRST_BLOCK;
     }
     else
     {
-        // received type from remote device
-        if (CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type ||
-                (CA_MSG_NONCONFIRM == pdu->hdr->coap_hdr_udp_t.type &&
-                        NULL != receivedData->responseInfo))
+        uint32_t code = pdu->hdr->coap_hdr_udp_t.code;
+        if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
+        {
+            // received message type is request
+            OIC_LOG_V(INFO, TAG, "num:%d, M:%d", block.num, block.m);
+
+            blockWiseStatus = CA_OPTION2_REQUEST;
+
+            res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK2, blockWiseStatus);
+            if (CA_STATUS_OK != res)
+            {
+                OIC_LOG(ERROR, TAG, "update has failed");
+                goto exit;
+            }
+
+            res = CAUpdateBlockData(data, block, COAP_OPTION_BLOCK2);
+            if (CA_STATUS_OK != res)
+            {
+                OIC_LOG(ERROR, TAG, "update has failed");
+                goto exit;
+            }
+        }
+        else
         {
-            OIC_LOG(DEBUG, TAG, "received ACK or NON");
+            // received message type is response
+            OIC_LOG(DEBUG, TAG, "received response message with block option2");
 
             // check the size option
             bool isSizeOption = CAIsPayloadLengthInPduWithBlockSizeOption(pdu,
                                                                           COAP_OPTION_SIZE2,
                                                                           &(data->payloadLength));
 
-            // check if received payload is exact
-            if (CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type)
+            uint32_t code = CA_RESPONSE_CODE(pdu->hdr->coap_hdr_udp_t.code);
+            if (CA_REQUEST_ENTITY_INCOMPLETE != code && CA_REQUEST_ENTITY_TOO_LARGE != code)
             {
+                // check if received payload is exact
                 blockWiseStatus = CACheckBlockErrorType(data, &block, receivedData,
                                                         COAP_OPTION_BLOCK2, dataLen);
             }
@@ -1076,9 +1081,7 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "update has failed");
-                    CARemoveBlockDataFromList(blockDataID);
-                    CADestroyBlockID(blockDataID);
-                    return res;
+                    goto exit;
                 }
             }
 
@@ -1093,15 +1096,7 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
                         CA_BLOCK_RECEIVED_ALREADY == blockWiseStatus)
                 {
                     OIC_LOG(DEBUG, TAG, "M bit is 1");
-
-                    if (CA_MSG_ACKNOWLEDGE == pdu->hdr->coap_hdr_udp_t.type)
-                    {
-                        blockWiseStatus = CA_OPTION2_ACK;
-                    }
-                    else
-                    {
-                        blockWiseStatus = CA_OPTION2_NON;
-                    }
+                    blockWiseStatus = CA_OPTION2_RESPONSE;
                 }
 
                 res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK2,
@@ -1109,58 +1104,33 @@ CAResult_t CASetNextBlockOption2(coap_pdu_t *pdu, const CAEndpoint_t *endpoint,
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "update has failed");
-                    CARemoveBlockDataFromList(blockDataID);
-                    CADestroyBlockID(blockDataID);
-                    return res;
+                    goto exit;
                 }
 
                 res = CAUpdateBlockData(data, block, COAP_OPTION_BLOCK2);
                 if (CA_STATUS_OK != res)
                 {
                     OIC_LOG(ERROR, TAG, "update has failed");
-                    CARemoveBlockDataFromList(blockDataID);
-                    CADestroyBlockID(blockDataID);
-                    return res;
+                    goto exit;
                 }
             }
         }
-        else // CON message and so on.
-        {
-            OIC_LOG_V(INFO, TAG, "num:%d, M:%d", block.num, block.m);
-
-            blockWiseStatus = CA_OPTION2_CON;
-
-            res = CAUpdateBlockOptionItems(data, pdu, &block, COAP_OPTION_BLOCK2, blockWiseStatus);
-            if (CA_STATUS_OK != res)
-            {
-                OIC_LOG(ERROR, TAG, "update has failed");
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return res;
-            }
-
-            res = CAUpdateBlockData(data, block, COAP_OPTION_BLOCK2);
-            if (CA_STATUS_OK != res)
-            {
-                OIC_LOG(ERROR, TAG, "update has failed");
-                CARemoveBlockDataFromList(blockDataID);
-                CADestroyBlockID(blockDataID);
-                return res;
-            }
-        }
     }
 
     res = CAProcessNextStep(pdu, receivedData, blockWiseStatus, blockDataID);
     if (CA_STATUS_OK != res)
     {
         OIC_LOG(ERROR, TAG, "setting has failed");
-        CARemoveBlockDataFromList(blockDataID);
-        CADestroyBlockID(blockDataID);
-        return res;
+        goto exit;
     }
 
     CADestroyBlockID(blockDataID);
-    return CA_STATUS_OK;
+    return CA_STATUS_OK;
+
+exit:
+    CARemoveBlockDataFromList(blockDataID);
+    CADestroyBlockID(blockDataID);
+    return res;
 }
 
 CAResult_t CAUpdateBlockOptionItems(CABlockData_t *currData, const coap_pdu_t *pdu,
@@ -1190,7 +1160,7 @@ CAResult_t CAUpdateBlockOptionItems(CABlockData_t *currData, const coap_pdu_t *p
         // update block option items
         switch (status)
         {
-            case CA_OPTION1_ACK:
+            case CA_OPTION1_RESPONSE:
                 if (currData->block1.num > block->num)
                 {
                     OIC_LOG(ERROR, TAG, "received incorrect block num");
@@ -1198,14 +1168,10 @@ CAResult_t CAUpdateBlockOptionItems(CABlockData_t *currData, const coap_pdu_t *p
                 }
                 block->num++;
                 break;
-            case CA_OPTION2_NON:
-                block->num++;
-                block->m = 0;
-                break;
-            case CA_OPTION2_CON:
+            case CA_OPTION2_REQUEST:
                 block->m = 0;
                 break;
-            case CA_OPTION2_ACK:
+            case CA_OPTION2_RESPONSE:
                 if (currData->block2.num > block->num)
                 {
                     OIC_LOG(ERROR, TAG, "received incorrect block num");
@@ -1235,7 +1201,7 @@ CAResult_t CAUpdateBlockOptionItems(CABlockData_t *currData, const coap_pdu_t *p
         if (CA_BLOCK_INCOMPLETE != status && CA_BLOCK_TOO_LARGE != status)
         {
             // negotiate block size
-            res = CANegotiateBlockSize(currData, block, pdu->hdr->coap_hdr_udp_t.type, blockType);
+            res = CANegotiateBlockSize(currData, block, pdu, blockType);
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "negotiation has failed");
@@ -1246,12 +1212,11 @@ CAResult_t CAUpdateBlockOptionItems(CABlockData_t *currData, const coap_pdu_t *p
     return res;
 }
 
-CAResult_t CAGetMoreBitFromBlock(size_t payloadLen, coap_block_t *block)
+CAResult_t CASetMoreBitFromBlock(size_t payloadLen, coap_block_t *block)
 {
     VERIFY_NON_NULL(block, TAG, "block");
 
-    if ((size_t)((block->num + 1) << (block->szx + BLOCK_NUMBER_IDX))
-        < payloadLen)
+    if ((size_t) ((block->num + 1) << (block->szx + BLOCK_NUMBER_IDX)) < payloadLen)
     {
         OIC_LOG(DEBUG, TAG, "Set the M-bit(1)");
         block->m = 1;
@@ -1266,18 +1231,27 @@ CAResult_t CAGetMoreBitFromBlock(size_t payloadLen, coap_block_t *block)
 }
 
 CAResult_t CANegotiateBlockSize(CABlockData_t *currData, coap_block_t *block,
-                                CAMessageType_t msgType, uint16_t blockType)
+                                coap_pdu_t *pdu, uint16_t blockType)
 {
     OIC_LOG(DEBUG, TAG, "IN-NegotiateBlockSize");
 
     VERIFY_NON_NULL(currData, TAG, "currData");
     VERIFY_NON_NULL(block, TAG, "block");
+    VERIFY_NON_NULL(pdu, TAG, "pdu");
+    VERIFY_NON_NULL(pdu->hdr, TAG, "pdu->hdr");
+
+    bool isReqMsg = false;
+    uint32_t code = pdu->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
+    {
+        isReqMsg = true;
+    }
 
     // #1. check the block option type
     if (COAP_OPTION_BLOCK2 == blockType)
     {
         // #2. check the message type
-        if (CA_MSG_ACKNOWLEDGE == msgType)
+        if (!isReqMsg)
         {
             if (block->szx > currData->block2.szx)
             {
@@ -1303,7 +1277,7 @@ CAResult_t CANegotiateBlockSize(CABlockData_t *currData, coap_block_t *block,
     }
     else if (COAP_OPTION_BLOCK1 == blockType)
     {
-        if (CA_MSG_ACKNOWLEDGE == msgType)
+        if (!isReqMsg)
         {
             if (block->szx < currData->block1.szx)
             {
@@ -1364,22 +1338,20 @@ CAResult_t CAUpdateBlockData(CABlockData_t *currData, coap_block_t block,
 CAResult_t CAUpdateMessageId(coap_pdu_t *pdu, const CABlockDataID_t *blockID)
 {
     VERIFY_NON_NULL(pdu, TAG, "pdu");
+    VERIFY_NON_NULL(pdu->hdr, TAG, "pdu->hdr");
     VERIFY_NON_NULL(blockID, TAG, "blockID");
 
-    // if CON message is sent, update messageId in block-wise transfer list
-    if (CA_MSG_CONFIRM == pdu->hdr->coap_hdr_udp_t.type)
+    // if message is sent, update messageId in block-wise transfer list
+    CAData_t * cadata = CAGetDataSetFromBlockDataList(blockID);
+    if (!cadata)
     {
-        CAData_t * cadata = CAGetDataSetFromBlockDataList(blockID);
-        if (!cadata)
-        {
-            OIC_LOG(ERROR, TAG, "CAData is unavailable");
-            return CA_STATUS_FAILED;
-        }
+        OIC_LOG(ERROR, TAG, "CAData is unavailable");
+        return CA_STATUS_FAILED;
+    }
 
-        if (cadata->requestInfo)
-        {
-            cadata->requestInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
-        }
+    if (cadata->requestInfo)
+    {
+        cadata->requestInfo->info.messageId = pdu->hdr->coap_hdr_udp_t.id;
     }
 
     return CA_STATUS_OK;
@@ -1396,35 +1368,33 @@ CAResult_t CAAddBlockOption(coap_pdu_t **pdu, const CAInfo_t *info,
     VERIFY_NON_NULL(endpoint, TAG, "endpoint");
     VERIFY_NON_NULL(options, TAG, "options");
 
+    CAResult_t res = CA_STATUS_OK;
     size_t dataLength = 0;
     if (info->payload)
     {
         dataLength = info->payloadSize;
-        OIC_LOG_V(DEBUG, TAG, "dataLength - %d", dataLength);
-    }
-
-    OIC_LOG_V(DEBUG, TAG, "previous payload - %s", (*pdu)->data);
-
-    CAResult_t res = CA_STATUS_OK;
-    uint32_t code = CA_RESPONSE_CODE((*pdu)->hdr->coap_hdr_udp_t.code);
-    if (CA_REQUEST_ENTITY_INCOMPLETE == code)
-    {
-        OIC_LOG(INFO, TAG, "don't use option");
-        return res;
+        OIC_LOG_V(DEBUG, TAG, "dataLength - %zu", dataLength);
     }
 
     CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
             (CAToken_t)(*pdu)->hdr->coap_hdr_udp_t.token,
             (*pdu)->hdr->coap_hdr_udp_t.token_length,
             endpoint->port);
-
-    if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+    if (NULL == blockDataID || blockDataID->idLength < 1)
     {
         OIC_LOG(ERROR, TAG, "blockId is null");
         res = CA_STATUS_FAILED;
         goto exit;
     }
 
+    uint32_t repCode = CA_RESPONSE_CODE((*pdu)->hdr->coap_hdr_udp_t.code);
+    if (CA_REQUEST_ENTITY_INCOMPLETE == repCode)
+    {
+        OIC_LOG(INFO, TAG, "don't use option");
+        res = CA_STATUS_OK;
+        goto exit;
+    }
+
     uint8_t blockType = CAGetBlockOptionType(blockDataID);
     if (COAP_OPTION_BLOCK2 == blockType)
     {
@@ -1469,28 +1439,33 @@ CAResult_t CAAddBlockOption(coap_pdu_t **pdu, const CAInfo_t *info,
         if (!coap_add_data(*pdu, dataLength, (const unsigned char *) info->payload))
         {
             OIC_LOG(INFO, TAG, "it have to use block");
+            res = CA_STATUS_FAILED;
+            goto exit;
         }
         else
         {
             OIC_LOG(INFO, TAG, "not Blockwise Transfer");
-            goto exit;
         }
     }
 
-    // if received message type is RESET from remote device,
-    // we have to use the updated message id to find token.
-    res = CAUpdateMessageId(*pdu, blockDataID);
-    if (CA_STATUS_OK != res)
+    uint32_t code = (*pdu)->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
     {
-        OIC_LOG(ERROR, TAG, "fail to update CON message id ");
-        goto exit;
+        // if received message type is RESET from remote device,
+        // we have to use the updated message id of request message to find token.
+        CABlockData_t *blockData = CAGetBlockDataFromBlockDataList(blockDataID);
+        if (blockData)
+        {
+            res = CAUpdateMessageId(*pdu, blockDataID);
+            if (CA_STATUS_OK != res)
+            {
+                OIC_LOG(ERROR, TAG, "fail to update message id");
+                goto exit;
+            }
+        }
     }
 
 exit:
-    if (CA_ADAPTER_IP == endpoint->adapter && 0 == endpoint->port)
-    {
-        CARemoveBlockDataFromList(blockDataID);
-    }
     CADestroyBlockID(blockDataID);
     OIC_LOG(DEBUG, TAG, "OUT-AddBlockOption");
     return res;
@@ -1516,34 +1491,28 @@ CAResult_t CAAddBlockOption2(coap_pdu_t **pdu, const CAInfo_t *info, size_t data
         return CA_STATUS_FAILED;
     }
 
-    CALogBlockInfo(block2);
-
-    uint8_t code = 0;
-    if (CA_MSG_ACKNOWLEDGE == (*pdu)->hdr->coap_hdr_udp_t.type ||
-            (CA_MSG_NONCONFIRM == (*pdu)->hdr->coap_hdr_udp_t.type &&
-                    CA_GET != (*pdu)->hdr->coap_hdr_udp_t.code))
+    CAResult_t res = CA_STATUS_OK;
+    uint32_t code = (*pdu)->hdr->coap_hdr_udp_t.code;
+    if (CA_GET != code && CA_POST != code && CA_PUT != code && CA_DELETE != code)
     {
-        CAGetMoreBitFromBlock(dataLength, block2);
-        CALogBlockInfo(block2);
+        CASetMoreBitFromBlock(dataLength, block2);
 
         // if block number is 0, add size2 option
         if (0 == block2->num)
         {
-            CAResult_t res = CAAddBlockSizeOption(*pdu, COAP_OPTION_SIZE2, dataLength, options);
+            res = CAAddBlockSizeOption(*pdu, COAP_OPTION_SIZE2, dataLength, options);
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "add has failed");
-                CARemoveBlockDataFromList(blockID);
-                return res;
+                goto exit;
             }
         }
 
-        CAResult_t res = CAAddBlockOptionImpl(block2, COAP_OPTION_BLOCK2, options);
+        res = CAAddBlockOptionImpl(block2, COAP_OPTION_BLOCK2, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
         if (block1->num)
@@ -1553,8 +1522,7 @@ CAResult_t CAAddBlockOption2(coap_pdu_t **pdu, const CAInfo_t *info, size_t data
             if (CA_STATUS_OK != res)
             {
                 OIC_LOG(ERROR, TAG, "add has failed");
-                CARemoveBlockDataFromList(blockID);
-                return res;
+                goto exit;
             }
             // initialize block number
             block1->num = 0;
@@ -1564,8 +1532,7 @@ CAResult_t CAAddBlockOption2(coap_pdu_t **pdu, const CAInfo_t *info, size_t data
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
         if (!coap_add_block(*pdu, dataLength, (const unsigned char *) info->payload,
@@ -1575,62 +1542,38 @@ CAResult_t CAAddBlockOption2(coap_pdu_t **pdu, const CAInfo_t *info, size_t data
             return CA_STATUS_FAILED;
         }
 
+        CALogBlockInfo(block2);
+
         if (!block2->m)
         {
             // if sent message is last response block message, remove data
             CARemoveBlockDataFromList(blockID);
         }
-        else
-        {
-            if (CA_MSG_NONCONFIRM == (*pdu)->hdr->coap_hdr_udp_t.type)
-            {
-                OIC_LOG(DEBUG, TAG, "NON, send next block..");
-                // update block data
-                block2->num++;
-                CAResult_t res = CAProcessNextStep(*pdu, NULL,
-                                                   CA_SENT_PREVIOUS_NON_MSG,
-                                                   blockID);
-                if (CA_STATUS_OK != res)
-                {
-                    OIC_LOG(ERROR, TAG, "failed to process next step");
-                    CARemoveBlockDataFromList(blockID);
-                    return res;
-                }
-            }
-        }
     }
     else
     {
-        OIC_LOG(DEBUG, TAG, "option2, not ACK msg");
-        CAResult_t res = CAAddBlockOptionImpl(block2, COAP_OPTION_BLOCK2, options);
+        OIC_LOG(DEBUG, TAG, "option2, not response msg");
+        res = CAAddBlockOptionImpl(block2, COAP_OPTION_BLOCK2, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
         res = CAAddOptionToPDU(*pdu, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
+        CALogBlockInfo(block2);
     }
 
     return CA_STATUS_OK;
 
-error:
-    OIC_LOG_V(ERROR, TAG, "error : %d", code);
-
-    char* phrase = coap_response_phrase(code);
-    if(phrase)
-    {
-        coap_add_data(*pdu, strlen(phrase),
-                      (unsigned char *) phrase);
-    }
-    return CA_STATUS_FAILED;
+exit:
+    CARemoveBlockDataFromList(blockID);
+    return res;
 }
 
 CAResult_t CAAddBlockOption1(coap_pdu_t **pdu, const CAInfo_t *info, size_t dataLength,
@@ -1652,115 +1595,96 @@ CAResult_t CAAddBlockOption1(coap_pdu_t **pdu, const CAInfo_t *info, size_t data
         return CA_STATUS_FAILED;
     }
 
-    CALogBlockInfo(block1);
-
-    if (CA_MSG_ACKNOWLEDGE == (*pdu)->hdr->coap_hdr_udp_t.type)
+    CAResult_t res = CA_STATUS_OK;
+    uint32_t code = (*pdu)->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
     {
-        OIC_LOG(DEBUG, TAG, "option1 and ACK msg..");
-        CAResult_t res = CAAddBlockOptionImpl(block1, COAP_OPTION_BLOCK1, options);
+        CASetMoreBitFromBlock(dataLength, block1);
+
+        // if block number is 0, add size1 option
+        if (0 == block1->num)
+        {
+            res = CAAddBlockSizeOption(*pdu, COAP_OPTION_SIZE1, dataLength, options);
+            if (CA_STATUS_OK != res)
+            {
+                OIC_LOG(ERROR, TAG, "add has failed");
+                goto exit;
+            }
+        }
+
+        // add block option to option list.
+        res = CAAddBlockOptionImpl(block1, COAP_OPTION_BLOCK1, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
+        // add option list to pdu.
         res = CAAddOptionToPDU(*pdu, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
-        if (!coap_add_data(*pdu, dataLength, (const unsigned char *) info->payload))
+        // add the payload data as the block size.
+        if (!coap_add_block(*pdu, dataLength, (const unsigned char *) info->payload, block1->num,
+                            block1->szx))
         {
-            OIC_LOG(ERROR, TAG, "failed to add payload");
+            OIC_LOG(ERROR, TAG, "Data length is smaller than the start index");
             return CA_STATUS_FAILED;
         }
-
-        // reset block-list after write block
-        if (0 == block1->m)
-        {
-            // remove data from list
-            CAResult_t res = CARemoveBlockDataFromList(blockID);
-            if (CA_STATUS_OK != res)
-            {
-                OIC_LOG(ERROR, TAG, "remove has failed");
-                return res;
-            }
-        }
     }
     else
     {
-        CAGetMoreBitFromBlock(dataLength, block1);
-
-        CAResult_t res = CA_STATUS_OK;
-        // if block number is 0, add size1 option
-        if (0 == block1->num)
-        {
-            res = CAAddBlockSizeOption(*pdu, COAP_OPTION_SIZE1, dataLength, options);
-            if (CA_STATUS_OK != res)
-            {
-                OIC_LOG(ERROR, TAG, "add has failed");
-                CARemoveBlockDataFromList(blockID);
-                return res;
-            }
-        }
+        OIC_LOG(DEBUG, TAG, "received response message with block option1");
 
+        // add block option to option list.
         res = CAAddBlockOptionImpl(block1, COAP_OPTION_BLOCK1, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
+        // add option list to pdu.
         res = CAAddOptionToPDU(*pdu, options);
         if (CA_STATUS_OK != res)
         {
             OIC_LOG(ERROR, TAG, "add has failed");
-            CARemoveBlockDataFromList(blockID);
-            return res;
+            goto exit;
         }
 
-        CALogBlockInfo(block1);
-
-        if (!coap_add_block(*pdu, dataLength, (const unsigned char *) info->payload,
-                            block1->num, block1->szx))
+        // add the payload data as the block size.
+        if (!coap_add_data(*pdu, dataLength, (const unsigned char *) info->payload))
         {
-            OIC_LOG(ERROR, TAG, "Data length is smaller than the start index");
+            OIC_LOG(ERROR, TAG, "failed to add payload");
             return CA_STATUS_FAILED;
         }
 
-        // check the message type and if message type is NON, next block message will be sent
-        if (CA_MSG_NONCONFIRM == (*pdu)->hdr->coap_hdr_udp_t.type)
+        // if it is last block message, remove block data from list.
+        if (0 == block1->m)
         {
-            if (block1->m)
-            {
-                OIC_LOG(DEBUG, TAG, "NON, send next block..");
-                // update block data
-                block1->num++;
-                CAResult_t res = CAProcessNextStep(*pdu, NULL,
-                                                   CA_SENT_PREVIOUS_NON_MSG,
-                                                   blockID);
-                if (CA_STATUS_OK != res)
-                {
-                    OIC_LOG(ERROR, TAG, "failed to process next step");
-                    CARemoveBlockDataFromList(blockID);
-                    return res;
-                }
-            }
-            else
+            // remove data from list
+            res = CARemoveBlockDataFromList(blockID);
+            if (CA_STATUS_OK != res)
             {
-                CARemoveBlockDataFromList(blockID);
+                OIC_LOG(ERROR, TAG, "remove has failed");
+                return res;
             }
         }
     }
 
+    CALogBlockInfo(block1);
+
     OIC_LOG(DEBUG, TAG, "OUT-AddBlockOption1");
 
     return CA_STATUS_OK;
+
+exit:
+    CARemoveBlockDataFromList(blockID);
+    return res;
 }
 
 CAResult_t CAAddBlockOptionImpl(coap_block_t *block, uint8_t blockType,
@@ -1866,7 +1790,7 @@ bool CAIsPayloadLengthInPduWithBlockSizeOption(coap_pdu_t *pdu,
         *totalPayloadLen = coap_decode_var_bytes(COAP_OPT_VALUE(option),
                                                  COAP_OPT_LENGTH(option));
 
-        OIC_LOG_V(DEBUG, TAG, "the total payload length to be received is [%d]bytes",
+        OIC_LOG_V(DEBUG, TAG, "the total payload length to be received is [%zu]bytes",
                   *totalPayloadLen);
 
         return true;
@@ -1895,8 +1819,7 @@ uint8_t CACheckBlockErrorType(CABlockData_t *currData, coap_block_t *receivedBlo
     if (COAP_OPTION_BLOCK1 == blockType)
     {
         size_t prePayloadLen = currData->receivedPayloadLen;
-        if (prePayloadLen != (size_t)BLOCK_SIZE(receivedBlock->szx)
-            * receivedBlock->num)
+        if (prePayloadLen != (size_t) BLOCK_SIZE(receivedBlock->szx) * receivedBlock->num)
         {
             if (receivedBlock->num > currData->block1.num + 1)
             {
@@ -1925,8 +1848,7 @@ uint8_t CACheckBlockErrorType(CABlockData_t *currData, coap_block_t *receivedBlo
 
     // #3. check if error check logic is required
     size_t optionLen = dataLen - blockPayloadLen;
-    if (receivedBlock->m && blockPayloadLen !=
-        (size_t)BLOCK_SIZE(receivedBlock->szx))
+    if (receivedBlock->m && blockPayloadLen != (size_t) BLOCK_SIZE(receivedBlock->szx))
     {
         // 413 Error handling of too large entity
         if (COAP_MAX_PDU_SIZE < BLOCK_SIZE(receivedBlock->szx) + optionLen)
@@ -1941,14 +1863,8 @@ uint8_t CACheckBlockErrorType(CABlockData_t *currData, coap_block_t *receivedBlo
                 if (COAP_MAX_PDU_SIZE >= BLOCK_SIZE(size) + optionLen)
                 {
                     OIC_LOG_V(ERROR, TAG, "replace sze with %d", size);
-                    if (COAP_OPTION_BLOCK2 == blockType)
-                    {
-                        currData->block2.szx = size;
-                    }
-                    else
-                    {
-                        currData->block1.szx = size;
-                    }
+                    currData->block2.szx = size;
+                    currData->block1.szx = size;
                     break;
                 }
             }
@@ -2054,7 +1970,7 @@ CAResult_t CAUpdatePayloadData(CABlockData_t *currData, const CAData_t *received
         // update received payload length
         currData->receivedPayloadLen += blockPayloadLen;
 
-        OIC_LOG_V(DEBUG, TAG, "updated payload: %s, len: %d", currData->payload,
+        OIC_LOG_V(DEBUG, TAG, "updated payload: %s, len: %zu", currData->payload,
                   currData->receivedPayloadLen);
     }
 
@@ -2068,33 +1984,69 @@ CAData_t* CACreateNewDataSet(const coap_pdu_t *pdu, const CAEndpoint_t *endpoint
     VERIFY_NON_NULL_RET(pdu->hdr, TAG, "pdu->hdr", NULL);
     VERIFY_NON_NULL_RET(endpoint, TAG, "endpoint", NULL);
 
-    CAInfo_t responseData = { .tokenLength = pdu->hdr->coap_hdr_udp_t.token_length };
-    responseData.token = (CAToken_t) OICMalloc(responseData.tokenLength);
-    if (!responseData.token)
+    CARequestInfo_t* requestInfo = NULL;
+    CAResponseInfo_t* responseInfo = NULL;
+
+    uint32_t code = pdu->hdr->coap_hdr_udp_t.code;
+    if (CA_GET == code || CA_POST == code || CA_PUT == code || CA_DELETE == code)
     {
-        OIC_LOG(ERROR, TAG, "out of memory");
-        return NULL;
-    }
-    memcpy(responseData.token, pdu->hdr->coap_hdr_udp_t.token, responseData.tokenLength);
+        CAInfo_t responseData = { .tokenLength = pdu->hdr->coap_hdr_udp_t.token_length };
+        responseData.token = (CAToken_t) OICMalloc(responseData.tokenLength);
+        if (!responseData.token)
+        {
+            OIC_LOG(ERROR, TAG, "out of memory");
+            return NULL;
+        }
+        memcpy(responseData.token, pdu->hdr->coap_hdr_udp_t.token, responseData.tokenLength);
 
-    CAResponseInfo_t* responseInfo = (CAResponseInfo_t*) OICCalloc(1, sizeof(CAResponseInfo_t));
-    if (!responseInfo)
+        responseInfo = (CAResponseInfo_t*) OICCalloc(1, sizeof(CAResponseInfo_t));
+        if (!responseInfo)
+        {
+            OIC_LOG(ERROR, TAG, "out of memory");
+            OICFree(responseData.token);
+            return NULL;
+        }
+        responseInfo->info = responseData;
+    }
+    else
     {
-        OIC_LOG(ERROR, TAG, "out of memory");
-        OICFree(responseData.token);
-        return NULL;
+        CAInfo_t requestData = { .tokenLength = pdu->hdr->coap_hdr_udp_t.token_length };
+        requestData.token = (CAToken_t) OICMalloc(requestData.tokenLength);
+        if (!requestData.token)
+        {
+            OIC_LOG(ERROR, TAG, "out of memory");
+            return NULL;
+        }
+        memcpy(requestData.token, pdu->hdr->coap_hdr_udp_t.token, requestData.tokenLength);
+
+        requestInfo = (CARequestInfo_t*) OICCalloc(1, sizeof(CARequestInfo_t));
+        if (!requestInfo)
+        {
+            OIC_LOG(ERROR, TAG, "out of memory");
+            OICFree(requestData.token);
+            return NULL;
+        }
+        requestInfo->info = requestData;
+
+        // get resource uri information from received response message
+        // to send next request message to remote device
+        CAResponseInfo_t resInfo = { 0 };
+        CAGetResponseInfoFromPDU(pdu, &resInfo, endpoint);
+
+        requestInfo->method = CA_GET;
+        requestInfo->info.resourceUri = resInfo.info.resourceUri;
     }
-    responseInfo->info = responseData;
 
     CAData_t *data = (CAData_t *) OICCalloc(1, sizeof(CAData_t));
     if (!data)
     {
         OIC_LOG(ERROR, TAG, "out of memory");
+        OICFree(requestInfo);
         OICFree(responseInfo);
         return NULL;
     }
 
-    data->requestInfo = NULL;
+    data->requestInfo = requestInfo;
     data->responseInfo = responseInfo;
     data->remoteEndpoint = CACloneEndpoint(endpoint);
     data->type = SEND_TYPE_UNICAST;
@@ -2118,8 +2070,7 @@ CAData_t *CACloneCAData(const CAData_t *data)
     {
         clone->requestInfo = CACloneRequestInfo(data->requestInfo);
     }
-
-    if (data->responseInfo)
+    else if (data->responseInfo)
     {
         clone->responseInfo = CACloneResponseInfo(data->responseInfo);
     }
@@ -2144,6 +2095,11 @@ CAResult_t CAUpdatePayloadToCAData(CAData_t *data, const CAPayload_t payload,
     switch (data->dataType)
     {
         case CA_REQUEST_DATA:
+            if (!data->requestInfo)
+            {
+                OIC_LOG(ERROR, TAG, "request info is null");
+                return CA_STATUS_FAILED;
+            }
             // allocate payload field
             newPayload = OICRealloc(data->requestInfo->info.payload, payloadLen);
             if (!newPayload)
@@ -2157,6 +2113,11 @@ CAResult_t CAUpdatePayloadToCAData(CAData_t *data, const CAPayload_t payload,
             break;
 
         case CA_RESPONSE_DATA:
+            if (!data->responseInfo)
+            {
+                OIC_LOG(ERROR, TAG, "response info is null");
+                return CA_STATUS_FAILED;
+            }
             // allocate payload field
             newPayload = OICRealloc(data->responseInfo->info.payload, payloadLen);
             if (!newPayload)
@@ -2226,6 +2187,7 @@ CAResult_t CAHandleBlockErrorResponse(coap_block_t *block, uint16_t blockType,
             break;
         default:
             OIC_LOG_V(ERROR, TAG, "there is no Error Code of BWT[%d]", responseResult);
+            return CA_STATUS_FAILED;
     }
 
     OIC_LOG(DEBUG, TAG, "OUT-HandleBlockErrorRes");
@@ -2352,7 +2314,7 @@ CAResult_t CAGetTokenFromBlockDataList(const coap_pdu_t *pdu, const CAEndpoint_t
     ca_mutex_unlock(g_context.blockDataListMutex);
 
     OIC_LOG(DEBUG, TAG, "OUT-CAGetTokenFromBlockDataList");
-    return CA_STATUS_OK;
+    return CA_STATUS_FAILED;
 }
 
 CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **blockData)
@@ -2360,13 +2322,40 @@ CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **
     VERIFY_NON_NULL(sendData, TAG, "sendData");
     VERIFY_NON_NULL(blockData, TAG, "blockData");
 
+    CABlockDataID_t* blockDataID;
+    if(sendData->requestInfo)
+    {
+        blockDataID = CACreateBlockDatablockId(
+                            (CAToken_t)sendData->requestInfo->info.token,
+                            sendData->requestInfo->info.tokenLength,
+                            sendData->remoteEndpoint->port);
+    }
+    else if(sendData->responseInfo)
+    {
+        blockDataID = CACreateBlockDatablockId(
+                            (CAToken_t)sendData->responseInfo->info.token,
+                            sendData->responseInfo->info.tokenLength,
+                            sendData->remoteEndpoint->port);
+    }
+    else
+    {
+        OIC_LOG(ERROR, TAG, "sendData doesn't have requestInfo or responseInfo");
+        return CA_STATUS_FAILED;
+    }
+
+    if (NULL == blockDataID || blockDataID->idLength < 1)
+    {
+        OIC_LOG(ERROR, TAG, "blockId is null");
+        CADestroyBlockID(blockDataID);
+        return CA_STATUS_FAILED;
+    }
+
     ca_mutex_lock(g_context.blockDataListMutex);
 
     size_t len = u_arraylist_length(g_context.dataList);
     for (size_t i = 0; i < len; i++)
     {
         CABlockData_t *currData = (CABlockData_t *) u_arraylist_get(g_context.dataList, i);
-
         if (!currData)
         {
             continue;
@@ -2380,25 +2369,11 @@ CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **
                     && currData->blockDataId->idLength > 0
                     && NULL != sendData->requestInfo->info.token)
             {
-                CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
-                        (CAToken_t)sendData->requestInfo->info.token,
-                        sendData->requestInfo->info.tokenLength,
-                        sendData->remoteEndpoint->port);
-
-                if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
-                {
-                    OIC_LOG(ERROR, TAG, "blockId is null");
-                    CADestroyBlockID(blockDataID);
-                    return CA_STATUS_FAILED;
-                }
-
                 if (CABlockidMatches(currData, blockDataID))
                 {
                     OIC_LOG(ERROR, TAG, "already sent");
-                    CADestroyBlockID(blockDataID);
                     continue;
                 }
-                CADestroyBlockID(blockDataID);
             }
         }
         else if (sendData->responseInfo) // sendData is responseMessage
@@ -2409,18 +2384,6 @@ CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **
                     && currData->blockDataId->idLength > 0
                     && NULL != sendData->responseInfo->info.token)
             {
-                CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
-                        (CAToken_t)sendData->responseInfo->info.token,
-                        sendData->responseInfo->info.tokenLength,
-                        sendData->remoteEndpoint->port);
-
-                if(NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
-                {
-                    OIC_LOG(ERROR, TAG, "blockId is null");
-                    CADestroyBlockID(blockDataID);
-                    return CA_STATUS_FAILED;
-                }
-
                 if (CABlockidMatches(currData, blockDataID))
                 {
                     // set sendData
@@ -2431,11 +2394,10 @@ CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **
                     }
                     currData->sentData = CACloneCAData(sendData);
                     *blockData = currData;
-                    CADestroyBlockID(blockDataID);
                     ca_mutex_unlock(g_context.blockDataListMutex);
+                    CADestroyBlockID(blockDataID);
                     return CA_STATUS_OK;
                 }
-                CADestroyBlockID(blockDataID);
             }
         }
         else
@@ -2446,6 +2408,8 @@ CAResult_t CACheckBlockDataValidation(const CAData_t *sendData, CABlockData_t **
     }
     ca_mutex_unlock(g_context.blockDataListMutex);
 
+    CADestroyBlockID(blockDataID);
+
     return CA_STATUS_FAILED;
 }
 
@@ -2470,8 +2434,7 @@ CABlockData_t *CAGetBlockDataFromBlockDataList(const CABlockDataID_t *blockID)
     return NULL;
 }
 
-coap_block_t *CAGetBlockOption(const CABlockDataID_t *blockID,
-                               uint16_t blockType)
+coap_block_t *CAGetBlockOption(const CABlockDataID_t *blockID, uint16_t blockType)
 {
     OIC_LOG(DEBUG, TAG, "IN-GetBlockOption");
     VERIFY_NON_NULL_RET(blockID, TAG, "blockID", NULL);
@@ -2490,7 +2453,7 @@ coap_block_t *CAGetBlockOption(const CABlockDataID_t *blockID,
             {
                 return &currData->block2;
             }
-            else
+            else if (COAP_OPTION_BLOCK1 == blockType)
             {
                 return &currData->block1;
             }
@@ -2545,7 +2508,7 @@ CABlockData_t *CACreateNewBlockData(const CAData_t *sendData)
     data->block1.szx = CA_DEFAULT_BLOCK_SIZE;
     data->block2.szx = CA_DEFAULT_BLOCK_SIZE;
     data->sentData = CACloneCAData(sendData);
-    if(!data->sentData)
+    if (!data->sentData)
     {
         OIC_LOG(ERROR, TAG, PCF("memory alloc has failed"));
         OICFree(data);
@@ -2560,16 +2523,23 @@ CABlockData_t *CACreateNewBlockData(const CAData_t *sendData)
         tokenLength = data->sentData->requestInfo->info.tokenLength;
         token = data->sentData->requestInfo->info.token;
     }
-    else if(data->sentData->responseInfo)
+    else if (data->sentData->responseInfo)
     {
         tokenLength = data->sentData->responseInfo->info.tokenLength;
         token = data->sentData->responseInfo->info.token;
     }
 
-    CABlockDataID_t* blockDataID = CACreateBlockDatablockId(
-            token, tokenLength,
-            data->sentData->remoteEndpoint->port);
-    if (NULL == blockDataID || NULL == blockDataID->id || blockDataID->idLength < 1)
+    if (!data->sentData->remoteEndpoint)
+    {
+        OIC_LOG(ERROR, TAG, "remoteEndpoint is null");
+        CADestroyDataSet(data->sentData);
+        OICFree(data);
+        return NULL;
+    }
+
+    CABlockDataID_t* blockDataID = CACreateBlockDatablockId(token, tokenLength,
+                                                            data->sentData->remoteEndpoint->port);
+    if (NULL == blockDataID || blockDataID->idLength < 1)
     {
         OIC_LOG(ERROR, TAG, "blockId is null");
         CADestroyBlockID(blockDataID);
@@ -2619,10 +2589,7 @@ CAResult_t CARemoveBlockDataFromList(const CABlockDataID_t *blockID)
             }
 
             // destroy memory
-            if (currData->sentData)
-            {
-                CADestroyDataSet(currData->sentData);
-            }
+            CADestroyDataSet(currData->sentData);
             CADestroyBlockID(currData->blockDataId);
             OICFree(currData->payload);
             OICFree(currData);
@@ -2635,30 +2602,6 @@ CAResult_t CARemoveBlockDataFromList(const CABlockDataID_t *blockID)
     return CA_STATUS_OK;
 }
 
-bool CAIsBlockDataInList(const CABlockDataID_t *blockID)
-{
-    OIC_LOG(DEBUG, TAG, "IN-IsBlockDataInList");
-    VERIFY_NON_NULL_RET(blockID, TAG, "blockID", false);
-
-    ca_mutex_lock(g_context.blockDataListMutex);
-
-    size_t len = u_arraylist_length(g_context.dataList);
-    for (size_t i = 0; i < len; i++)
-    {
-        CABlockData_t *currData = (CABlockData_t *) u_arraylist_get(g_context.dataList, i);
-        if (CABlockidMatches(currData, blockID))
-        {
-            OIC_LOG(DEBUG, TAG, "found block data");
-            ca_mutex_unlock(g_context.blockDataListMutex);
-            return true;
-        }
-    }
-    ca_mutex_unlock(g_context.blockDataListMutex);
-
-    OIC_LOG(DEBUG, TAG, "OUT-IsBlockDataInList");
-    return false;
-}
-
 void CADestroyDataSet(CAData_t* data)
 {
     VERIFY_NON_NULL_VOID(data, TAG, "data");
@@ -2672,11 +2615,9 @@ void CADestroyDataSet(CAData_t* data)
 CABlockDataID_t* CACreateBlockDatablockId(const CAToken_t token, uint8_t tokenLength,
                                           uint16_t portNumber)
 {
-    VERIFY_NON_NULL_RET(token, TAG, "token", NULL);
-
-    char port[PORT_LENGTH] = {0,};
-    port[0] = (char)((portNumber>>8) & 0xFF);
-    port[1] = (char)(portNumber & 0xFF);
+    char port[PORT_LENGTH] = { 0, };
+    port[0] = (char) ((portNumber >> 8) & 0xFF);
+    port[1] = (char) (portNumber & 0xFF);
 
     CABlockDataID_t* blockDataID = (CABlockDataID_t *) OICMalloc(sizeof(CABlockDataID_t));
     if (!blockDataID)
@@ -2693,7 +2634,11 @@ CABlockDataID_t* CACreateBlockDatablockId(const CAToken_t token, uint8_t tokenLe
         return NULL;
     }
 
-    memcpy(blockDataID->id, token, tokenLength);
+    if (token)
+    {
+        memcpy(blockDataID->id, token, tokenLength);
+    }
+
     memcpy(blockDataID->id + tokenLength, port, sizeof(port));
 
     OIC_LOG(DEBUG, TAG, "BlockID is ");