diff options
author | Matt Caswell <matt@openssl.org> | 2016-04-25 13:56:44 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2016-06-16 09:50:48 +0100 |
commit | cf3404fcc77aaf592c95326cbdd25612a8af6878 (patch) | |
tree | f85b17508cae4736176c27266148a23f64a7571a /crypto/evp/encode.c | |
parent | 2ac6115d9ee35308300b82d96078d03d81e7d320 (diff) | |
download | openssl-cf3404fcc77aaf592c95326cbdd25612a8af6878.tar.gz |
Change the return type of EVP_EncodeUpdate
Previously EVP_EncodeUpdate returned a void. However there are a couple
of error conditions that can occur. Therefore the return type has been
changed to an int, with 0 indicating error and 1 indicating success.
Reviewed-by: Rich Salz <rsalz@openssl.org>
Diffstat (limited to 'crypto/evp/encode.c')
-rw-r--r-- | crypto/evp/encode.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/crypto/evp/encode.c b/crypto/evp/encode.c index bd2bbc0b09..e026a8f0ca 100644 --- a/crypto/evp/encode.c +++ b/crypto/evp/encode.c @@ -114,7 +114,7 @@ void EVP_EncodeInit(EVP_ENCODE_CTX *ctx) ctx->line_num = 0; } -void EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, +int EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, const unsigned char *in, int inl) { int i, j; @@ -122,12 +122,12 @@ void EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, *outl = 0; if (inl <= 0) - return; + return 0; OPENSSL_assert(ctx->length <= (int)sizeof(ctx->enc_data)); if (ctx->length - ctx->num > inl) { memcpy(&(ctx->enc_data[ctx->num]), in, inl); ctx->num += inl; - return; + return 1; } if (ctx->num != 0) { i = ctx->length - ctx->num; @@ -153,12 +153,14 @@ void EVP_EncodeUpdate(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl, if (total > INT_MAX) { /* Too much output data! */ *outl = 0; - return; + return 0; } if (inl != 0) memcpy(&(ctx->enc_data[0]), in, inl); ctx->num = inl; *outl = total; + + return 1; } void EVP_EncodeFinal(EVP_ENCODE_CTX *ctx, unsigned char *out, int *outl) |