diff options
author | Chet Ramey <chet.ramey@case.edu> | 2011-11-22 20:02:29 -0500 |
---|---|---|
committer | Chet Ramey <chet.ramey@case.edu> | 2011-11-22 20:02:29 -0500 |
commit | 47a7673055e4f82ee6c803df04eb1e2a87e717b7 (patch) | |
tree | f1c478dc01fe88583bab27b633237c2428e9803d /assoc.c | |
parent | 680d15e19f2b8f7b6ee1535f923e7be6a95c4358 (diff) | |
download | android_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.c | 5 |
1 files changed, 5 insertions, 0 deletions
@@ -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); |