Coverity Scan: Analysis completed for openssl/openssl

2024-04-29 Thread scan-admin


Your request for analysis of openssl/openssl has been completed 
successfully.
The results are available at 
https://u15810271.ct.sendgrid.net/ls/click?upn=u001.AxU2LYlgjL6eX23u9ErQy-2BKADyCpvUKOL6EWmZljiu7On3ENqKj59w4rEawU-2BYZgjBJ8Ay1pFYeVslaCebcWhw-3D-3DloXE_bGLyfjQQhnPAMp6Vz8xksML07NHC5l7WMnVZN7GWb6ogw0DJbMvtp5-2Fd-2BxspSO3Z0mSkMveDALUZviu-2F3S6GnP3fH-2BF4W9Oo8xxwx0XGWipj456xOFqJN8byJaXzHddLj6CHGtxmDDMUtZlbwVEpCOkNCXFEiyTPJSJTre3O8-2Fwghc040GndUE4AsR9Lv26uXfDQJIPlK-2Fg5-2F-2FHF94zRH1ONzVn6e1k5zOR7KTf1gAE-3D

Build ID: 609534

Analysis Summary:
   New defects found: 0
   Defects eliminated: 0



[openssl/openssl] 6de46a: doc: clarify SSL_CIPHER_description allocation

2024-04-29 Thread Daniel McCarney
  Branch: refs/heads/openssl-3.3
  Home:   https://github.com/openssl/openssl
  Commit: 6de46a0896df368d99a86223f6e9b35e0c95464b
  
https://github.com/openssl/openssl/commit/6de46a0896df368d99a86223f6e9b35e0c95464b
  Author: Daniel McCarney 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M doc/man3/SSL_CIPHER_get_name.pod

  Log Message:
  ---
  doc: clarify SSL_CIPHER_description allocation

Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Paul Dale 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/23921)

(cherry picked from commit 6a4a714045415be6720f4165c4d70a0ff229a26a)



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


[openssl/openssl] 364b67: doc: clarify SSL_CIPHER_description allocation

2024-04-29 Thread Daniel McCarney
  Branch: refs/heads/openssl-3.1
  Home:   https://github.com/openssl/openssl
  Commit: 364b67017440d22c526cb84c8f1701e403e65e69
  
https://github.com/openssl/openssl/commit/364b67017440d22c526cb84c8f1701e403e65e69
  Author: Daniel McCarney 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M doc/man3/SSL_CIPHER_get_name.pod

  Log Message:
  ---
  doc: clarify SSL_CIPHER_description allocation

Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Paul Dale 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/23921)

(cherry picked from commit 6a4a714045415be6720f4165c4d70a0ff229a26a)



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


[openssl/openssl] 361c6e: doc: clarify SSL_CIPHER_description allocation

2024-04-29 Thread Daniel McCarney
  Branch: refs/heads/openssl-3.2
  Home:   https://github.com/openssl/openssl
  Commit: 361c6ee5f55f73f530eadd6a2a20e98bf2964f3e
  
https://github.com/openssl/openssl/commit/361c6ee5f55f73f530eadd6a2a20e98bf2964f3e
  Author: Daniel McCarney 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M doc/man3/SSL_CIPHER_get_name.pod

  Log Message:
  ---
  doc: clarify SSL_CIPHER_description allocation

Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Paul Dale 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/23921)

(cherry picked from commit 6a4a714045415be6720f4165c4d70a0ff229a26a)



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


[openssl/openssl] d76fc9: doc: clarify SSL_CIPHER_description allocation

2024-04-29 Thread Daniel McCarney
  Branch: refs/heads/openssl-3.0
  Home:   https://github.com/openssl/openssl
  Commit: d76fc993a9a83ce251040f3fe30f5f38a980b115
  
https://github.com/openssl/openssl/commit/d76fc993a9a83ce251040f3fe30f5f38a980b115
  Author: Daniel McCarney 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M doc/man3/SSL_CIPHER_get_name.pod

  Log Message:
  ---
  doc: clarify SSL_CIPHER_description allocation

Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Paul Dale 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/23921)

(cherry picked from commit 6a4a714045415be6720f4165c4d70a0ff229a26a)



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


