Re: [PATCH v4] most: core: fix resource leak in most_register_interface error paths
From: Dan Carpenter
Date: Thu Nov 27 2025 - 12:06:29 EST
On Thu, Nov 27, 2025 at 04:53:37PM +0000, Navaneeth K wrote:
> The function most_register_interface() did not correctly release resources
> if it failed early (before registering the device). In these cases, it
> returned an error code immediately, leaking the memory allocated for the
> interface.
>
> Fix this by initializing the device early via device_initialize() and
> calling put_device() on all error paths.
>
> The most_register_interface() is expected to call put_device() on
> error which frees the resources allocated in the caller. The
> put_device() either calls release_mdev() or dim2_release(),
> depending on the caller.
>
> Switch to using device_add() instead of device_register() to handle
> the split initialization.
>
> Acked-by: Abdun Nihaal <abdun.nihaal@xxxxxxxxx>
> Signed-off-by: Navaneeth K <knavaneeth786@xxxxxxxxx>
> ---
Thanks!
Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
regards,
dan carpenter