fix error message in logger.

If path to log file was too long, we was printing uninitialized
variable. This patch fixes this issue.

Author: lplewa

src/utils/utils_log.c

@@ -156,20 +156,23 @@ void util_log_init(void) {
             len = strlen(arg);
         }
 
-        if (len <= MAX_FILE_PATH) {
-            memcpy(file, arg, len);
-            file[len] = '\0';
-            loggerConfig.output = fopen(file, "w");
+        if (len > MAX_FILE_PATH) {
+            loggerConfig.output = stderr;
+            LOG_ERR("Cannot open output file - path too long");
+            loggerConfig.output = NULL;
+            return;
         }
+
+        memcpy(file, arg, len);
+        file[len] = '\0';
+        loggerConfig.output = fopen(file, "w");
         if (!loggerConfig.output) {
             loggerConfig.output = stderr;
             LOG_ERR("Cannot open output file %s - logging disabled", file);
             loggerConfig.output = NULL;
             return;
         }
-    }
-
-    else {
+    } else {
         loggerConfig.output = stderr;
         LOG_ERR("Logging output not set - logging disabled");
         loggerConfig.output = NULL;

test/utils/utils_log.cpp

@@ -90,6 +90,7 @@ void helper_checkConfig(util_log_config_t *expected, util_log_config_t *is) {
 }
 
 TEST_F(test, parseEnv_errors) {
+    expected_message = "";
     loggerConfig = {0, 0, LOG_ERROR, LOG_ERROR, NULL};
     expect_fput_count = 0;
     expected_stream = stderr;
@@ -105,6 +106,9 @@ TEST_F(test, parseEnv_errors) {
     helper_checkConfig(&b, &loggerConfig);
     helper_log_init("_level:debug");
     helper_checkConfig(&b, &loggerConfig);
+    expect_fput_count = 1;
+    expected_message = "[ERROR UMF] Cannot open output file - path too long\n";
+    helper_log_init(("output:file," + std::string(300, 'x')).c_str());
 }
 
 TEST_F(test, parseEnv) {
@@ -148,17 +152,20 @@ TEST_F(test, parseEnv) {
                                              output.first + ";" +
                                              timestamp.first + ";" + pid.first;
                         b = loggerConfig = {0, 0, LOG_ERROR, LOG_ERROR, NULL};
+                        expect_fput_count = 0;
+                        expect_fopen_count = 0;
+                        expected_stream = stderr;
+                        expected_message = "";
+                        expected_filename = "";
+                        auto n = output.first.find(',');
+                        if (n != std::string::npos) {
+                            expected_filename = output.first.substr(n + 1);
+                        }
+
                         if (output.second != NULL) {
                             b.output = output.second;
                             if (output.second == MOCK_FILE_PTR) {
                                 expect_fopen_count = 1;
-                            } else {
-                                expect_fopen_count = 0;
-                            }
-                            auto n = output.first.find(',');
-                            if (n != std::string::npos) {
-                                expected_filename =
-                                    output.first.substr(n + 1).c_str();
                             }
                             expected_stream = output.second;
                             b.timestamp = timestamp.second;
@@ -168,13 +175,14 @@ TEST_F(test, parseEnv) {
                             b.level = (util_log_level_t)logLevel.second;
                             if (logLevel.second <= LOG_INFO) {
                                 expect_fput_count = 1;
-                            } else {
-                                expect_fput_count = 0;
                             }
                         } else {
                             expect_fput_count = 1;
-                            expect_fopen_count = 0;
-                            expected_stream = stderr;
+                            if (expected_filename.size() > MAX_FILE_PATH) {
+                                expected_message =
+                                    "[ERROR UMF] Cannot open output file - "
+                                    "path too long\n";
+                            }
                         }
                         helper_log_init(envVar.c_str());
                         helper_checkConfig(&b, &loggerConfig);