diff options
| author | Alexander Viro <viro@parcelfarce.linux.theplanet.co.uk> | 2004-07-12 21:30:17 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@ppc970.osdl.org> | 2004-07-12 21:30:17 -0700 |
| commit | 8e231e7263a0486eef92b91cc120dcadeaf63232 (patch) | |
| tree | 1a4dc3f68960d336925ec9e916d3884143ee7c30 | |
| parent | e4d478f77230c900ca8c0589c34deeea944662ef (diff) | |
[PATCH] compat_fillonedir() warning fix
access_ok() expects a pointer, not unsigned long. It's not a
problem on platforms that have this guy done as a macro (or ones that
do not use fs/compat.c at all), but that's still wrong and on some
platforms that care we actually have access_ok() as inlined function.
Bogus cast removed.
| -rw-r--r-- | fs/compat.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/compat.c b/fs/compat.c index 3e8c96c9b870..07f06a3076d7 100644 --- a/fs/compat.c +++ b/fs/compat.c @@ -824,7 +824,7 @@ static int compat_fillonedir(void *__buf, const char *name, int namlen, return -EINVAL; buf->result++; dirent = buf->dirent; - if (!access_ok(VERIFY_WRITE, (unsigned long)dirent, + if (!access_ok(VERIFY_WRITE, dirent, (unsigned long)(dirent->d_name + namlen + 1) - (unsigned long)dirent)) goto efault; |