Re: [PATCH v3 2/4] tracing/user_events: Introduce multi-format events

2024-02-21 Thread Beau Belgrave
On Wed, Feb 21, 2024 at 10:08:33AM -0500, Steven Rostedt wrote: > On Wed, 14 Feb 2024 17:50:44 + > Beau Belgrave wrote: > > > +static char *user_event_group_system_multi_name(void) > > +{ > > + char *system_name; > > + int len = sizeof(USER_EVENTS_MULTI_SYSTEM) + 1; > > FYI, the

Re: [PATCH v3 2/4] tracing/user_events: Introduce multi-format events

2024-02-21 Thread Beau Belgrave
On Wed, Feb 21, 2024 at 10:21:04AM -0500, Steven Rostedt wrote: > On Wed, 14 Feb 2024 17:50:44 + > Beau Belgrave wrote: > > > Currently user_events supports 1 event with the same name and must have > > the exact same format when referenced by multiple programs. This opens > > an opportunity

Re: [PATCH v3 2/4] tracing/user_events: Introduce multi-format events

2024-02-21 Thread Steven Rostedt
On Wed, 14 Feb 2024 17:50:44 + Beau Belgrave wrote: > Currently user_events supports 1 event with the same name and must have > the exact same format when referenced by multiple programs. This opens > an opportunity for malicous or poorly thought through programs to > create events that

Re: [PATCH v3 2/4] tracing/user_events: Introduce multi-format events

2024-02-21 Thread Steven Rostedt
On Wed, 14 Feb 2024 17:50:44 + Beau Belgrave wrote: > +static char *user_event_group_system_multi_name(void) > +{ > + char *system_name; > + int len = sizeof(USER_EVENTS_MULTI_SYSTEM) + 1; FYI, the sizeof() will include the "\0" so no need for "+ 1", but I don't think this matters

[PATCH v3 2/4] tracing/user_events: Introduce multi-format events

2024-02-14 Thread Beau Belgrave
Currently user_events supports 1 event with the same name and must have the exact same format when referenced by multiple programs. This opens an opportunity for malicous or poorly thought through programs to create events that others use with different formats. Another scenario is user programs