tree b7d7b6fbc425ed891d940edf38825aa36cafa8c2
parent 44662f90cda7ce0b65e77a7f1eefe45fb9053a4e
author Takashi Iwai <tiwai@suse.de> 1550591209 +0100
committer Mark Brown <broonie@kernel.org> 1550664757 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 iQFHBAABCgAxFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlxtRDkTHGJyb29uaWVA
 a2VybmVsLm9yZwAKCRAk1otyXVSH0DDUB/96YLSMUMV6PyOtR027vxVxK2LB4lvf
 bmy5vCHzn9KHgm9Mjn3M0CilLxDf+oejcYEwWimQ7DL7g7nrMBACBR9j4UEOg5sX
 ftjrJ0l1odG++dpKh+qdi+gAZ9jE7CnfBCXh7/bpzCpRp9E71K0kW620uACrlQS5
 Wx4ZuASsR9NfO1B2WOws2zCHuHRCEBQcTFHtMFseslMVuu8uQtZcnnYqNuIG7Hob
 dGkw7aVWdCGEVm+4s97jP8GpPpjTmgo5pAq8egvSddxGIDnVQrX8R2XF+sQRG5ce
 7x3NGrwAXvwiLcwBhO6LSxWOtkf10M5320yiSrAyzf5QcKxKeQCbf3s/
 =HcnK
 -----END PGP SIGNATURE-----

ASoC: simple-card: Fix of-node refcount unbalance in DAI-link parser

The function simple_for_each_link() has a few missing places that
forgot unrefereing of-nodes after the use.  The main do-while loop
may abort when loop=0, and this leaves the node object still
referenced.  A similar leak is found in the error handling of NULL
codec that aborts the loop as well.  Last but not least, the inner
for_each_child_of_node() loop may abort in the middle, and this leaks
the refcount of the iterator node.

This patch addresses these missing refcount issues.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Signed-off-by: Mark Brown <broonie@kernel.org>