From owner-cvs-all  Thu Aug 19 10:48:45 1999
Delivered-To: cvs-all@freebsd.org
Received: from pcnet1.pcnet.com (pcnet1.pcnet.com [204.213.232.3])
	by hub.freebsd.org (Postfix) with ESMTP
	id CA9D01514C; Thu, 19 Aug 1999 10:48:33 -0700 (PDT)
	(envelope-from eischen@vigrid.com)
Received: (from eischen@localhost)
	by pcnet1.pcnet.com (8.8.7/PCNet) id NAA21199;
	Thu, 19 Aug 1999 13:47:15 -0400 (EDT)
Date: Thu, 19 Aug 1999 13:47:15 -0400 (EDT)
From: Daniel Eischen <eischen@vigrid.com>
Message-Id: <199908191747.NAA21199@pcnet1.pcnet.com>
To: alfred@FreeBSD.org, cvs-all@FreeBSD.org,
	cvs-committers@FreeBSD.org
Subject: Re: cvs commit: src/lib/libc_r/uthread uthread_kern.c
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk

> alfred      1999/08/19 09:49:54 PDT
> 
>   Modified files:
>     lib/libc_r/uthread   uthread_kern.c 
>   Log:
>   handle under/overflow of time values in a more robust manner,
>   there may be an overflow that need to be adjusted more than once.

I think this is a coverup for the real problem.  Timevals and Timespecs
should be validated in pthread_cond_timedwait(), select(), nanosleep(),
etc, and errno set appropriately (EINVAL returned for pthread_cond_timedwait).
Correcting for the subtraction of two _valid_ tv_nsec fields should only
need to be done once.

Dan Eischen
eischen@vigrid.com


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message