Re: [ovs-dev] [PATCH v2 5/6] stream-ssl: Check for ferror and feof in read_cert_file().

[Eelco Chaudron via dev]

On 18 May 2026, at 16:36, Eelco Chaudron wrote:

> On 5 May 2026, at 17:59, Mike Pattrick via dev wrote:
>
>> Previously read_cert_file() only checked for EOF as returned by getc
>> when reading a cert file. This patch checks for ferror and feof before
>> continuing to loop over file contents.
>>
>> Found with clang analyze.
>>
>> Fixes: 9467fe624698 ("Add SSL support to "stream" library and OVSDB.")
>> Signed-off-by: Mike Pattrick <m...@redhat.com>
>
> Thanks for taking my suggestion, but your commit message needs some updating.

I guess if you just reply here with a suggestion, I can change it
when applying. The rest seems fine, so there is no need to respin
the whole series for this.

> //Eelco

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