various cleanups

[PATCH 0/7] various cleanups

Peter Krempa posted 7 patches 1 year, 3 months ago

Download series mbox

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

src/libxl/libxl_conf.c                     | 17 ++++------
src/storage/storage_backend_iscsi.c        | 17 ++++------
src/storage/storage_backend_iscsi_direct.c | 23 ++++++--------
src/storage/storage_util.c                 | 36 ++++++++++------------
src/util/vircrypto.c                       | 28 ++++++++---------
5 files changed, 49 insertions(+), 72 deletions(-)

Expand all Fold all

[PATCH 0/7] various cleanups

Posted by Peter Krempa 1 year, 3 months ago

Collection of cleanup patches from my previous series attempting to
remove secrets clearing refactored so that it doesn't depend on the
patches which were not accepted.

Peter Krempa (7):
  virCryptoEncryptDataAESgnutls: Restructure control flow
  virStorageBackendISCSISetAuth: Refactor cleanup
  virStorageBackendISCSISetAuth: Use g_strndup to '\0' terminate data
  virStorageBackendISCSIDirectSetAuth: Refactor cleanup
  virStorageBackendISCSIDirectSetAuth: Use 'g_strndup' to '\0' terminate
    data
  libxlMakeNetworkDiskSrc: Refactor cleanup
  storageBackendCreateQemuImgSecretPath: Refactor cleanup

 src/libxl/libxl_conf.c                     | 17 ++++------
 src/storage/storage_backend_iscsi.c        | 17 ++++------
 src/storage/storage_backend_iscsi_direct.c | 23 ++++++--------
 src/storage/storage_util.c                 | 36 ++++++++++------------
 src/util/vircrypto.c                       | 28 ++++++++---------
 5 files changed, 49 insertions(+), 72 deletions(-)

-- 
2.39.1

Re: [PATCH 0/7] various cleanups

Posted by Martin Kletzander 1 year, 3 months ago

On Tue, Jan 31, 2023 at 05:00:48PM +0100, Peter Krempa wrote:
>Collection of cleanup patches from my previous series attempting to
>remove secrets clearing refactored so that it doesn't depend on the
>patches which were not accepted.
>
>Peter Krempa (7):
>  virCryptoEncryptDataAESgnutls: Restructure control flow
>  virStorageBackendISCSISetAuth: Refactor cleanup
>  virStorageBackendISCSISetAuth: Use g_strndup to '\0' terminate data
>  virStorageBackendISCSIDirectSetAuth: Refactor cleanup
>  virStorageBackendISCSIDirectSetAuth: Use 'g_strndup' to '\0' terminate
>    data
>  libxlMakeNetworkDiskSrc: Refactor cleanup
>  storageBackendCreateQemuImgSecretPath: Refactor cleanup

Reviewed-by: Martin Kletzander <mkletzan@redhat.com>

>
> src/libxl/libxl_conf.c                     | 17 ++++------
> src/storage/storage_backend_iscsi.c        | 17 ++++------
> src/storage/storage_backend_iscsi_direct.c | 23 ++++++--------
> src/storage/storage_util.c                 | 36 ++++++++++------------
> src/util/vircrypto.c                       | 28 ++++++++---------
> 5 files changed, 49 insertions(+), 72 deletions(-)
>
>-- 
>2.39.1
>