[alsa-devel] DPCM: suspicious code in dpcm_prune_paths()

Mark Brown broonie at kernel.org
Thu Sep 19 14:19:59 CEST 2019


On Tue, Sep 10, 2019 at 11:20:17AM +0200, Guennadi Liakhovetski wrote:

> This code in dpcm_prune_paths() seems to have no effect:

> 		/* is there a valid CODEC DAI widget for this BE */
> 		for_each_rtd_codec_dai(dpcm->be, i, dai) {
> 			widget = dai_get_widget(dai, stream);
> 
> 			/* prune the BE if it's no longer in our active list */
> 			if (widget && widget_in_list(list, widget))
> 				continue;
> 		}

> Neither of the variables, that are modified in this loop are used outside 
> of it. Was the "continue" maybe meant to continue the encompassing loop?

Liam?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20190919/0b1f6a2c/attachment-0001.sig>


More information about the Alsa-devel mailing list