mbox

[PULL,0/3] crypto: autofree patches

Message ID 20190822105302.26823-1-berrange@redhat.com (mailing list archive)
State New, archived
Headers show

Pull-request

https://github.com/berrange/qemu tags/autofree-pull-request

Message

Daniel P. Berrangé Aug. 22, 2019, 10:52 a.m. UTC 
The following changes since commit fe066b4848bab4f9365a419f3c8ba59ccecf67c0:

  Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-08-21 16:59:22 +0100)

are available in the Git repository at:

  https://github.com/berrange/qemu tags/autofree-pull-request

for you to fetch changes up to 57b9f113fce2a2231a47e9295c1d461e9ff7f0f7:

  crypto: use auto cleanup for many stack variables (2019-08-22 10:56:57 +0100)

----------------------------------------------------------------
require newer glib2 to enable autofree'ing of stack variables exiting scope

* Bump minium glib2 version to 2.48
* Convert much of the crypto code to use automatic memory free functions

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

Daniel P. Berrangé (3):
  glib: bump min required glib library version to 2.48
  crypto: define cleanup functions for use with g_autoptr
  crypto: use auto cleanup for many stack variables

 configure                   |  2 +-
 crypto/afsplit.c            | 28 ++++----------
 crypto/block-luks.c         | 74 +++++++++++--------------------------
 crypto/block.c              | 15 +++-----
 crypto/hmac-glib.c          |  5 ---
 crypto/pbkdf.c              |  5 +--
 crypto/secret.c             | 39 ++++++++-----------
 crypto/tlscredsanon.c       | 16 +++-----
 crypto/tlscredspsk.c        |  5 +--
 crypto/tlscredsx509.c       | 16 +++-----
 include/crypto/block.h      |  2 +
 include/crypto/cipher.h     |  2 +
 include/crypto/hmac.h       |  2 +
 include/crypto/ivgen.h      |  2 +
 include/crypto/tlssession.h |  2 +
 include/glib-compat.h       | 42 +--------------------
 16 files changed, 78 insertions(+), 179 deletions(-)

Comments

Peter Maydell Aug. 23, 2019, 9:41 a.m. UTC | #1 
On Thu, 22 Aug 2019 at 11:54, Daniel P. Berrangé <berrange@redhat.com> wrote:
>
> The following changes since commit fe066b4848bab4f9365a419f3c8ba59ccecf67c0:
>
>   Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-08-21 16:59:22 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/berrange/qemu tags/autofree-pull-request
>
> for you to fetch changes up to 57b9f113fce2a2231a47e9295c1d461e9ff7f0f7:
>
>   crypto: use auto cleanup for many stack variables (2019-08-22 10:56:57 +0100)
>
> ----------------------------------------------------------------
> require newer glib2 to enable autofree'ing of stack variables exiting scope
>
> * Bump minium glib2 version to 2.48
> * Convert much of the crypto code to use automatic memory free functions
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.2
for any user-visible changes.

-- PMM