Browse Source

fix(set_env): use `clone_to_cstring` instead of `unsafe_to_cstring`

Rickard Andersson 2 years ago
parent
commit
f048ad13b5
1 changed files with 2 additions and 2 deletions
  1. 2 2
      core/os/os_linux.odin

+ 2 - 2
core/os/os_linux.odin

@@ -894,8 +894,8 @@ get_env :: proc(key: string, allocator := context.allocator) -> (value: string)
 
 
 set_env :: proc(key, value: string) -> Errno {
 set_env :: proc(key, value: string) -> Errno {
 	runtime.DEFAULT_TEMP_ALLOCATOR_TEMP_GUARD()
 	runtime.DEFAULT_TEMP_ALLOCATOR_TEMP_GUARD()
-	key_cstring := strings.unsafe_string_to_cstring(strings.concatenate({key, "\x00"}, context.temp_allocator))
-	value_cstring := strings.unsafe_string_to_cstring(strings.concatenate({value, "\x00"}, context.temp_allocator))
+	key_cstring := strings.clone_to_cstring(key, context.temp_allocator)
+	value_cstring := strings.clone_to_cstring(value, context.temp_allocator)
 	// NOTE(GoNZooo): `setenv` instead of `putenv` because it copies both key and value more commonly
 	// NOTE(GoNZooo): `setenv` instead of `putenv` because it copies both key and value more commonly
 	res := _unix_setenv(key_cstring, value_cstring, 1)
 	res := _unix_setenv(key_cstring, value_cstring, 1)
 	if res < 0 {
 	if res < 0 {