Re: [PATCH] ASoC: Add a sanity check before using dai driver name

From: Mark Brown
Date: Tue Aug 22 2017 - 12:18:04 EST


On Tue, Aug 22, 2017 at 10:45:12PM +0800, Jeffy Chen wrote:

> - return dai;
> + if (!dlc->dai_name)
> + return dai;
> + if (!strcmp(dai->name, dlc->dai_name))
> + return dai;

You want (dlc->dai_name && !strcmp(dai->name, dlc->dai_name)) for this
to be equivalent don't you?

> + if (dai->driver->name &&
> + !strcmp(dai->driver->name, dlc->dai_name))

Attachment: signature.asc
Description: PGP signature