From: Colin Cross <ccr...@android.com>

Check the return value of remap_pfn_range and return an error if
it fails.

Signed-off-by: Colin Cross <ccr...@android.com>
Signed-off-by: John Stultz <john.stu...@linaro.org>
---
 drivers/staging/android/ion/ion_heap.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/android/ion/ion_heap.c 
b/drivers/staging/android/ion/ion_heap.c
index ce31561..ecc14fd 100644
--- a/drivers/staging/android/ion/ion_heap.c
+++ b/drivers/staging/android/ion/ion_heap.c
@@ -76,6 +76,7 @@ int ion_heap_map_user(struct ion_heap *heap, struct 
ion_buffer *buffer,
        unsigned long offset = vma->vm_pgoff * PAGE_SIZE;
        struct scatterlist *sg;
        int i;
+       int ret;
 
        for_each_sg(table->sgl, sg, table->nents, i) {
                struct page *page = sg_page(sg);
@@ -91,8 +92,10 @@ int ion_heap_map_user(struct ion_heap *heap, struct 
ion_buffer *buffer,
                        offset = 0;
                }
                len = min(len, remainder);
-               remap_pfn_range(vma, addr, page_to_pfn(page), len,
+               ret = remap_pfn_range(vma, addr, page_to_pfn(page), len,
                                vma->vm_page_prot);
+               if (ret)
+                       return ret;
                addr += len;
                if (addr >= vma->vm_end)
                        return 0;
-- 
1.8.3.2

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to