mbox

[PULL,v1,(for,2.5),0/4] Fix misc memory leaks & bugs in crypto code

Message ID 1447861664-16283-1-git-send-email-berrange@redhat.com
State New
Headers show

Pull-request

git://github.com/berrange/qemu.git tags/qcrypto-fixes-20151118-1

Message

Daniel P. Berrangé Nov. 18, 2015, 3:47 p.m. UTC 
Running test-crypto-tlscredsx509 & test-crypto-tlssession under
valgrind identified 3 memory leaks and one other bug. This short
series fixes them.

The following changes since commit c27e9014d56fa4880e7d741275d887c3a5949997:

  Merge remote-tracking branch 'remotes/kraxel/tags/pull-vnc-20151116-1' into staging (2015-11-17 12:34:07 +0000)

are available in the git repository at:

  git://github.com/berrange/qemu.git tags/qcrypto-fixes-20151118-1

for you to fetch changes up to 08cb175a24d642a40e41db2fef2892b0a1ab504e:

  crypto: avoid passing NULL to access() syscall (2015-11-18 15:42:26 +0000)

----------------------------------------------------------------
Pull qcrypto fixes 2015/11/18 v1

----------------------------------------------------------------

Daniel P. Berrange (4):
  crypto: fix leak of gnutls_dh_params_t data on credential unload
  crypto: fix mistaken setting of Error in success code path
  crypto: fix leaks in TLS x509 helper functions
  crypto: avoid passing NULL to access() syscall

 crypto/tlscredsx509.c           | 7 ++++++-
 crypto/tlssession.c             | 4 ++--
 tests/crypto-tls-x509-helpers.c | 2 ++
 3 files changed, 10 insertions(+), 3 deletions(-)

Comments

Peter Maydell Nov. 18, 2015, 5:06 p.m. UTC | #1 
On 18 November 2015 at 15:47, Daniel P. Berrange <berrange@redhat.com> wrote:
> Running test-crypto-tlscredsx509 & test-crypto-tlssession under
> valgrind identified 3 memory leaks and one other bug. This short
> series fixes them.
>
> The following changes since commit c27e9014d56fa4880e7d741275d887c3a5949997:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-vnc-20151116-1' into staging (2015-11-17 12:34:07 +0000)
>
> are available in the git repository at:
>
>   git://github.com/berrange/qemu.git tags/qcrypto-fixes-20151118-1
>
> for you to fetch changes up to 08cb175a24d642a40e41db2fef2892b0a1ab504e:
>
>   crypto: avoid passing NULL to access() syscall (2015-11-18 15:42:26 +0000)
>
> ----------------------------------------------------------------
> Pull qcrypto fixes 2015/11/18 v1
>
> ----------------------------------------------------------------
>
> Daniel P. Berrange (4):
>   crypto: fix leak of gnutls_dh_params_t data on credential unload
>   crypto: fix mistaken setting of Error in success code path
>   crypto: fix leaks in TLS x509 helper functions
>   crypto: avoid passing NULL to access() syscall
>
>  crypto/tlscredsx509.c           | 7 ++++++-
>  crypto/tlssession.c             | 4 ++--
>  tests/crypto-tls-x509-helpers.c | 2 ++
>  3 files changed, 10 insertions(+), 3 deletions(-)

Applied, thanks.

-- PMM