We should free 'feature->irq_ctx[idx].name' to avoid 'name' memory leak when request_irq() failed. Signed-off-by: Ma Ke <make24@xxxxxxxxxxx> --- drivers/fpga/dfl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/fpga/dfl.c b/drivers/fpga/dfl.c index 094ee97ea26c..c52ebece5aef 100644 --- a/drivers/fpga/dfl.c +++ b/drivers/fpga/dfl.c @@ -1911,7 +1911,7 @@ static int do_set_irq_trigger(struct dfl_feature *feature, unsigned int idx, feature->irq_ctx[idx].name, trigger); if (!ret) { feature->irq_ctx[idx].trigger = trigger; - return ret; + goto free_name; } eventfd_ctx_put(trigger); -- 2.25.1