[openssl/openssl] 6a4a71: doc: clarify SSL_CIPHER_description allocation

2024-04-29 Thread Daniel McCarney
  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 6a4a714045415be6720f4165c4d70a0ff229a26a
  
https://github.com/openssl/openssl/commit/6a4a714045415be6720f4165c4d70a0ff229a26a
  Author: Daniel McCarney 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M doc/man3/SSL_CIPHER_get_name.pod

  Log Message:
  ---
  doc: clarify SSL_CIPHER_description allocation

Previously the documentation for `SSL_CIPHER_description` said:
> If buf is provided, it must be at least 128 bytes, otherwise a buffer
> will be allocated using OPENSSL_malloc().

In reality, `OPENSSL_malloc` is only invoked if the provided `buf`
argument is `NULL`. If the `buf` arg is not `NULL`, but smaller than
128 bytes, the function returns `NULL` without attempting to allocate
a new buffer for the description.

This commit adjusts the documentation to better describe the implemented
behaviour.

CLA: trivial

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Paul Dale 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/23921)



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


[openssl/openssl] d0b039: archive artifacts before upload

2024-04-29 Thread Dmitry Misharov
  Branch: refs/heads/openssl-3.3
  Home:   https://github.com/openssl/openssl
  Commit: d0b039a5ef97cc360e2008035fe66dc353724ff6
  
https://github.com/openssl/openssl/commit/d0b039a5ef97cc360e2008035fe66dc353724ff6
  Author: Dmitry Misharov 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M .github/workflows/ci.yml
M .github/workflows/cross-compiles.yml
M .github/workflows/make-test

  Log Message:
  ---
  archive artifacts before upload

Some CI jobs produce a significant amount artifacts and it takes a lot
of time to upload them into GitHub artifacts storage. It will be much
faster to upload only one archive with artifacts.

Reviewed-by: Neil Horman 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/24264)

(cherry picked from commit 58ffcbbdc3302a35cea317aeee6b76987907ee60)



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


[openssl/openssl] 58ffcb: archive artifacts before upload

2024-04-29 Thread Dmitry Misharov
  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 58ffcbbdc3302a35cea317aeee6b76987907ee60
  
https://github.com/openssl/openssl/commit/58ffcbbdc3302a35cea317aeee6b76987907ee60
  Author: Dmitry Misharov 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M .github/workflows/ci.yml
M .github/workflows/cross-compiles.yml
M .github/workflows/make-test

  Log Message:
  ---
  archive artifacts before upload

Some CI jobs produce a significant amount artifacts and it takes a lot
of time to upload them into GitHub artifacts storage. It will be much
faster to upload only one archive with artifacts.

Reviewed-by: Neil Horman 
Reviewed-by: Tom Cosgrove 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/24264)



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


[openssl/openssl] 8a5a6a: updated to oqs-provider 0.6.0

2024-04-29 Thread Michael Baentsch
  Branch: refs/heads/openssl-3.3
  Home:   https://github.com/openssl/openssl
  Commit: 8a5a6a15e2338ccde91d5970f64f3d66ae5d18bd
  
https://github.com/openssl/openssl/commit/8a5a6a15e2338ccde91d5970f64f3d66ae5d18bd
  Author: Michael Baentsch <57787676+baent...@users.noreply.github.com>
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M oqs-provider
M test/recipes/95-test_external_oqsprovider_data/oqsprovider.sh

  Log Message:
  ---
  updated to oqs-provider 0.6.0

Reviewed-by: Dmitry Belyavskiy 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/24261)

(cherry picked from commit 7b1829fa37922a37ef9259fc1bc4038829e4fd73)



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


[openssl/openssl] 55f390: updated to oqs-provider 0.6.0

2024-04-29 Thread Michael Baentsch
  Branch: refs/heads/openssl-3.2
  Home:   https://github.com/openssl/openssl
  Commit: 55f390c79f8c4c2225149588cd9f9e63bfb28909
  
https://github.com/openssl/openssl/commit/55f390c79f8c4c2225149588cd9f9e63bfb28909
  Author: Michael Baentsch <57787676+baent...@users.noreply.github.com>
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M oqs-provider
M test/recipes/95-test_external_oqsprovider_data/oqsprovider.sh

  Log Message:
  ---
  updated to oqs-provider 0.6.0

