[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
CVS: cvs.openbsd.org: src
- To: source-changes_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org
- Subject: CVS: cvs.openbsd.org: src
- From: David Gwynne <dlg_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org>
- Date: Thu, 20 Apr 2006 19:35:27 -0600 (MDT)
Module name: src
Changes by: dlg_(_at_)_cvs_(_dot_)_openbsd_(_dot_)_org 2006/04/20 19:35:27
sys/kern : kern_timeout.c
back out my last commit.
art pointed out that timeout_set is the initializer of timeout structs.
this means that the ONQUEUE flag could be set when timeout_set is given
freshly allocated memory. my commit suddenly introduced the requirement
that you bzero a timeout before initialising it. without the bzero we
could generate false positives about the timeout being already queued.
art did produce a diff that would walk the queues when the flag was set
to see if it really was in the lists, but deraadt considers this too much
of a hit.
Visit your host, monkey.org