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