Refactor qemuDomainSecretAESSetup

[PATCH v2 0/5] Refactor qemuDomainSecretAESSetup

Peter Krempa posted 5 patches 4 years, 1 month ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1584353498.git.pkrempa@redhat.com

src/qemu/qemu_domain.c | 178 ++++++++++++++++++-----------------------
src/qemu/qemu_domain.h |   2 +
2 files changed, 82 insertions(+), 98 deletions(-)

Expand all Fold all

[PATCH v2 0/5] Refactor qemuDomainSecretAESSetup

Posted by Peter Krempa 4 years, 1 month ago

As requested in:

https://www.redhat.com/archives/libvir-list/2020-March/msg00377.html

this is an incremental refactor of qemuDomainSecretAESSetup. Posted only
as an individual refactor to omit re-sending the 26 other patches from
the original series.

Peter Krempa (5):
  qemuDomainSecretInfo: Register autoptr cleanup function
  qemuDomainSecretAESSetup: Automatically free non-secret locals
  qemuDomainSecretAESSetup: Allocate and return 'secinfo' here
  qemuDomainSecretAESSetup: Split out lookup of secret data
  Remove qemuDomainSecretInfoNew

 src/qemu/qemu_domain.c | 178 ++++++++++++++++++-----------------------
 src/qemu/qemu_domain.h |   2 +
 2 files changed, 82 insertions(+), 98 deletions(-)

-- 
2.24.1

Re: [PATCH v2 0/5] Refactor qemuDomainSecretAESSetup

Posted by Ján Tomko 4 years, 1 month ago

On a Monday in 2020, Peter Krempa wrote:
>As requested in:
>
>https://www.redhat.com/archives/libvir-list/2020-March/msg00377.html
>
>this is an incremental refactor of qemuDomainSecretAESSetup. Posted only
>as an individual refactor to omit re-sending the 26 other patches from
>the original series.
>
>Peter Krempa (5):
>  qemuDomainSecretInfo: Register autoptr cleanup function
>  qemuDomainSecretAESSetup: Automatically free non-secret locals
>  qemuDomainSecretAESSetup: Allocate and return 'secinfo' here
>  qemuDomainSecretAESSetup: Split out lookup of secret data
>  Remove qemuDomainSecretInfoNew
>
> src/qemu/qemu_domain.c | 178 ++++++++++++++++++-----------------------
> src/qemu/qemu_domain.h |   2 +
> 2 files changed, 82 insertions(+), 98 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano