Eugene Syromiatnikov
8e28f5c0b9
Move the open-coded aligned alloc in a separate file and use it in FIPS module
...
Factor the open-coded aligned allocation implementation in a separate
file and use it instead of just returning NULL in CRYPTO_aligned_alloc
implementation in the FIPS provider.
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Neil Horman <nhorman@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28295 )
2025-09-16 09:59:13 -04:00
Eugene Syromiatnikov
89f1f9bd73
crypto/mem.c: check for overflow in size calculation in CRYPTO_aligned_alloc
...
The open-coded implementation performs addition of size and alignment,
that may overflow.
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
Reviewed-by: Matt Caswell <matt@openssl.org >
Reviewed-by: Paul Dale <ppzgs1@gmail.com >
Reviewed-by: Neil Horman <nhorman@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28059 )
2025-08-08 12:22:10 -04:00
Eugene Syromiatnikov
1104e80c8d
crypto/mem.c: check the alignment for being a power of 2 in CRYPTO_aligned_alloc
...
Otherwise the roundup calculation performed in the open-coded implementation
may put the pointer out of bounds.
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
Reviewed-by: Matt Caswell <matt@openssl.org >
Reviewed-by: Paul Dale <ppzgs1@gmail.com >
Reviewed-by: Neil Horman <nhorman@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28059 )
2025-08-08 12:22:10 -04:00
Eugene Syromiatnikov
731fc62908
crypto/params_dup.c: add overflow check to ossl_param_buf_alloc
...
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
Reviewed-by: Matt Caswell <matt@openssl.org >
Reviewed-by: Paul Dale <ppzgs1@gmail.com >
Reviewed-by: Neil Horman <nhorman@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28059 )
2025-08-08 12:22:10 -04:00
Eugene Syromiatnikov
fa9b7b930e
Add array memory allocation routines
...
Such routines allow alleviating the need to perform explicit integer
overflow check during allocation size calculation and generally make
the allocations more semantic (as they signify that a collection
of NUM items, each occupying SIZE bytes is being allocated), which paves
the road for additional correctness checks in the future.
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
Reviewed-by: Matt Caswell <matt@openssl.org >
Reviewed-by: Paul Dale <ppzgs1@gmail.com >
Reviewed-by: Neil Horman <nhorman@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28059 )
2025-08-08 12:22:10 -04:00
Eugene Syromiatnikov
bd1c59739d
crypto/mem.c: factor out memory allocation failure reporting
...
Signed-off-by: Eugene Syromiatnikov <esyr@openssl.org >
Reviewed-by: Saša Nedvědický <sashan@openssl.org >
Reviewed-by: Matt Caswell <matt@openssl.org >
Reviewed-by: Paul Dale <ppzgs1@gmail.com >
Reviewed-by: Neil Horman <nhorman@openssl.org >
(Merged from https://github.com/openssl/openssl/pull/28059 )
2025-08-08 12:22:10 -04:00
Eugene Syromiatnikov