Re: CVS commit: src

[Taylor R Campbell <campbell+netbsd-source-changes-d%mumble.net@localhost>]

   From: "Maxime Villard" <maxv%netbsd.org@localhost>
   Date: Wed, 16 Apr 2014 18:55:20 +0000

   An (un)privileged user can easily make the kernel dereference a NULL
   pointer.

   The kernel allows 'data' to be NULL; it's the fs's responsibility to
   ensure that it isn't NULL (if the fs actually needs data).

In most cases of the changes you made, there is already a test for the
length of the data buffer.  Is this not guaranteed to be zero if data
is null?  It seems to me that the length test ought to suffice, and if
anything the null pointer test should be an assertion, not a check.