On Fri, 5 Nov 2021 11:08:39 -0400 Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > We can not rely on enums being signed or unsigned, so the above needs to be: > > if ((unsigned int)type >= TRACEFS_DYNEVENT_MAX) > > > + return NULL; > > > And we should also not undefined types (more than one type masked in), so "we should also not allow undefined types" -- Steve > we really need an unsigned int:
|