ASoC: wcd9335: fix a leaked reference by adding missing of_node_put
authorWen Yang <wen.yang99@zte.com.cn>
Tue, 26 Feb 2019 08:17:49 +0000 (16:17 +0800)
committerMark Brown <broonie@kernel.org>
Tue, 2 Apr 2019 05:57:56 +0000 (12:57 +0700)
The call to of_parse_phandle returns a node pointer with refcount
incremented thus it must be explicitly decremented after the last
usage.

Detected by coccinelle with the following warnings:
./sound/soc/codecs/wcd9335.c:5193:2-8: ERROR: missing of_node_put; acquired a node pointer with refcount incremented on line 5183, but without a correspon    ding object release within this function.

Signed-off-by: Wen Yang <wen.yang99@zte.com.cn>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Mark Brown <broonie@kernel.org>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: Dan Carpenter <dan.carpenter@oracle.com> (commit_signer:1/11=9%,authored:1/11=9%)
Cc: alsa-devel@alsa-project.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/wcd9335.c

index 981f88a5f615412f4b3d09eebc597536d378ec70..a04a7cedd99dee6a7272b3ab324405048b80de1b 100644 (file)
@@ -5188,6 +5188,7 @@ static int wcd9335_slim_status(struct slim_device *sdev,
 
        wcd->slim = sdev;
        wcd->slim_ifc_dev = of_slim_get_device(sdev->ctrl, ifc_dev_np);
+       of_node_put(ifc_dev_np);
        if (!wcd->slim_ifc_dev) {
                dev_err(dev, "Unable to get SLIM Interface device\n");
                return -EINVAL;