Re: Bash 5.2.0: Memory leak with $(

2024-01-19 Thread Chet Ramey
On 1/15/24 8:36 PM, pou...@tutanota.com wrote: Would you please consider releasing these as an official patch? I have the patch for the original report queued up for the next batch of patches. -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars longa, vita

Re: Bash 5.2.0: Memory leak with $(

2024-01-15 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
> On Mon, Jan 8, 2024, 12:26 wrote: > >> Do any of the other six patches in that report also apply to Bash 5.2? >> > > Yes, all but the one for the `kv' builtin which did not exist yet. See > attached. > >> >> Would you please consider releasing these as an official patch?

Re: Bash 5.2.0: Memory leak with $(

2024-01-11 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
Jan 10, 2024, 15:58 by grishale...@gmail.com: > On Mon, Jan 8, 2024, 12:26 <> pou...@tutanota.com> > wrote: > >> Do any of the other six patches in that report also apply to Bash 5.2? >> > > Yes, all but the one for the `kv' builtin which did not exist yet. See > attached. > >> >> Nice! Will th

Re: Bash 5.2.0: Memory leak with $(

2024-01-10 Thread Grisha Levit
On Mon, Jan 8, 2024, 12:26 wrote: > Do any of the other six patches in that report also apply to Bash 5.2? > Yes, all but the one for the `kv' builtin which did not exist yet. See attached. > From 711ab85262884f2b91f09eceb9aefd0e2426ce67 Mon Sep 17 00:00:00 2001 From: Grisha Levit Date: Sat, 3

Re: Bash 5.2.0: Memory leak with $(

2024-01-08 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
Jan 8, 2024, 09:23 by chet.ra...@case.edu: > > *** ../bash-5.2-patched/builtins/evalstring.c Tue Dec 13 12:53:21 2022 > --- builtins/evalstring.c Tue Nov 28 17:25:39 2023 > *** > *** 763,766 > --- 773,779 > if (fnp) > *fnp = fn; > + else > + free (fn); > + > retu

Re: Bash 5.2.0: Memory leak with $(

2024-01-08 Thread Chet Ramey
On 1/7/24 1:17 AM, pourko--- via Bug reports for the GNU Bourne Again SHell wrote:  [2]:  > https://git.savannah.gnu.org/cgit/bash.git/diff/builtins/evalstring.c?h=devel&id=81f7b44564cd1510788035cea7c59631865a7db2&dt=1#n766 Could we maybe get a patch for the 5.2 series? *** ../bash-5.2-p

Re: Bash 5.2.0: Memory leak with $(

2024-01-08 Thread Chet Ramey
On 1/7/24 12:59 AM, Grisha Levit wrote: On Sun, Jan 7, 2024, 00:26 pourko--- via Bug reports for the GNU Bourne Again SHell wrote: For demonstration, put a $( The bug is not present in bashes before 5.2.0. I believe this is fixed in (yet unreleased) Bash 5.3. See report [1] applied in [2]

Re: Bash 5.2.0: Memory leak with $(

2024-01-07 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
Jan 7, 2024, 00:00 by grishale...@gmail.com: > I believe this is fixed in (yet unreleased) Bash 5.3. See report [1] > applied in [2]. > > [1]: https://lists.gnu.org/archive/html/bug-bash/2023-06/msg00045.html > [2]: > https://git.savannah.gnu.org/cgit/bash.git/diff/builtins/evalstring.c?h=devel

Re: Bash 5.2.0: Memory leak with $(

2024-01-06 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
Jan 6, 2024, 23:59 by grishale...@gmail.com: > > > On Sun, Jan 7, 2024, 00:26 pourko--- via Bug reports for the GNU Bourne Again > SHell <> bug-bash@gnu.org> > wrote: > >> For demonstration, put a $(> becomes very noticeable: >> > > >> The bug is not present in bashes before 5.2.0. >> > > I belie

Re: Bash 5.2.0: Memory leak with $(

2024-01-06 Thread Grisha Levit
On Sun, Jan 7, 2024, 00:26 pourko--- via Bug reports for the GNU Bourne Again SHell wrote: > For demonstration, put a $( becomes very noticeable: > The bug is not present in bashes before 5.2.0. > I believe this is fixed in (yet unreleased) Bash 5.3. See report [1] applied in [2]. [1]: https

Bash 5.2.0: Memory leak with $(

2024-01-06 Thread pourko--- via Bug reports for the GNU Bourne Again SHell
For demonstration, put a $(/tmp/file while :;do   var=$(