diff options
author | Nick Piggin <npiggin@suse.de> | 2007-02-14 12:36:32 +0100 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-02-15 09:57:03 -0800 |
commit | 4a76ef036ac415c28d1adbaf2b7a74040b84d4c2 (patch) | |
tree | 3379adb0f6ede8e276e100c6705a6aeb6a5c5feb /mm | |
parent | 30fcffed8149df18592b3e006b829232b7b3844f (diff) | |
download | lwn-4a76ef036ac415c28d1adbaf2b7a74040b84d4c2.tar.gz lwn-4a76ef036ac415c28d1adbaf2b7a74040b84d4c2.zip |
[PATCH] mincore: fill in results properly
Paper bag time. Thanks to Randy for noticing that I didn't actually assign
'present' to anything.
Unfortunately my original patch passed the few simple test cases I gave it,
purely by coincidence.
Signed-off-by: Nick Piggin <npiggin@suse.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/mincore.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/mm/mincore.c b/mm/mincore.c index 9780097e3812..9c1d0a426e95 100644 --- a/mm/mincore.c +++ b/mm/mincore.c @@ -125,6 +125,8 @@ static long do_mincore(unsigned long addr, unsigned char *vec, unsigned long pag #endif } } + + vec[i] = present; } pte_unmap_unlock(ptep-1, ptl); @@ -135,6 +137,9 @@ none_mapped: pgoff = linear_page_index(vma, addr); for (i = 0; i < nr; i++, pgoff++) vec[i] = mincore_page(vma->vm_file->f_mapping, pgoff); + } else { + for (i = 0; i < nr; i++) + vec[i] = 0; } return nr; |