FreeBSD's login code is used as an example case in the "Principles of Secure
Coding" coursera course. While FreeBSD correctly checks for wrong
preconditions it could also check for the setenv post-conditon.
It is clear that setenv(3) will not fail, however it wouldn't hurt to make sure
the return value matches our expectacions.
Credit for this change is due to Matthew Bishop, PhD