Re: [ovs-dev] [PATCH 4/9] process: Check return value of get_raw_process_info().

[Eelco Chaudron via dev]

On 17 Mar 2026, at 19:55, Mike Pattrick via dev wrote:

> In get_process_info(), the call to get_raw_process_info() for the
> parent process did not check the return value.  If the call failed
> (e.g., parent process no longer exists), the uninitialized parent
> struct would be used in strcmp() and other operations, leading to
> undefined behavior.
>
> Fix by checking the return value and returning early on failure.
>
> Found with clang analyze.
>
> Fixes: ff1d2c1626b2 ("process: Consolidate process related APIs.")
> Signed-off-by: Mike Pattrick <m...@redhat.com>

The change looks good to me.

Acked-by: Eelco Chaudron <echau...@redhat.com>

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev