Gabriele Monaco <[email protected]> writes: > On Sun, 2025-11-16 at 15:35 +0000, Nam Cao wrote: >> - reacting = rv_create_file("reacting_on", RV_MODE_WRITE, root_dir, >> NULL, &reacting_on_fops); >> - if (!reacting) >> - goto rm_available; >> + struct dentry *reacting = >> + rv_create_file("reacting_on", RV_MODE_WRITE, root_dir, NULL, >> &reacting_on_fops); > > Nothing is removing "reacting_on" in case of successive failure, is it? > Am I missing anything or couldn't we just set both variables to __free() ?
We can. I overlooked this one :( Nam
