Le 27/10/2015 11:27, Jarkko Sakkinen a écrit :
On Fri, Oct 23, 2015 at 10:37:33AM +0300, Jarkko Sakkinen wrote:
On Thu, Oct 22, 2015 at 10:32:41PM +0200, Christophe JAILLET wrote:
Reference to the 'np' node is dropped before dereferencing the 'sizep' and
'basep' pointers, which could by then point to junk if the node has been
freed.
Refactor code to call 'of_node_pup' later.
Signed-off-by: Christophe JAILLET <christophe.jaillet@xxxxxxxxxx>
LGTM.
Is there anyone able to provide Tested-by for this?
Christophe, were you able to reproduce the crash (insmod/rmmod couple
of times maybe?) and validate that it was gone after fixing the bug?
Hi,
no, I never triggered the bug.
This is just something noticed while looking at potential issues related
to incorrect use of 'of_node_pup'.
I only compile tested the patch.
Best regards,
CJ
Reviewed-by: Jarkko Sakkinen <jarkko.sakkinen@xxxxxxxxxxxxxxx>
/Jarkko
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
