Revision 2cff98b99c469880ce830cbcde015b53b67e0a7b authored by Dean Nelson on 29 April 2015, 15:09:18 UTC, committed by Will Deacon on 29 April 2015, 16:39:39 UTC
__dma_alloc() does a PAGE_ALIGN() on the passed in size argument before
doing anything else. __dma_free() does not. And because it doesn't, it is
possible to leak memory should size not be an integer multiple of PAGE_SIZE.

The solution is to add a PAGE_ALIGN() to __dma_free() like is done in
__dma_alloc().

Additionally, this patch removes a redundant PAGE_ALIGN() from
__dma_alloc_coherent(), since __dma_alloc_coherent() can only be called
from __dma_alloc(), which already does a PAGE_ALIGN() before the call.

Cc: stable@vger.kernel.org
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
Signed-off-by: Dean Nelson <dnelson@redhat.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
1 parent 6829e27
History
File Mode Size
Kconfig -rw-r--r-- 1.7 KB
Makefile -rw-r--r-- 456 bytes
bcm_sf2.c -rw-r--r-- 27.2 KB
bcm_sf2.h -rw-r--r-- 3.6 KB
bcm_sf2_regs.h -rw-r--r-- 7.3 KB
mv88e6060.c -rw-r--r-- 6.7 KB
mv88e6123_61_65.c -rw-r--r-- 8.3 KB
mv88e6131.c -rw-r--r-- 8.4 KB
mv88e6171.c -rw-r--r-- 8.7 KB
mv88e6352.c -rw-r--r-- 12.5 KB
mv88e6xxx.c -rw-r--r-- 33.5 KB
mv88e6xxx.h -rw-r--r-- 12.6 KB

back to top