On Mon, 26 Jan 2026 10:43:58 +0000
Vincent Donnefort <[email protected]> wrote:
> +
> +static int __cmp_events(const void *id, const void *evt)
> +{
> + return (long)id - ((struct remote_event *)evt)->id;
Hmm, just to be safe, we should probably make everything int here, to not
have any overflow issues:
static int __cmp_events(const void *key, const void *data)
{
const struct remote_event *evt = data;
int id = (int)((long)key);
return id - (int)evt->id;
}
-- Steve
> +}
> +
> +static struct remote_event *trace_remote_find_event(struct trace_remote
> *remote, unsigned short id)
> +{
> + return bsearch((const void *)(unsigned long)id, remote->events,
> remote->nr_events,
> + sizeof(*remote->events), __cmp_events);
> +}