Re: [PATCH 2/2] virStorageEncryptionSecretCopy: Properly copy internals
On Thu, Mar 19, 2020 at 18:14:15 +0100, Ján Tomko wrote:
On a Thursday in 2020, Peter Krempa wrote:
> virStorageEncryptionSecretPtr may have a string inside it, thus we must
> copy the string too. Use virSecretLookupDefCopy to do that.
>
> Likely caused by 756b46ddd24.
Please remove the period from the end.
Also, at the times of that commit, memcpy was sufficient:
struct _virStorageEncryptionSecret {
int type; /* virStorageEncryptionSecretType */
unsigned char uuid[VIR_UUID_BUFLEN];
};
So this actually fixes commit 47e88b33b which switched to using
virSecretLookupDef.
Oh, okay, I didn't look as closely into the history. I'll update the
problematic commit.