tree c1da6cd6fe03bfadb3276bd30423c7d4b105ef41
parent 50431d94e732ba71b66a83c5435890728e313095
author J. Bruce Fields <bfields@citi.umich.edu> 1201041642 -0500
committer J. Bruce Fields <bfields@citi.umich.edu> 1201902135 -0500

nfsd: more careful input validation in nfsctl write methods

Neil Brown points out that we're checking buf[size-1] in a couple places
without first checking whether size is zero.

Actually, given the implementation of simple_transaction_get(), buf[-1]
is zero, so in both of these cases the subsequent check of the value of
buf[size-1] will catch this case.

But it seems fragile to depend on that, so add explicit checks for this
case.

Signed-off-by: J. Bruce Fields <bfields@citi.umich.edu>
Acked-by: NeilBrown <neilb@suse.de>