Date: Wed, 20 Feb 2008 16:27:07 +0300 From: Anton Yuzhaninov <citrin@citrin.ru> To: freebsd-current@freebsd.org Subject: Re: tcsh in current-8.0 coredump Message-ID: <47BC2AAB.5090605@citrin.ru> In-Reply-To: <7F7C8315-AB96-425E-B942-0DB55BFDBCF3@FreeBSD.org> References: <47BB4E5D.7010505@citrin.ru> <7F7C8315-AB96-425E-B942-0DB55BFDBCF3@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 20.02.2008 4:56, Rui Paulo wrote: > > On Feb 19, 2008, at 9:47 PM, Anton Yuzhaninov wrote: > >> Problem was described here: >> http://docs.freebsd.org/cgi/mid.cgi?131632274.20070319100945 >> http://mx.gw.com/pipermail/tcsh-bugs/2007-March/000481.html >> >> This was fixed for RELENG_7: >> >> http://www.freebsd.org/cgi/cvsweb.cgi/src/contrib/tcsh/sh.lex.c >> Revision 1.1.1.8 (vendor branch): download - view: text, markup, >> annotated - select for diffs >> Tue Apr 3 15:51:53 2007 UTC (10 months, 2 weeks ago) by mp >> Branches: ZOULAS, MAIN >> CVS tags: tcsh_6_15p1, RELENG_7_BP, RELENG_7_0_BP, RELENG_7_0, RELENG_7 >> Diff to: previous 1.1.1.7: preferred, colored >> Changes since revision 1.1.1.7: +2 -1 lines >> >> Import vendor patch to fix postcmd regression in tcsh-6.15.00. >> ------- >> >> But this bug was not fixed in HEAD. >> > > Are you sure? I seem to recall this was fixed even before RELENG_7 was > tagged. > $ cvs up $ cvs diff -r HEAD -r RELENG_7 contrib/tcsh/sh.lex.c Index: contrib/tcsh/sh.lex.c =================================================================== RCS file: /home/ncvs/src/contrib/tcsh/sh.lex.c,v retrieving revision 1.1.1.9 retrieving revision 1.1.1.8 diff -u -r1.1.1.9 -r1.1.1.8 --- contrib/tcsh/sh.lex.c 15 Oct 2007 16:54:07 -0000 1.1.1.9 +++ contrib/tcsh/sh.lex.c 3 Apr 2007 15:51:53 -0000 1.1.1.8 @@ -851,7 +851,8 @@ return (en); } slhs.len = 0; - Strbuf_append(&slhs, lhsb.s); + if (lhsb.s != NULL && lhsb.len != 0) + Strbuf_append(&slhs, lhsb.s); Strbuf_terminate(&slhs); if (exclc) en = dosub(sc, en, global); As you can see from cvs diff, null pointer check present in RELENG_7. but absent in HEAD -- WBR, Anton Yuzhaninov
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?47BC2AAB.5090605>