From: Peter Chen <peter.chen@nxp.com>
To: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
Cc: "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>,
dl-linux-imx <linux-imx@nxp.com>
Subject: [1/1] usb: chipidea: fix static checker warning for NULL pointer
Date: Fri, 18 Jan 2019 01:35:52 +0000 [thread overview]
Message-ID: <VI1PR04MB532727B3D471639D918E75B48B9C0@VI1PR04MB5327.eurprd04.prod.outlook.com> (raw)
>
> On Thu, Jan 17, 2019 at 09:24:20AM +0000, Peter Chen wrote:
> > During the static checker, "data->usbmisc_data" may be NULL.
>
> I can not quite parse this sentance. Do you mean that a static checker found this
> issue?
I think "yes", Dan Carpenter reported it.
https://www.spinics.net/lists/linux-usb/msg175431.html
>
> > Fix it by adding this pointer judgement before using.
> >
> > Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
> > Signed-off-by: Peter Chen <peter.chen@nxp.com>
> > ---
> > drivers/usb/chipidea/ci_hdrc_imx.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/usb/chipidea/ci_hdrc_imx.c
> > b/drivers/usb/chipidea/ci_hdrc_imx.c
> > index e81de9ca8729..9b45aa422e69 100644
> > --- a/drivers/usb/chipidea/ci_hdrc_imx.c
> > +++ b/drivers/usb/chipidea/ci_hdrc_imx.c
> > @@ -316,7 +316,8 @@ static int ci_hdrc_imx_probe(struct platform_device
> *pdev)
> > if (IS_ERR(data->usbmisc_data))
> > return PTR_ERR(data->usbmisc_data);
> >
> > - if (of_usb_get_phy_mode(dev->of_node) ==
> USBPHY_INTERFACE_MODE_HSIC) {
> > + if ((of_usb_get_phy_mode(dev->of_node) ==
> USBPHY_INTERFACE_MODE_HSIC)
> > + && data->usbmisc_data) {
>
> Are you sure that is correct? If usbmisc_data is NULL, then you just took out a
> bunch of checking and lookups. Don't you normally need to do this even if that
> pointer is NULL?
>
If usbmisc_data is NULL, there is a NULL pointer deference. And if usbmisc_data
is NULL, we don't need to do rest of this if {} statement.
This can be error pointers on error or NULL if there is no misc data.
316 if (IS_ERR(data->usbmisc_data))
317 return PTR_ERR(data->usbmisc_data);
318
319 if (of_usb_get_phy_mode(dev->of_node) == USBPHY_INTERFACE_MODE_HSIC) {
320 pdata.flags |= CI_HDRC_IMX_IS_HSIC;
321 data->usbmisc_data->hsic = 1;
^^^^^^^^^^^^^^^^^^^^^^^^
> Will the rest of the function still work properly if that is the case?
>
At this if {} statement, the usbmisc_data needs to be set. The rest of code in this probe can work proper
if usbmisc_data is NULL.
Peter
next reply other threads:[~2019-01-18 1:35 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-01-18 1:35 Peter Chen [this message]
-- strict thread matches above, loose matches on Subject: below --
2019-01-17 10:58 [1/1] usb: chipidea: fix static checker warning for NULL pointer Greg Kroah-Hartman
2019-01-17 9:24 Peter Chen
2018-12-24 1:18 Peter Chen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=VI1PR04MB532727B3D471639D918E75B48B9C0@VI1PR04MB5327.eurprd04.prod.outlook.com \
--to=peter.chen@nxp.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-imx@nxp.com \
--cc=linux-usb@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.