diff options
| author |  vnugent <public@vaughnnugent.com> | 2024-11-03 12:44:22 -0500 |
|---|---|---|
| committer | vnugent <public@vaughnnugent.com> | 2024-11-03 12:44:22 -0500 |
| commit | e0d30c1d8f407bfef05a9cc36398bb0894a96c39 (patch) | |
| tree | fd2f467d907ab79e8d903defdf8241ffab7139b9 /src | |
| parent | b8a85bed14044c2d40b9cad3c19f878468e4f408 (diff) | |
Once over and fix all valgrind warnings
Diffstat (limited to 'src')
| -rw-r--r-- | src/providers/openssl-helpers.c | 5 | ||||
| -rw-r--r-- | src/providers/openssl.c | 6 |
2 files changed, 5 insertions, 6 deletions
diff --git a/src/providers/openssl-helpers.c b/src/providers/openssl-helpers.c index bc3a272..5369457 100644 --- a/src/providers/openssl-helpers.c +++ b/src/providers/openssl-helpers.c @@ -2,7 +2,7 @@ * Copyright (c) 2024 Vaughn Nugent * * Package: noscrypt -* File: providers/openssl.c +* File: providers/openssl-helpers.c * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public License @@ -113,8 +113,9 @@ _IMPLSTB cstatus_t _osslEvpUpdate(const struct ossl_evp_state* state, cspan_t da ncSpanGetSizeC(data) ); break; - + /* Cipher is not supported by this api */ default: + DEBUG_ASSERT2(0, "Called update on an invalid state type"); break; } diff --git a/src/providers/openssl.c b/src/providers/openssl.c index 73a7902..925386d 100644 --- a/src/providers/openssl.c +++ b/src/providers/openssl.c @@ -167,7 +167,6 @@ _osslEvpFree(&evpState); return result; - } #endif /* !_IMPL_CRYPTO_SHA256_HMAC */ @@ -186,7 +185,6 @@ const struct _hkdf_state* state; DEBUG_ASSERT(ctx != NULL); - _overflow_check(data.size); state = (const struct _hkdf_state*)ctx; @@ -231,8 +229,7 @@ handler.update = _ossl_hkdf_update; handler.finish = _ossl_hkdf_finish; - - _overflow_check(prk.size); + _overflow_check(info.size); _overflow_check(okm.size); @@ -336,6 +333,7 @@ /* * Possible static asser that int size must be 32bit or smaller + * so it can be cast safely to uint32 */ if (bytesWritten < 0 || bytesWritten > INT32_MAX) { |