On 17 Mar 2026, at 19:55, Mike Pattrick via dev wrote:
> In jsonrpc_session_recv() use nullable_string_is_equal() instead of
> strcmp() in cases where we haven't already checked that the pointer
> isn't NULL.
>
> Found with clang analyze.
>
> Fixes: 4931f33ad9e2 ("ovsdb-server: Factor out complication by using
> jsonrpc_session.")
> Signed-off-by: Mike Pattrick <[email protected]>
Thanks for looking into this Mike. The change looks good to me.
Acked-by: Eelco Chaudron <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
- [ovs-dev] [PATCH 4/9] process: Check return value o... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 4/9] process: Check retur... Eelco Chaudron via dev
- [ovs-dev] [PATCH 5/9] json: Move buffer declaration... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 5/9] json: Move buffer de... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 5/9] json: Move buffe... Mike Pattrick via dev
- [ovs-dev] [PATCH 6/9] json: Check stream state in j... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 6/9] json: Check stream s... Eelco Chaudron via dev
- [ovs-dev] [PATCH 3/9] odp-util: Initialize snat and... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 3/9] odp-util: Initialize... Eelco Chaudron via dev
- [ovs-dev] [PATCH 7/9] jsonrpc: Avoid strcmp on null... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 7/9] jsonrpc: Avoid strcm... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 7/9] jsonrpc: Avoid strcm... Ilya Maximets
- [ovs-dev] [PATCH 8/9] stream-ssl: Check for ferror ... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 8/9] stream-ssl: Check fo... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 8/9] stream-ssl: Chec... Mike Pattrick via dev
- [ovs-dev] [PATCH 9/9] daemon: Increase scope of sta... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 9/9] daemon: Increase sco... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 1/9] classifier: Avoid undefin... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 1/9] classifier: Avoid un... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 1/9] classifier: Avoid undefin... 0-day Robot
