Path: csiph.com!goblin3!goblin2!goblin1!goblin.stu.neva.ru!usenet.stanford.edu!not-for-mail From: fireshen Newsgroups: gnu.bash.bug Subject: Re: memleak in execute_function? Date: Wed, 31 Jul 2019 17:57:28 -0700 (MST) Lines: 13 Approved: bug-bash@gnu.org Message-ID: References: <1564586812344-0.post@n7.nabble.com> <1564621048237-0.post@n7.nabble.com> NNTP-Posting-Host: lists.gnu.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Trace: usenet.stanford.edu 1564621014 10737 209.51.188.17 (1 Aug 2019 00:56:54 GMT) X-Complaints-To: action@cs.stanford.edu To: Bug-bash@gnu.org Envelope-to: Bug-bash@gnu.org In-Reply-To: X-detected-operating-system: by eggs.gnu.org: Mac OS X [generic] X-Received-From: 162.253.133.57 X-BeenThere: bug-bash@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Bug reports for the GNU Bourne Again SHell List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Mailman-Original-Message-ID: <1564621048237-0.post@n7.nabble.com> X-Mailman-Original-References: <1564586812344-0.post@n7.nabble.com> Xref: csiph.com gnu.bash.bug:15287 Yeah, I agree with your opinion, in the branch "if (subshell =3D=3D 0) ", w= e have the function "maybe_restore_getopt_state" to free gs; however, in the branc= h "else", it seems like forget to free "gs", or we don't have to free it=EF= =BC=9F -- Sent from: http://gnu-bash.2382.n7.nabble.com/