[Qemu-commits] [qemu/qemu] 0072d2: crypto: fix leak in ivgen essiv init

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0072d2a9fce4835ab2b9ee70aaca0169fb25fa0c
      
https://github.com/qemu/qemu/commit/0072d2a9fce4835ab2b9ee70aaca0169fb25fa0c
  Author: Li Qiang <address@hidden>
  Date:   2017-02-27 (Mon, 27 Feb 2017)

  Changed paths:
    M crypto/ivgen-essiv.c

  Log Message:
  -----------
  crypto: fix leak in ivgen essiv init

On error path, the 'salt' doesn't been freed thus leading
a memory leak. This patch avoid this.

Signed-off-by: Li Qiang <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: 32c813e6c2a857b93b897901b7e20281397528a3
      
https://github.com/qemu/qemu/commit/32c813e6c2a857b93b897901b7e20281397528a3
  Author: Prasad J Pandit <address@hidden>
  Date:   2017-02-27 (Mon, 27 Feb 2017)

  Changed paths:
    M crypto/cipher.c

  Log Message:
  -----------
  crypto: assert cipher algorithm is always valid

Crypto routines 'qcrypto_cipher_get_block_len' and
'qcrypto_cipher_get_key_len' return non-zero cipher block and key
lengths from static arrays 'alg_block_len[]' and 'alg_key_len[]'
respectively. Returning 'zero(0)' value from either of them would
likely lead to an error condition.

Signed-off-by: Prasad J Pandit <address@hidden>
Signed-off-by: Daniel P. Berrange <address@hidden>


  Commit: 8f2d7c341184a95d05476ea3c45dbae2b9ddbe51
      
https://github.com/qemu/qemu/commit/8f2d7c341184a95d05476ea3c45dbae2b9ddbe51
  Author: Peter Maydell <address@hidden>
  Date:   2017-02-27 (Mon, 27 Feb 2017)

  Changed paths:
    M crypto/cipher.c
    M crypto/ivgen-essiv.c

  Log Message:
  -----------
  Merge remote-tracking branch 
'remotes/berrange/tags/pull-qcrypto-2017-02-27-1' into staging

Merge qcrypto 2017/02/27 v1

# gpg: Signature made Mon 27 Feb 2017 13:37:34 GMT
# gpg:                using RSA key 0xBE86EBB415104FDF
# gpg: Good signature from "Daniel P. Berrange <address@hidden>"
# gpg:                 aka "Daniel P. Berrange <address@hidden>"
# Primary key fingerprint: DAF3 A6FD B26B 6291 2D0E  8E3F BE86 EBB4 1510 4FDF

* remotes/berrange/tags/pull-qcrypto-2017-02-27-1:
  crypto: assert cipher algorithm is always valid
  crypto: fix leak in ivgen essiv init

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/3b1d8169844f...8f2d7c341184