svn commit: r185309 - head/sys/geom/vinum

Rink Springer rink at FreeBSD.org
Tue Nov 25 11:20:22 PST 2008


On Tue, Nov 25, 2008 at 07:13:58PM +0000, Ulf Lilleengen wrote:
> Author: lulf
> Date: Tue Nov 25 19:13:58 2008
> New Revision: 185309
> URL: http://svn.freebsd.org/changeset/base/185309
> 
> Log:
>   - Fix a potential NULL pointer reference. Note that this cannot happen in
>     practice, but it is a good programming practice nontheless and it allows the
>     kernel to not depend on userland correctness.

If it can't happen in practice, why not just ASSERT on them? This would
be useful for debugging, and it basically tells whoever reads the code
about the scenario's you are dealing with...

-- 
Rink P.W. Springer                                - http://rink.nu
"Anyway boys, this is America. Just because you get more votes doesn't
 mean you win." - Fox Mulder


More information about the svn-src-all mailing list