From: Peter Stephenson <p.stephenson@samsung.com>
To: <zsh-workers@zsh.org>
Subject: Re: Valgrind tests (was: Re: Zsh: [7] + 23074 suspended (tty output))
Date: Mon, 15 Jul 2019 10:29:17 +0100 [thread overview]
Message-ID: <1563182957.4386.6.camel@samsung.com> (raw)
In-Reply-To: <CAKc7PVDFnYkS4sX+==ivpyegO4t5TUh_752u6-x-bPV+bg1SPA@mail.gmail.com>
On Thu, 2019-07-04 at 01:29 +0200, Sebastian Gniazdowski wrote:
> The tests apparently reveal one memory leak currently:
>
> ==52847== 79 (24 direct, 55 indirect) bytes in 1 blocks are definitely
> lost in loss record 349 of 550
> ==52847== at 0x10017B545: malloc (vg_replace_malloc.c:302)
> ==52847== by 0x10004899E: zalloc (mem.c:966)
> ==52847== by 0x10004316D: znewlinklist (linklist.c:120)
> ==52847== by 0x10003989D: addfilelist (jobs.c:1297)
> ==52847== by 0x100017AC0: getoutputfile (exec.c:4796)
> ==52847== by 0x100078474: stringsubst (subst.c:254)
> ==52847== by 0x100077E46: prefork (subst.c:142)
> ==52847== by 0x10001A24F: execfuncdef (exec.c:2567)
> ==52847== by 0x10001DC23: execcmd_exec (exec.c:3896)
> ==52847== by 0x10001AAF6: execpline2 (exec.c:1927)
> ==52847== by 0x100015938: execpline (exec.c:1658)
> ==52847== by 0x1000150E2: execlist (exec.c:1413)
By the way, I think these are benign --- addfilelist() adds a file to be
tidied up when a job exits, but these appear to happening in subshells.
This means there's no job clear-up --- but the shell simply exits. So
the memory isn't recovered in the subshell but that doesn't cause a
long-term build-up.
I haven't followed every single one of this type, though.
pws
next prev parent reply other threads:[~2019-07-15 9:30 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-15 17:22 Zsh: [7] + 23074 suspended (tty output) TS
2018-09-15 18:10 ` Bart Schaefer
2018-09-15 18:51 ` TS
2018-09-15 19:06 ` TS
2018-09-16 8:40 ` Vincent Lefevre
2018-09-16 15:40 ` TS
2018-09-16 17:51 ` Peter Stephenson
2018-09-17 20:13 ` TS
2018-09-18 17:59 ` Peter Stephenson
2018-09-18 1:16 ` Vincent Lefevre
2018-09-18 17:58 ` Peter Stephenson
2018-09-20 12:01 ` Vincent Lefevre
2018-09-20 16:11 ` Daniel Shahaf
2018-09-18 19:30 ` TS
2018-09-19 3:38 ` TS
2018-09-19 14:23 ` Daniel Tameling
2018-09-19 16:12 ` Peter Stephenson
2018-09-19 18:15 ` TS
2018-09-17 14:01 ` Sebastian Gniazdowski
2018-09-17 14:49 ` Valgrind tests (was: Re: Zsh: [7] + 23074 suspended (tty output)) Daniel Shahaf
2018-09-18 5:21 ` Sebastian Gniazdowski
2018-09-18 15:55 ` Daniel Shahaf
2018-09-19 5:04 ` Sebastian Gniazdowski
2019-07-03 23:29 ` Sebastian Gniazdowski
2019-07-03 23:31 ` Sebastian Gniazdowski
2019-07-03 23:45 ` Sebastian Gniazdowski
2019-07-13 14:31 ` Sebastian Gniazdowski
2019-07-14 6:39 ` Daniel Shahaf
2019-07-14 22:17 ` Sebastian Gniazdowski
2019-07-15 8:54 ` Peter Stephenson
2019-07-15 9:29 ` Peter Stephenson [this message]
2019-07-15 18:45 ` Sebastian Gniazdowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1563182957.4386.6.camel@samsung.com \
--to=p.stephenson@samsung.com \
--cc=zsh-workers@zsh.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).