Reviewed-by: Dmitry Belyavskiy 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/24261)

(cherry picked from commit 7b1829fa37922a37ef9259fc1bc4038829e4fd73)



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


[openssl/openssl] 7b1829: updated to oqs-provider 0.6.0

2024-04-29 Thread Michael Baentsch
  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 7b1829fa37922a37ef9259fc1bc4038829e4fd73
  
https://github.com/openssl/openssl/commit/7b1829fa37922a37ef9259fc1bc4038829e4fd73
  Author: Michael Baentsch <57787676+baent...@users.noreply.github.com>
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M oqs-provider
M test/recipes/95-test_external_oqsprovider_data/oqsprovider.sh

  Log Message:
  ---
  updated to oqs-provider 0.6.0

Reviewed-by: Dmitry Belyavskiy 
Reviewed-by: Tomas Mraz 
(Merged from https://github.com/openssl/openssl/pull/24261)



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


[openssl/openssl] 705441: 82-test_ocsp_cert_chain.t: Terminate the server af...

2024-04-29 Thread Tomáš Mráz
  Branch: refs/heads/master
  Home:   https://github.com/openssl/openssl
  Commit: 7054412ea8bb49d9522c3dd99982e41bf08c3ef7
  
https://github.com/openssl/openssl/commit/7054412ea8bb49d9522c3dd99982e41bf08c3ef7
  Author: Tomas Mraz 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M test/recipes/82-test_ocsp_cert_chain.t

  Log Message:
  ---
  82-test_ocsp_cert_chain.t: Terminate the server after 1 connection

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
(Merged from https://github.com/openssl/openssl/pull/23857)


  Commit: f4fcc21fdccfde90bda6f8a94d7f4e07f947e38f
  
https://github.com/openssl/openssl/commit/f4fcc21fdccfde90bda6f8a94d7f4e07f947e38f
  Author: Tomas Mraz 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M test/recipes/82-test_ocsp_cert_chain.t

  Log Message:
  ---
  82-test_ocsp_cert_chain.t: kill -HUP the server after client quits

This ensures even if the connection for some reason
fails, the server will terminate and the test won't get
stuck.

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
(Merged from https://github.com/openssl/openssl/pull/23857)


Compare: https://github.com/openssl/openssl/compare/933f57dfe216...f4fcc21fdccf

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


[openssl/openssl] 0f438e: 82-test_ocsp_cert_chain.t: Terminate the server af...

2024-04-29 Thread Tomáš Mráz
  Branch: refs/heads/openssl-3.3
  Home:   https://github.com/openssl/openssl
  Commit: 0f438e9158d2029d763e469034377a3d76201c8f
  
https://github.com/openssl/openssl/commit/0f438e9158d2029d763e469034377a3d76201c8f
  Author: Tomas Mraz 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M test/recipes/82-test_ocsp_cert_chain.t

  Log Message:
  ---
  82-test_ocsp_cert_chain.t: Terminate the server after 1 connection

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
(Merged from https://github.com/openssl/openssl/pull/23857)

(cherry picked from commit 7054412ea8bb49d9522c3dd99982e41bf08c3ef7)


  Commit: 6bd3d18330c7bd2fe6d97aa538605a95c96b57ad
  
https://github.com/openssl/openssl/commit/6bd3d18330c7bd2fe6d97aa538605a95c96b57ad
  Author: Tomas Mraz 
  Date:   2024-04-29 (Mon, 29 Apr 2024)

  Changed paths:
M test/recipes/82-test_ocsp_cert_chain.t

  Log Message:
  ---
  82-test_ocsp_cert_chain.t: kill -HUP the server after client quits

This ensures even if the connection for some reason
fails, the server will terminate and the test won't get
stuck.

Reviewed-by: Matt Caswell 
Reviewed-by: Tom Cosgrove 
(Merged from https://github.com/openssl/openssl/pull/23857)

(cherry picked from commit f4fcc21fdccfde90bda6f8a94d7f4e07f947e38f)


Compare: https://github.com/openssl/openssl/compare/b4604d525263...6bd3d18330c7

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