diff options
author | Chuck Lever <chuck.lever@oracle.com> | 2015-06-11 13:47:10 -0400 |
---|---|---|
committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2015-06-11 14:01:06 -0400 |
commit | 5fd23f7e1d74c0fd100ffb0b04dc85727760d9ea (patch) | |
tree | 8a594c7cb198270c731413c6594eb82fa826b0e1 /net/sunrpc/debugfs.c | |
parent | 4ed0d83d057c16efeca4210ba8fb2b5029333fb3 (diff) |
SUNRPC: Address kbuild warning in net/sunrpc/debugfs.c
Cross-compile test on ARCH=mn10300:
In file included from include/linux/list.h:8:0,
from include/linux/wait.h:6,
from include/linux/fs.h:6,
from include/linux/debugfs.h:18,
from net/sunrpc/debugfs.c:7:
net/sunrpc/debugfs.c: In function 'fault_disconnect_write':
include/linux/kernel.h:723:17: warning: comparison of distinct pointer
types lacks a cast
(void) (&_min1 == &_min2); \
^
>> net/sunrpc/debugfs.c:307:8: note: in expansion of macro 'min'
len = min(len, sizeof(buffer) - 1);
Fixes: ('SUNRPC: Transport fault injection')
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'net/sunrpc/debugfs.c')
-rw-r--r-- | net/sunrpc/debugfs.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/sunrpc/debugfs.c b/net/sunrpc/debugfs.c index 7cc1b8a6ef6d..e7b4d93566df 100644 --- a/net/sunrpc/debugfs.c +++ b/net/sunrpc/debugfs.c | |||
@@ -304,7 +304,8 @@ fault_disconnect_write(struct file *filp, const char __user *user_buf, | |||
304 | { | 304 | { |
305 | char buffer[16]; | 305 | char buffer[16]; |
306 | 306 | ||
307 | len = min(len, sizeof(buffer) - 1); | 307 | if (len >= sizeof(buffer)) |
308 | len = sizeof(buffer) - 1; | ||
308 | if (copy_from_user(buffer, user_buf, len)) | 309 | if (copy_from_user(buffer, user_buf, len)) |
309 | return -EFAULT; | 310 | return -EFAULT; |
310 | buffer[len] = '\0'; | 311 | buffer[len] = '\0'; |