too late null checks

Andre Oppermann andre at freebsd.org
Thu Aug 19 13:18:18 PDT 2004


Ted Unangst wrote:
> Bugs found with Coverity's automated analysis.  In each case, either the 
> NULL check is unnecessary, or it's too late because the variable was 
> already deref'd.
...
> netinet/tcp_subr.c:tcp_mtudisc
> tp has already been dereferenced to obtain isipv6 result

Thanks, fixed.

Did you run your entire tool on the FreeBSD tree or is this subset of the
available tests and checks?

-- 
Andre


More information about the freebsd-hackers mailing list