On 17 Mar 2026, at 19:55, Mike Pattrick via dev wrote:
> Previously json_from_stream() read a file until a 0 byte read. Clang
> complained that fread on a file in eof or error state is an undefined
> behaviour.
>
> Found with clang analyze.
>
> Fixes: f38b84ea2b6b ("Implement JSON parsing and serialization.")
> Signed-off-by: Mike Pattrick <[email protected]>
Thanks Mike for the patch! This change makes sense to me.
Acked-by: Eelco Chaudron <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
- [ovs-dev] [PATCH 1/9] classifier: Avoid undefined b... Mike Pattrick via dev
- [ovs-dev] [PATCH 2/9] dp-packet-gso: Check pac... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 2/9] dp-packet-gso: C... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 2/9] dp-packet-gs... Mike Pattrick via dev
- [ovs-dev] [PATCH 4/9] process: Check return va... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 4/9] process: Check r... Eelco Chaudron via dev
- [ovs-dev] [PATCH 5/9] json: Move buffer declar... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 5/9] json: Move buffe... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 5/9] json: Move b... Mike Pattrick via dev
- [ovs-dev] [PATCH 6/9] json: Check stream state... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 6/9] json: Check stre... Eelco Chaudron via dev
- [ovs-dev] [PATCH 3/9] odp-util: Initialize sna... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 3/9] odp-util: Initia... Eelco Chaudron via dev
- [ovs-dev] [PATCH 7/9] jsonrpc: Avoid strcmp on... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 7/9] jsonrpc: Avoid s... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 7/9] jsonrpc: Avoid s... Ilya Maximets
- [ovs-dev] [PATCH 8/9] stream-ssl: Check for fe... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 8/9] stream-ssl: Chec... Eelco Chaudron via dev
- Re: [ovs-dev] [PATCH 8/9] stream-ssl: ... Mike Pattrick via dev
- [ovs-dev] [PATCH 9/9] daemon: Increase scope o... Mike Pattrick via dev
- Re: [ovs-dev] [PATCH 9/9] daemon: Increase... Eelco Chaudron via dev
