Re: [PATCH 2/2] ASoC: spacemit: init *dp to NULL before error paths

"Troy Mitchell" <[email protected]> Sun, 02 Aug 2026 19:29:42 -0700

Newsgroups	gmane.linux.ports.riscv,gmane.linux.sound,gmane.linux.kernel
Message-ID	<[email protected]>

> The current caller does check the return value before using dp, so this
> isn't an active bug. Still, initialize *dp to NULL upfront as a defensive
> measure, consistent with how core helpers like _snd_pcm_new() handle
> their optional output parameters.

If devm_kmemdup() fails, spacemit_i2s_init_dai() returns -ENOMEM. The
sole caller checks the return value and returns immediately, so it never
accesses dai on that path. On success, dp is non-NULL and *dp is assigned
before the function returns.

Therefore, I do not see a path where initializing *dp to NULL has any
effect. Could you clarify what case this change is intended to handle?

                                            - Troy

_______________________________________________
linux-riscv mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-riscv

signature.asc (application/pgp-signature, 248 B)