[openssl/openssl] 32185d: Fix second error from Coverity-161057

Neil Horman noreply at github.com
Fri Jul 26 17:40:56 UTC 2024


  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 32185d513cf8732ee0a85875ac61ee4389a86bbb
      https://github.com/openssl/openssl/commit/32185d513cf8732ee0a85875ac61ee4389a86bbb
  Author: Neil Horman <nhorman at openssl.org>
  Date:   2024-07-26 (Fri, 26 Jul 2024)

  Changed paths:
    M ssl/ssl_conf.c

  Log Message:
  -----------
  Fix second error from Coverity-161057

Coverity flagged a second error in this code

we're comparing block_padding and hs_padding for >= 0, which is always
true

With the change to the use of strtoul, inputs that are preceded with a -
(i.e. negative values), are caught already, so the check is redundant

just remove the check entirely

Reviewed-by: Paul Dale <ppzgs1 at gmail.com>
Reviewed-by: Tom Cosgrove <tom.cosgrove at arm.com>
(Merged from https://github.com/openssl/openssl/pull/24993)



To unsubscribe from these emails, change your notification settings at https://github.com/openssl/openssl/settings/notifications


More information about the openssl-commits mailing list