5bedeb2fb3
Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
47 lines
1.4 KiB
Diff
47 lines
1.4 KiB
Diff
BASH PATCH REPORT
|
|
=================
|
|
|
|
Bash-Release: 4.2
|
|
Patch-ID: bash42-011
|
|
|
|
Bug-Reported-by: "David Parks" <davidparks21@yahoo.com>
|
|
Bug-Reference-ID: <014101cc82c6$46ac1540$d4043fc0$@com>
|
|
Bug-Reference-URL: http://lists.gnu.org/archive/html/bug-bash/2011-10/msg00031.html
|
|
|
|
Bug-Description:
|
|
|
|
Overwriting a value in an associative array causes the memory allocated to
|
|
store the key on the second and subsequent assignments to leak.
|
|
|
|
Patch (apply with `patch -p0'):
|
|
|
|
*** ../bash-4.2-patched/assoc.c 2009-08-05 20:19:40.000000000 -0400
|
|
--- ./assoc.c 2011-10-04 20:23:07.000000000 -0400
|
|
***************
|
|
*** 78,81 ****
|
|
--- 78,86 ----
|
|
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;
|
|
*** ../bash-4.2-patched/patchlevel.h Sat Jun 12 20:14:48 2010
|
|
--- ./patchlevel.h Thu Feb 24 21:41:34 2011
|
|
***************
|
|
*** 26,30 ****
|
|
looks for to find the patch level (for the sccs version string). */
|
|
|
|
! #define PATCHLEVEL 10
|
|
|
|
#endif /* _PATCHLEVEL_H_ */
|
|
--- 26,30 ----
|
|
looks for to find the patch level (for the sccs version string). */
|
|
|
|
! #define PATCHLEVEL 11
|
|
|
|
#endif /* _PATCHLEVEL_H_ */
|