From: Viktor Dukhovni <openssl-users@dukhovni.org>
Date: Sat, 21 Nov 2015 05:59:07 +0000 (-0500)
Subject: Good hygiene with size_t output argument.
X-Git-Tag: OpenSSL_1_0_2e~22
X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=03bf7127808e3a8fbdddda42763458d5da14127e;p=oweals%2Fopenssl.git

Good hygiene with size_t output argument.

Though the callers check the function return value and ignore the
size_t output argument on failure, it is still often not ideal to
store -1 in a size_t on error.  That might signal an unduly large
buffer.  Instead set the size_t to 0, to indicate no space.

Reviewed-by: Richard Levitte <levitte@openssl.org>
---

diff --git a/ssl/s3_cbc.c b/ssl/s3_cbc.c
index f3aa8781e2..557622f513 100644
--- a/ssl/s3_cbc.c
+++ b/ssl/s3_cbc.c
@@ -520,7 +520,7 @@ int ssl3_cbc_digest_record(const EVP_MD_CTX *ctx,
          */
         OPENSSL_assert(0);
         if (md_out_size)
-            *md_out_size = -1;
+            *md_out_size = 0;
         return 0;
     }