On Tue, Aug 26, 2025 at 8:35 AM Eelco Chaudron via dev < [email protected]> wrote:
> Coverity reported a possible null dereference of t->reply > when handling errors in the "committing" state for "transact" requests. > > In practice, t->reply is always non-NULL at this point, but static analysis > cannot infer this guarantee. Adding ovs_assert(t->reply) documents the > invariant for developers and ensures debug builds will catch any misuse. > > No change in runtime behavior; this only improves code safety and > satisfies static analysis. > > Signed-off-by: Eelco Chaudron <[email protected]> > --- > Acked-by: Mike Pattrick <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
