mmap: check ->vm_ops before dereferencing

Check whether the VMA has a vm_ops before calling close, just like we check vm_ops before calling open a few dozen lines higher up in the function. Signed-off-by: Rik van Riel <[email protected]> Reported-by: Dan Carpenter <[email protected]> Signed-off-by: Linus Torvalds <[email protected]>
chewitt · Apr 27, 2010 · 5892753 · 5892753
1 parent a231a1f
commit 5892753
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/mm/mmap.c b/mm/mmap.c
@@ -1977,7 +1977,8 @@ static int __split_vma(struct mm_struct * mm, struct vm_area_struct * vma,
 		return 0;
 
 	/* Clean everything up if vma_adjust failed. */
-	new->vm_ops->close(new);
+	if (new->vm_ops && new->vm_ops->close)
+		new->vm_ops->close(new);
 	if (new->vm_file) {
 		if (vma->vm_flags & VM_EXECUTABLE)
 			removed_exe_file_vma(mm);