Merge pull request #425 from ldorau/Do_not_use_c_str_beyond_its_scope

ldorau · web-flow · commit 5f4652f9a2f8 · 2024-04-10T10:48:15.000+02:00
Do not use c_str() beyond its scope
diff --git a/test/utils/utils_log.cpp b/test/utils/utils_log.cpp
@@ -476,6 +476,7 @@ TEST_F(test, plog_long_error) {
 #endif
     strerror_ret_static = 0;
     helper_test_plog(LOG_DEBUG, "%s", "example log");
+    strerr = NULL; // do not use tmp.c_str() beyond its scope
 }
 
 TEST_F(test, log_pmacros) {

Original file line number	Diff line number	Diff line change
`@@ -476,6 +476,7 @@ TEST_F(test, plog_long_error) {`
`476`	`476`	`#endif`
`477`	`477`	`strerror_ret_static = 0;`
`478`	`478`	`helper_test_plog(LOG_DEBUG, "%s", "example log");`
	`479`	`+ strerr = NULL; // do not use tmp.c_str() beyond its scope`
`479`	`480`	`}`
`480`	`481`
`481`	`482`	`TEST_F(test, log_pmacros) {`