On 8/3/23 13:29, Claudio Fontana wrote: > On 8/3/23 12:36, Michal Privoznik wrote: >> Inside of securityselinuxhelper we still use malloc() + >> memset(.., 0, ...) combo. Convert it to g_new0(). >> >> Signed-off-by: Michal Privoznik <mpriv...@redhat.com> > > I don't think it is a good idea to mix Glib g_new, g_free etc with malloc, > calloc, free. > > If you go with g_new0 here, imo you need to also change the calls to free() > to g_free. > > The alternative is to use calloc instead, then you could leave the existing > free calls alone.
Ah, good catch! Let me drop this from the series and post a switch to calloc() then. Thanks! Michal
