Re: [PATCH 2/2] remote: properly initialize objects in ACL helpers

FYI: Gmail decided that your whole series is spam. I'm not sure whether it's just gmail's spam filter being silly or your mail infra has something wrong, but just so you know. On Sun, Mar 17, 2024 at 18:08:50 +0100, Denis V. Lunev wrote: > Commit 2ecdf259299813c2c674377e22a0acbce5ccbbb2 is idealogically corrent, > but unfortunately is incomplete. There are other similar objects in the > module which are used also without proper initialization. The commit you mention was justifying the change from stack-allocated to heap allocated in order to decrease the stack frame size, which was too big due to the fact that 'virDomainDef' is too big. With the other structs that isn't that much of a problem, so I don't think the justification of "Commit 2ecdf259299813 being incomplete" is relevant and thus please add your own justification or adapt what I wrote there as a separate justification, but IMO commit 2ecdf259299813 is not incomplete based on what it tried to do.