From: Diana Craciun <diana.craciun@oss.nxp.com> To: linux-kernel@vger.kernel.org, laurentiu.tudor@nxp.com Cc: stuyoder@gmail.com, leoyang.li@nxp.com, gregkh@linuxfoundation.org, linux-arm-kernel@lists.infradead.org, Diana Craciun <diana.craciun@oss.nxp.com> Subject: [PATCH v5 10/13] bus/fsl_mc: Do not rely on caller to provide non NULL mc_io Date: Tue, 29 Sep 2020 11:54:38 +0300 [thread overview] Message-ID: <20200929085441.17448-11-diana.craciun@oss.nxp.com> (raw) In-Reply-To: <20200929085441.17448-1-diana.craciun@oss.nxp.com> Before destroying the mc_io, check first that it was allocated. Signed-off-by: Diana Craciun <diana.craciun@oss.nxp.com> --- drivers/bus/fsl-mc/mc-io.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/bus/fsl-mc/mc-io.c b/drivers/bus/fsl-mc/mc-io.c index a30b53f1d87d..305015486b91 100644 --- a/drivers/bus/fsl-mc/mc-io.c +++ b/drivers/bus/fsl-mc/mc-io.c @@ -129,7 +129,12 @@ int __must_check fsl_create_mc_io(struct device *dev, */ void fsl_destroy_mc_io(struct fsl_mc_io *mc_io) { - struct fsl_mc_device *dpmcp_dev = mc_io->dpmcp_dev; + struct fsl_mc_device *dpmcp_dev; + + if (!mc_io) + return; + + dpmcp_dev = mc_io->dpmcp_dev; if (dpmcp_dev) fsl_mc_io_unset_dpmcp(mc_io); -- 2.17.1
WARNING: multiple messages have this Message-ID (diff)
From: Diana Craciun <diana.craciun@oss.nxp.com> To: linux-kernel@vger.kernel.org, laurentiu.tudor@nxp.com Cc: gregkh@linuxfoundation.org, Diana Craciun <diana.craciun@oss.nxp.com>, linux-arm-kernel@lists.infradead.org, stuyoder@gmail.com, leoyang.li@nxp.com Subject: [PATCH v5 10/13] bus/fsl_mc: Do not rely on caller to provide non NULL mc_io Date: Tue, 29 Sep 2020 11:54:38 +0300 [thread overview] Message-ID: <20200929085441.17448-11-diana.craciun@oss.nxp.com> (raw) In-Reply-To: <20200929085441.17448-1-diana.craciun@oss.nxp.com> Before destroying the mc_io, check first that it was allocated. Signed-off-by: Diana Craciun <diana.craciun@oss.nxp.com> --- drivers/bus/fsl-mc/mc-io.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/bus/fsl-mc/mc-io.c b/drivers/bus/fsl-mc/mc-io.c index a30b53f1d87d..305015486b91 100644 --- a/drivers/bus/fsl-mc/mc-io.c +++ b/drivers/bus/fsl-mc/mc-io.c @@ -129,7 +129,12 @@ int __must_check fsl_create_mc_io(struct device *dev, */ void fsl_destroy_mc_io(struct fsl_mc_io *mc_io) { - struct fsl_mc_device *dpmcp_dev = mc_io->dpmcp_dev; + struct fsl_mc_device *dpmcp_dev; + + if (!mc_io) + return; + + dpmcp_dev = mc_io->dpmcp_dev; if (dpmcp_dev) fsl_mc_io_unset_dpmcp(mc_io); -- 2.17.1 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-09-29 8:55 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-09-29 8:54 [PATCH v5 00/13] bus/fsl-mc: Extend mc-bus driver functionalities in preparation for mc-bus VFIO support Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 01/13] bus/fsl-mc: Do no longer export the total number of irqs outside dprc_scan_objects Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 02/13] bus/fsl-mc: Add a new parameter to dprc_scan_objects function Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 03/13] bus/fsl-mc: add support for 'driver_override' in the mc-bus Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 04/13] bus/fsl-mc: Set the QMAN/BMAN region flags Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 05/13] bus/fsl-mc: Cache the DPRC API version Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 06/13] bus/fsl-mc: Add dprc-reset-container support Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 07/13] bus/fsl-mc: Export dprc_scan/dprc_remove functions to be used by multiple entities Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 08/13] bus/fsl-mc: Export a cleanup function for DPRC Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 09/13] bus/fsl-mc: Add a container setup function Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` Diana Craciun [this message] 2020-09-29 8:54 ` [PATCH v5 10/13] bus/fsl_mc: Do not rely on caller to provide non NULL mc_io Diana Craciun 2020-09-29 8:54 ` [PATCH v5 11/13] bus/fsl-mc: Export IRQ pool handling functions to be used by VFIO Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 12/13] bus/fsl-mc: Extend ICID size from 16bit to 32bit Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 8:54 ` [PATCH v5 13/13] bus/fsl-mc: Add a new version for dprc_get_obj_region command Diana Craciun 2020-09-29 8:54 ` Diana Craciun 2020-09-29 11:06 ` [PATCH v5 00/13] bus/fsl-mc: Extend mc-bus driver functionalities in preparation for mc-bus VFIO support Laurentiu Tudor 2020-09-29 11:06 ` Laurentiu Tudor 2020-10-02 13:55 ` Greg KH 2020-10-02 13:55 ` Greg KH 2020-10-02 13:56 ` Laurentiu Tudor 2020-10-02 13:56 ` Laurentiu Tudor 2020-10-02 14:05 ` Greg KH 2020-10-02 14:05 ` Greg KH 2020-10-15 2:27 ` Alex Williamson 2020-10-15 2:27 ` Alex Williamson 2020-10-15 17:27 ` Greg KH 2020-10-15 17:27 ` Greg KH
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=20200929085441.17448-11-diana.craciun@oss.nxp.com \ --to=diana.craciun@oss.nxp.com \ --cc=gregkh@linuxfoundation.org \ --cc=laurentiu.tudor@nxp.com \ --cc=leoyang.li@nxp.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=stuyoder@gmail.com \ /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: linkBe 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.