>> Thanks for the patch, but this one is already taken care of along >> with other similar uses of kmalloc/copy: >> >> http://marc.info/?l=linux-rdma&m=148656088729538&w=2 >> > > Michael's patch doesn't change user_sdma_free_request() so it introduces > a kfreeing an error pointer bug.
Did you notice that another local variable “tmp” was introduced in the update step “[PATCH 27/27] IB/hfi1: Code reuse with memdup_copy” so that the mentioned function will usually get a null pointer after a failure there? Regards, Markus
