[alsa-devel] [PATCH v7 07/11] ASoC: topology: Add support to configure existing physical DAI links

[Lin, Mengdong]

> -----Original Message-----
> From: Mark Brown [mailto:broonie at kernel.org]
> Sent: Saturday, November 5, 2016 1:22 AM
> To: mengdong.lin at linux.intel.com
> Cc: alsa-devel at alsa-project.org; tiwai at suse.de;
> liam.r.girdwood at linux.intel.com; Shah, Hardik T <hardik.t.shah at intel.com>;
> Singh, Guneshwor O <guneshwor.o.singh at intel.com>; Koul, Vinod
> <vinod.koul at intel.com>; Ughreja, Rakesh A <rakesh.a.ughreja at intel.com>;
> Lin, Mengdong <mengdong.lin at intel.com>
> Subject: Re: [PATCH v7 07/11] ASoC: topology: Add support to configure
> existing physical DAI links
> 
> On Thu, Nov 03, 2016 at 01:04:27AM +0800, mengdong.lin at linux.intel.com
> wrote:
> 
> > +	name = strlen(cfg->name) ? cfg->name : NULL;
> > +	stream_name = strlen(cfg->stream_name) ? cfg->stream_name : NULL;
> 
> Please replace these with normal if statements.  Also, are we sure that we've
> sufficiently audited the input paths to ensure that the strings are always
> terminated?

Okay. I submitted patch to fix this -  ASoC: topology: Check name strings of physical DAI links.

This another minor ABI update, just a name change of a field and backward compatible.

Thanks again for your review!
Mengdong