Date: Tue, 14 Jun 2005 19:24:24 -0700 (PDT) From: Doug White <dwhite@gumbysoft.com> To: Ed Maste <emaste@phaedrus.sandvine.ca> Cc: freebsd-current@freebsd.org Subject: Re: savecore(8) increments /var/crash/bounds on each boot Message-ID: <20050614191338.N24745@carver.gumbysoft.com> In-Reply-To: <20050613205052.GA91486@sandvine.com> References: <20050613192308.GA87640@sandvine.com> <20050613130317.G2682@carver.gumbysoft.com> <20050613205052.GA91486@sandvine.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, 13 Jun 2005, Ed Maste wrote: > On Mon, Jun 13, 2005 at 01:03:50PM -0700, Doug White wrote: > > > On Mon, 13 Jun 2005, Ed Maste wrote: > > > > > I notice that as of sbin/savecore/savecore.c 1.72 and in 5.4-RELEASE > > > savecore increments the number in /var/crash/bounds on each boot, > > > regardless of whether it rebooted due to panic or was a clean shutdown. > > > > > > Is this the desired behaviour or an unintentional side effect? > > > We've been monitoring the value in bounds to detect panics, which > > > of course doesn't work anymore. > > > > Its certainly not the historical behavior :-) > > > > Think you could submit a patch with the fix? > > The attached patch does the trick. In -vv mode the bounds used to > be included with the first/last dump header output -- I just replaced > it with -1. I don't see how this works. It should generate a bunch of use-uninitialized warning since bounds is used in a sprintf at line 376, in DoFile(): 376: sprintf(buf, "info.%d", bounds); plus additional sprintf's down the file, since the value of bounds is only set by that getbounds() call. The correct fix is to put a else goto done; just above the "newfile" label so it doesn't unconditionally rewrite the bounds file on every call. -- 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?20050614191338.N24745>