fix buffer overflow in test_security_dir

Signed-off-by: Abby Xu <[email protected]>
ros2 · May 1, 2019 · aa0b4b1 · aa0b4b1
1 parent 64610d2
commit aa0b4b1
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/rcl/test/rcl/test_security_directory.cpp b/rcl/test/rcl/test_security_directory.cpp
@@ -109,7 +109,7 @@ TEST_F(TestGetSecureRoot, successScenarios) {
       TEST_SECURITY_DIRECTORY_RESOURCES_DIR_NAME, allocator);
   std::string putenv_input = ROS_SECURITY_ROOT_DIRECTORY_VAR_NAME "=";
   putenv_input += base_lookup_dir_fqn;
-  memcpy(g_envstring, putenv_input.c_str(), sizeof(g_envstring) - 1);
+  memcpy(g_envstring, putenv_input.c_str(), std::min(putenv_input.length(), sizeof(g_envstring) - 1));
   putenv_wrapper(g_envstring);
   /* --------------------------
    * Namespace  : Root