aboutsummaryrefslogtreecommitdiffstats
path: root/assoc.c
diff options
context:
space:
mode:
authorChet Ramey <chet.ramey@case.edu>2011-11-22 20:02:29 -0500
committerChet Ramey <chet.ramey@case.edu>2011-11-22 20:02:29 -0500
commit47a7673055e4f82ee6c803df04eb1e2a87e717b7 (patch)
treef1c478dc01fe88583bab27b633237c2428e9803d /assoc.c
parent680d15e19f2b8f7b6ee1535f923e7be6a95c4358 (diff)
downloadandroid_external_bash-47a7673055e4f82ee6c803df04eb1e2a87e717b7.tar.gz
android_external_bash-47a7673055e4f82ee6c803df04eb1e2a87e717b7.tar.bz2
android_external_bash-47a7673055e4f82ee6c803df04eb1e2a87e717b7.zip
Bash-4.2 patch 11
Diffstat (limited to 'assoc.c')
-rw-r--r--assoc.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/assoc.c b/assoc.c
index bbc7b17..4561de4 100644
--- a/assoc.c
+++ b/assoc.c
@@ -77,6 +77,11 @@ assoc_insert (hash, key, value)
b = hash_search (key, hash, HASH_CREATE);
if (b == 0)
return -1;
+ /* If we are overwriting an existing element's value, we're not going to
+ use the key. Nothing in the array assignment code path frees the key
+ string, so we can free it here to avoid a memory leak. */
+ if (b->key != key)
+ free (key);
FREE (b->data);
b->data = value ? savestring (value) : (char *)0;
return (0);