Date: Fri, 15 Aug 2003 19:13:57 -0700 (PDT) From: Doug White <dwhite@gumbysoft.com> To: Eugene Grosbein <eugen@grosbein.pp.ru> Cc: bug-followup@freebsd.org Subject: Re: bin/55346: /bin/sh eats memory and CPU infinitely Message-ID: <20030815191010.S16351@carver.gumbysoft.com> In-Reply-To: <20030815211321.A11482@grosbein.pp.ru> References: <20030815211321.A11482@grosbein.pp.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, 15 Aug 2003, Eugene Grosbein wrote: > I think I've found a memory leak in /bin/sh. > There is a case when dowait() and does frees resources of > completed job correctly. Here is a patch: Your idea has some merit but your patch does not conform to normal style. > -#if JOBS > if (done) > - deljob(jp); > -#endif > + freejob(jp); You should not remove the ifdef and change the indentation of deljob. Just s/del/free and be done with it :-) I highly doubt anyone needs to disable job control in sh, but in the off chance they do, you might as well keep the ifdef around. Note that freejob calls deljob. -- Doug White | FreeBSD: The Power to Serve dwhite@gumbysoft.com | www.FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030815191010.S16351>