Re: unset associative array may cause segfault

2009-08-25 Thread Chet Ramey
Chris F.A. Johnson wrote: > On Tue, 25 Aug 2009, Chet Ramey wrote: > >> Roman Rakus wrote: >>> reproducer: >>> declare -A aaa >>> aaa='somestring' >>> unset aaa >>> >>> This commands cause segmentation fault, free unallocated memory. >>> Chet do you already fix it? Here is my fast patch. >> I alre

Re: unset associative array may cause segfault

2009-08-25 Thread Chris F.A. Johnson
On Tue, 25 Aug 2009, Chet Ramey wrote: > Roman Rakus wrote: > > reproducer: > > declare -A aaa > > aaa='somestring' > > unset aaa > > > > This commands cause segmentation fault, free unallocated memory. > > Chet do you already fix it? Here is my fast patch. > > I already fixed it, thanks. D

Re: unset associative array may cause segfault

2009-08-25 Thread Chet Ramey
> >> reproducer: > >> declare -A aaa > >> aaa='somestring' > >> unset aaa > >> > >> This commands cause segmentation fault, free unallocated memory. > >> Chet do you already fix it? Here is my fast patch. > >> > > > > I already fixed it, thanks. Your patch introduces large memory leaks; > >

Re: unset associative array may cause segfault

2009-08-25 Thread Roman Rakus
On 08/25/2009 04:47 PM, Chet Ramey wrote: Roman Rakus wrote: reproducer: declare -A aaa aaa='somestring' unset aaa This commands cause segmentation fault, free unallocated memory. Chet do you already fix it? Here is my fast patch. I already fixed it, thanks. Your patch introduces l

Re: unset associative array may cause segfault

2009-08-25 Thread Chet Ramey
Roman Rakus wrote: > reproducer: > declare -A aaa > aaa='somestring' > unset aaa > > This commands cause segmentation fault, free unallocated memory. > Chet do you already fix it? Here is my fast patch. I already fixed it, thanks. Your patch introduces large memory leaks; the right place to fix

unset associative array may cause segfault

2009-08-25 Thread Roman Rakus
reproducer: declare -A aaa aaa='somestring' unset aaa This commands cause segmentation fault, free unallocated memory. Chet do you already fix it? Here is my fast patch. RR diff -up bash-4.0/hashlib.c.alloc bash-4.0/hashlib.c --- bash-4.0/hashlib.c.alloc2009-08-25 16:25:21.0 +0200 +++