On Tue, 30 Jul 2024 11:06:57 -0400 Steven Rostedt <[email protected]> wrote:
> Link: > https://lore.kernel.org/all/[email protected]/ > > Cc: [email protected] > Fixes: b63db58e2fa5d ("eventfs/tracing: Add callback for release of an > eventfs_inode") > Reported-by: Mathias Krause <[email protected]> > Tested-by: Mathias Krause <[email protected]> Hi Mathias, I kept your "Tested-by" tag because you did test the part that fixes the bug you reported. I just added on top of that one to handle other possibly cases. Let me know if you want me to drop that tag, or if you tested this version as well. Thanks, -- Steve > Signed-off-by: Steven Rostedt (Google) <[email protected]> > --- > Changes since v3: > https://lore.kernel.org/[email protected] > > - After inspecting the code, there's several users of event_file_data() > that return the trace_event_file descriptor when it could be set to be > freed. Add a new helper function: event_file_file() that will make sure > the event_mutex is held and check the flag and return NULL if the file is > freed.
