Re: [PATCH] bind_assoc_variable: free key if cannot assign

2024-05-31 Thread Chet Ramey
On 5/30/24 5:30 PM, Grisha Levit wrote: Avoid leaking expansion of `x' in `declare -Ar A; A[x]=' Thanks for the report. I think a more appropriate place to fix this is assign_array_element_internal(), since that's where the key is allocated. Chet -- ``The lyf so short, the craft so long to le

[PATCH] bind_assoc_variable: free key if cannot assign

2024-05-30 Thread Grisha Levit
Avoid leaking expansion of `x' in `declare -Ar A; A[x]=' --- arrayfunc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arrayfunc.c b/arrayfunc.c index e85ba6e5..aaa1963c 100644 --- a/arrayfunc.c +++ b/arrayfunc.c @@ -302,6 +302,7 @@ bind_assoc_variable (SHELL_VAR *entry, const char *name, c