[PATCH v2 0/6] ASoC: topology: Propagate error appropriately
Amadeusz Sławiński
amadeuszx.slawinski at linux.intel.com
Wed Apr 8 10:51:26 CEST 2020
On 3/30/2020 6:38 PM, Pierre-Louis Bossart wrote:
>
>
> 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 at linux.intel.com>
>
> We probably want Ranjani to double-check this series though.
>
Hi Ranjani, can you take another look, I would like for this to get
merged before I forget about it ;)
>>
>> sound/soc/soc-topology.c | 113 ++++++++++++++++++++++++++++++---------
>> 1 file changed, 88 insertions(+), 25 deletions(-)
>>
More information about the Alsa-devel
mailing list