Subject: Re: CVS commit: src/sys/arch/sparc64/sparc64
To: Christos Zoulas <christos@netbsd.org>
From: Martin Husemann <martin@NetBSD.org>
List: source-changes
Date: 05/31/2005 19:10:49
On Tue, May 31, 2005 at 12:51:57AM +0000, Christos Zoulas wrote:
> 
> Module Name:	src
> Committed By:	christos
> Date:		Tue May 31 00:51:57 UTC 2005
> 
> Modified Files:
> 	src/sys/arch/sparc64/sparc64: clock.c
> 
> Log Message:
> Deal with the fact that time is volatile and todr_*() are not. This needs
> to be fixed properly.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -r1.71 -r1.72 src/sys/arch/sparc64/sparc64/clock.c

I don't understand this commit message - what does this change fix?
Even after Michaels fixes, there is still at least one potential use
of an unintialized value in line 888

                int deltat;
                time = tv;
                deltat = time.tv_sec - base;

tv might be uninitialized here.

Martin