On Mon, Oct 05, 2020 at 12:22:26AM +0200, Ján Tomko wrote: > Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> > --- ... > > for (i = 0; i < sharedFolders.count; i++) { > ISharedFolder *sharedFolder = sharedFolders.items[i]; > @@ -3798,10 +3789,8 @@ vboxDumpAudio(virDomainDefPtr def, vboxDriverPtr data G_GNUC_UNUSED, > PRUint32 audioController = AudioControllerType_AC97; > > def->nsounds = 1; > - if (VIR_ALLOC_N(def->sounds, def->nsounds) < 0) > - return; > - if (VIR_ALLOC(def->sounds[0]) < 0) > - return; > + def->sounds = g_new0(virDomainSoundDefPtr, def->nsounds); just use 1 for the counter and drop the nsounds attribute initialization completely. Reviewed-by: Erik Skultety <eskultet@xxxxxxxxxx>