[Qemu-devel] [PULL 0/3] crypto: autofree patches

Daniel P. Berrangé posted 3 patches 4 years, 8 months ago
Test docker-clang@ubuntu passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190822105302.26823-1-berrange@redhat.com
Maintainers: "Daniel P. Berrangé" <berrange@redhat.com>
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(-)
[Qemu-devel] [PULL 0/3] crypto: autofree patches
Posted by Daniel P. Berrangé 4 years, 8 months ago
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(-)

-- 
2.21.0


Re: [Qemu-devel] [PULL 0/3] crypto: autofree patches
Posted by Peter Maydell 4 years, 8 months ago
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