svcrpc: Revert "sunrpc/cache.h: replace simple_strtoul"
authorJ. Bruce Fields <bfields@redhat.com>
Wed, 14 Nov 2012 15:48:05 +0000 (10:48 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Thu, 15 Nov 2012 12:40:32 +0000 (07:40 -0500)
commit621eb19ce1ec216e03ad354cb0c4061736b2a436
tree57767888ba129e174a61398451a8ad405fd2028a
parent2b4cf668a7b8f84182a35f07152d8b6f012629d2
svcrpc: Revert "sunrpc/cache.h: replace simple_strtoul"

Commit bbf43dc888833ac0539e437dbaeb28bfd4fbab9f "sunrpc/cache.h: replace
simple_strtoul" introduced new range-checking which could cause get_int
to fail on unsigned integers too large to be represented as an int.

We could parse them as unsigned instead--but it turns out svcgssd is
actually passing down "-1" in some cases.  Which is perhaps stupid, but
there's nothing we can do about it now.

So just revert back to the previous "sloppy" behavior that accepts
either representation.

Cc: stable@vger.kernel.org
Reported-by: Sven Geggus <lists@fuchsschwanzdomain.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
include/linux/sunrpc/cache.h