Re: [PATCH v2 0/6] ASoC: topology: Propagate error appropriately

[Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>]

On 3/27/20 3:47 PM, Amadeusz Sławiński wrote:
> v1:
>    Check if kstrdup succeeded.
> 
> v2:
>    Remove unneeded freeing, which is performed in another place by dobj
>    handlers.
> 
>    Additionally for functions which have return status which was ignored,
>    perform success checks and handle failures in appropriate way.
> 
> Amadeusz Sławiński (6):
>    ASoC: topology: Add missing memory checks
>    ASoC: topology: Check return value of soc_tplg_create_tlv
>    ASoC: topology: Check return value of soc_tplg_*_create
>    ASoC: topology: Check soc_tplg_add_route return value
>    ASoC: topology: Check return value of pcm_new_ver
>    ASoC: topology: Check return value of soc_tplg_dai_config

Looks good to me

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

We probably want Ranjani to double-check this series though.

> 
>   sound/soc/soc-topology.c | 113 ++++++++++++++++++++++++++++++---------
>   1 file changed, 88 insertions(+), 25 deletions(-)
>