diff options
author | Sherry Yang <sherryy@android.com> | 2017-10-20 20:58:59 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-10-21 04:14:20 -0400 |
commit | ae65c8510f3319dfb2114cc48d476b81232e27b3 (patch) | |
tree | 3a94ab0f22bdab90c95d824b0d736b294c467233 /drivers/android/binder_alloc.c | |
parent | a0c2baaf81bd53dc76fccdddc721ba7dbb62be21 (diff) |
android: binder: Fix null ptr dereference in debug msg
Don't access next->data in kernel debug message when the
next buffer is null.
Acked-by: Arve Hjønnevåg <arve@android.com>
Signed-off-by: Sherry Yang <sherryy@android.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/android/binder_alloc.c')
-rw-r--r-- | drivers/android/binder_alloc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/android/binder_alloc.c b/drivers/android/binder_alloc.c index e12072b1d507..c2819a3d58a6 100644 --- a/drivers/android/binder_alloc.c +++ b/drivers/android/binder_alloc.c | |||
@@ -560,7 +560,7 @@ static void binder_delete_free_buffer(struct binder_alloc *alloc, | |||
560 | binder_alloc_debug(BINDER_DEBUG_BUFFER_ALLOC, | 560 | binder_alloc_debug(BINDER_DEBUG_BUFFER_ALLOC, |
561 | "%d: merge free, buffer %pK do not share page with %pK or %pK\n", | 561 | "%d: merge free, buffer %pK do not share page with %pK or %pK\n", |
562 | alloc->pid, buffer->data, | 562 | alloc->pid, buffer->data, |
563 | prev->data, next->data); | 563 | prev->data, next ? next->data : NULL); |
564 | binder_update_page_range(alloc, 0, buffer_start_page(buffer), | 564 | binder_update_page_range(alloc, 0, buffer_start_page(buffer), |
565 | buffer_start_page(buffer) + PAGE_SIZE, | 565 | buffer_start_page(buffer) + PAGE_SIZE, |
566 | NULL); | 566 | NULL); |