summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAl Viro <viro@zeniv.linux.org.uk>2013-11-15 21:55:52 -0500
committerAl Viro <viro@zeniv.linux.org.uk>2013-11-15 22:04:09 -0500
commitdb51242d89b3059a46a3cf2f3339f8cd975cb954 (patch)
treed687e082dae5df046ece9cc37e72094ace69f9a1
parent049ffa8ab33a63b3bff672d1a0ee6a35ad253fe8 (diff)
downloadlinux-db51242d89b3059a46a3cf2f3339f8cd975cb954.tar.gz
linux-db51242d89b3059a46a3cf2f3339f8cd975cb954.tar.bz2
linux-db51242d89b3059a46a3cf2f3339f8cd975cb954.zip
dump_align(): fix the dumb braino
Mea culpa - original variant used 64-by-32-bit division, which got caught very late. Getting rid of that wasn't hard, but I'd managed to botch the calling conventions in process ;-/ Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r--fs/coredump.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/coredump.c b/fs/coredump.c
index 62406b6959b6..a2856f7bb613 100644
--- a/fs/coredump.c
+++ b/fs/coredump.c
@@ -733,7 +733,7 @@ int dump_align(struct coredump_params *cprm, int align)
{
unsigned mod = cprm->written & (align - 1);
if (align & (align - 1))
- return -EINVAL;
- return mod ? dump_skip(cprm, align - mod) : 0;
+ return 0;
+ return mod ? dump_skip(cprm, align - mod) : 1;
}
EXPORT_SYMBOL(dump_align);