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.