Re: [PATCH] fpga: dfl: fix potential memory leak in vfio_intx_enable()

From: Markus Elfring
Date: Sun Jun 23 2024 - 08:53:17 EST


> We should free 'feature->irq_ctx[idx].name' to avoid 'name'
> memory leak when request_irq() failed.

1. Please choose an imperative wording for an improved change description.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.10-rc4#n94

2. Would you like to add any tags (like “Fixes” and “Cc”) accordingly?

3. How do you think about to use a summary phrase like “Avoid memory leak
in do_set_irq_trigger()”?

4. Under which circumstances will development interests grow for increasing
the application of scope-based resource management?
https://elixir.bootlin.com/linux/v6.10-rc4/source/include/linux/cleanup.h#L8


Regards,
Markus