Certificate Logging Improvement 73/24473/2
authorOleksii Beketov <ol.beketov@samsung.com>
Mon, 26 Mar 2018 12:29:35 +0000 (15:29 +0300)
committerOleksii Beketov <ol.beketov@samsung.com>
Mon, 26 Mar 2018 12:29:35 +0000 (15:29 +0300)
Allow to log the cert entirely

Change-Id: I69cdef6f49323dda8c0d24df8b8f702f86847664
Signed-off-by: Oleksii Beketov <ol.beketov@samsung.com>
resource/csdk/logger/include/experimental/logger.h
resource/csdk/security/provisioning/src/ownershiptransfermanager.c
resource/csdk/security/src/credresource.c

index cc1844d..f256403 100644 (file)
@@ -44,7 +44,11 @@ extern "C"
 #define PCF(str) str
 
 // Max buffer size used in variable argument log function
+#ifndef NDEBUG
+#define MAX_LOG_V_BUFFER_SIZE (2048)
+#else
 #define MAX_LOG_V_BUFFER_SIZE (256)
+#endif
 
 // Setting this flag for a log level means that the corresponding log message
 // contains private data. This kind of message is logged only when a call to
index c839967..9f73007 100644 (file)
@@ -834,7 +834,7 @@ static OCStackResult SaveOwnerPSK(OCProvisionDev_t *selectedDeviceInfo)
 
     if (CA_STATUS_OK == pskRet)
     {
-        OIC_LOG(DEBUG, TAG,"Owner PSK dump:\n");
+        OIC_LOG(DEBUG, TAG,"Owner PSK dump:");
         OIC_LOG_BUFFER(DEBUG, TAG,ownerPSK, OWNER_PSK_LENGTH_128);
         //Generating new credential for provisioning tool
         OicSecCred_t *cred = GenerateCredential(&selectedDeviceInfo->doxm->deviceID,
index e8b2356..166dcc8 100644 (file)
@@ -3325,13 +3325,24 @@ void LogCert(uint8_t *data, size_t len, OicEncodingType_t encoding, const char*
                 mbedRet = mbedtls_x509_crt_info(infoBuf, CERT_INFO_BUF_LEN, tag, &mbedCert);
                 if (0 < mbedRet)
                 {
+                    int pos = strlen(infoBuf)-1;
+                    if (infoBuf[pos] == '\n')
+                    {
+                        infoBuf[pos] = '\0';
+                    }
                     OIC_LOG_V(DEBUG, tag, "%s", infoBuf);
                 }
             }
             mbedtls_x509_crt_free(&mbedCert);
 
             // raw pem dump
-            OIC_LOG_V(DEBUG, tag, "%s", pem);
+            int pos = strlen((char *)pem)-1;
+            if (pem[pos] == '\n')
+            {
+                pem[pos] = '\0';
+            }
+            snprintf(infoBuf, CERT_INFO_BUF_LEN, "\n%s", pem);
+            OIC_LOG_V(DEBUG, tag, "%s", infoBuf);
         }
         if ( true == needTofreePem )
         {
@@ -3426,15 +3437,12 @@ void LogCredResource(OicSecCred_t *cred, const char* tag, const char* label)
     OIC_LOG_V(DEBUG, tag, "=== %s ========================", (NULL != label) ? label : "cred" );
     VERIFY_NOT_NULL(tag, cred, ERROR);
 
-    OIC_LOG(DEBUG, tag, "\n");
-
     uuid = NULL;
     if (OCConvertUuidToString(gRownerId.id, uuidString))
     {
         uuid = uuidString;
     }
     OIC_LOG_V(DEBUG, tag, "rowner uuid:  %s", (NULL != uuid) ? uuid : "None or Error");
-    OIC_LOG(DEBUG, tag, "\n");
 
     LL_FOREACH(cred, curCred)
     {
@@ -3444,7 +3452,6 @@ void LogCredResource(OicSecCred_t *cred, const char* tag, const char* label)
     }
 
     exit:
-        OIC_LOG(DEBUG, tag, "\n");
         OIC_LOG(DEBUG, tag, "============================================================");
 
     return;