Date: Wed, 1 Jun 2016 09:57:41 -0700 (PDT) From: Don Lewis <truckman@FreeBSD.org> To: src-committers@freebsd.org Cc: svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r301139 - head/bin/sh Message-ID: <201606011657.u51Gvf1J035954@gw.catspoiler.org> In-Reply-To: <201606011656.u51GuTlZ040844@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 1 Jun, To: src-committers@freebsd.org wrote: > Author: truckman > Date: Wed Jun 1 16:56:29 2016 > New Revision: 301139 > URL: https://svnweb.freebsd.org/changeset/base/301139 > > Log: > The (i < PROMPTLEN - 1) test added by r300442 in the code for the default > case of \c in the prompt format string is a no-op. We already passed > this test at the top of the loop, and i has not yet been incremented in > this path. Change this test to (i < PROMPTLEN - 2). > > Reported by: Coverity > CID: 1008328 > Reviewed by: cem > MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D6552
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201606011657.u51Gvf1J035954>