Add missing break statements, per Coverity.
More missing break statements found by Coverity.
Adding missing break statement, and make the code clearer.
Remove some code which would never be reached, pointed out by Coverity.
Remove redundant check, pointed out by Coverity.
Coverity's analysis correctly identified some dead/unreachable code.  Clean it
up here.
Handle the EISDIR case first; the following code would handle it automatically,
but we want to handle EISDIR as a special case.
A few more missing break statements found by Coverity.
Remove code which is logically dead/unreachable, per Coverity.
Remove unreachable code, found by Coverity.
Handle the case where opening a syslog socket might fail.
Watch for short reads when reading in a controls request.
Avoid possible (but unlikely) null pointer dereference.
Paranoidly check for NULL from strchr(3); this case should've been caught
in the directive handler.
Avoid possible null pointer dereference, per Coverity.
Avoid possible null pointer dereferences (unlikely) when handling badly
configured SFTPAuthorized{Host,User}Keys parameters.
Coverity found another place where possibly badly formatted Trace settings
could trigger a null pointer dereference.
Handle (rare) cases where lseek(2) might return an error.
More paranoid error handling, prompted by Coverity.
Handle possible lseek(2) error when scrubbing scoreboard via ftpscrub.