Re: [PATCH 2/2] virStorageEncryptionSecretCopy: Properly copy internals

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux