Initialize va_filerev in vattr_null() to avoid leaking stack garbage;
authorguenther <guenther@openbsd.org>
Tue, 13 Oct 2015 09:11:48 +0000 (09:11 +0000)
committerguenther <guenther@openbsd.org>
Tue, 13 Oct 2015 09:11:48 +0000 (09:11 +0000)
commit9d4de16fe023193a84c3ca763e61653a383fd0cc
tree6dc64f6a47daa305bb3da58eb33eabecbfcd06fd
parentde0f9d0602fe6cbe74964a59e031511b188e3f8e
Initialize va_filerev in vattr_null() to avoid leaking stack garbage;
problem pointed out by Martin Natano (natano (at) natano.net)

Also, stop chaining assignments (foo = bar = baz) in vattr_null().
The exact meaning of those depends on the order of the sizes-and-
signednesses of the lvalues, making them fragile: a statement here
mixed *six* types, but managed to get them in a safe order.  Delete
a 20+ year old XXX comment that was almost certainly bemoaning a bug
from when they were in an unsafe order.

ok deraadt@ miod@
sys/kern/vfs_subr.c