]> git.karo-electronics.de Git - karo-tx-linux.git/commit
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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 17 Jan 2013 16:46:08 +0000 (08:46 -0800)
commit243bddd7b189dbbb4561e33e1f34a2d361da2f9b
tree14c182ce6a8e5de9db7e9a7ef248ddbd15df9bf7
parentfac767ddb0a4eb6b6d8d416044263305db4a05e0
svcrpc: Revert "sunrpc/cache.h: replace simple_strtoul"

commit 621eb19ce1ec216e03ad354cb0c4061736b2a436 upstream.

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.

Reported-by: Sven Geggus <lists@fuchsschwanzdomain.de>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
include/linux/sunrpc/cache.h