On Thu, 2018-04-05 at 14:23 +0300, Dan Carpenter wrote: > The problem here is that we allocate "data". Then we do > "data = PTR_ALIGN(data, 8);" and then we free the aligned pointer and > not the one we allocated. That seems pretty pointless, since kmalloc guarantees such alignment for sure. Better to just remove PTR_ALIGN()? johannes