On Mon, Jul 24, 2017 at 02:24:14AM +0000, Qiang Zhao wrote: > On Sat 7/22/2017 3:34 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > > > -----Original Message----- > > From: Dan Carpenter [mailto:dan.carpenter@xxxxxxxxxx] > > Sent: Saturday, July 22, 2017 3:34 PM > > To: Qiang Zhao <qiang.zhao@xxxxxxx> > > Cc: Leo Li <leoyang.li@xxxxxxx>; linuxppc-dev@xxxxxxxxxxxxxxxx; kernel- > > janitors@xxxxxxxxxxxxxxx > > Subject: [PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm() > > > > If "pdev = of_find_device_by_node(np2);" fails then it would lead to a NULL > > dereference. This function is called from probe() and we're using managed > > resources so we can just return without doing a manual cleanup. > > You mean it will be cleaned up automatically? Yes. At module unload. regards, dan carpenter -- 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