On 17 Mar 2026, at 19:55, Mike Pattrick via dev wrote:
> Clang analyzer incorrectly reports that an uninitialized buffer is
> passed to json_parser_feed(). The buffer is initialized by fread()
> before use, but declaring it inside the loop scope confuses the
> analyzer's control flow analysis on subsequent loop iterations.
>
> Found with clang analyze.
>
> Fixes: f38b84ea2b6b ("Implement JSON parsing and serialization.")
> Signed-off-by: Mike Pattrick <[email protected]>
Hi Mike,
If I apply your next patch and undo this change, I get no static analyzer
warning.
I'm using clang version 19.1.7 (Fedora 19.1.7-5.fc41).
Cheers,
Eelco
_______________________________________________
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
