Re: [PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm()

2017-07-24 Thread Dan Carpenter
> > Cc: Leo Li ; linuxppc-dev@lists.ozlabs.org; kernel- > > janit...@vger.kernel.org > > Subject: Re: [PATCH 1/2] fsl/qe: NULL dereference on error in > > ucc_of_parse_tdm() > > > > On Mon, Jul 24, 2017 at 02:24:14AM +, Qiang Zhao wrote: > > > On Sat 7/22/

RE: [PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm()

2017-07-24 Thread Qiang Zhao
[mailto:dan.carpen...@oracle.com] > > > Sent: Saturday, July 22, 2017 3:34 PM > > > To: Qiang Zhao > > > Cc: Leo Li ; linuxppc-dev@lists.ozlabs.org; > > > kernel- janit...@vger.kernel.org > > > Subject: [PATCH 1/2] fsl/qe: NULL dereference on error in >

Re: [PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm()

2017-07-24 Thread Dan Carpenter
> > Cc: Leo Li ; linuxppc-dev@lists.ozlabs.org; kernel- > > janit...@vger.kernel.org > > 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 > >

RE: [PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm()

2017-07-23 Thread Qiang Zhao
; 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 with

[PATCH 1/2] fsl/qe: NULL dereference on error in ucc_of_parse_tdm()

2017-07-22 Thread Dan Carpenter
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. Fixes: 35ef1c20fdb2 ("fsl/qe: Add QE TDM lib") Signed-off-by: Dan Carpenter diff