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);
> +}


Reply via email to