Eelco Chaudron <echau...@redhat.com> writes:

> Coverity reports that daemon_set_new_user() may receive a large
> unsigned value from get_sysconf_buffer_size(), due to sysconf()
> returning -1 and being cast to size_t.
>
> Although this would likely lead to an allocation failure and abort,
> it's better to handle the error in place.
>
> Signed-off-by: Eelco Chaudron <echau...@redhat.com>
>
> ---
> v2: added comment to the code on -1 return reasons.
> ---

LGTM,

Acked-by: Aaron Conole <acon...@redhat.com>

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

Reply via email to