[libvirt PATCH 0/2] fixes allocation issues leading to crashes

Ján Tomko jtomko at redhat.com
Tue Oct 13 09:37:32 UTC 2020


On a Monday in 2020, Pavel Hrdina wrote:
>Our libvirt-dbus tests discovered allocation issues introduced by recent
>rewrite to use g_new0 instead of VIR_ALLOC_N by allocating array one
>element shorter in some places which can lead to crashes of client
>applications counting on a fact that the returned arrays are NULL
>terminated.
>

Oops, I did not notice that my vim macro [0] dropped these (due to looking
for the ')' from the left side).
There seem to be no other cases like this in my changes where the
expression contains a parenthesis.

Thanks for fixing this.

Jano

>Pavel Hrdina (2):
>  conf: fix g_new0 allocation
>  conf: virsecretobj: fix g_new0 allocation
>
> src/conf/virinterfaceobj.c  | 2 +-
> src/conf/virnetworkobj.c    | 4 ++--
> src/conf/virnodedeviceobj.c | 2 +-
> src/conf/virsecretobj.c     | 2 +-
> 4 files changed, 5 insertions(+), 5 deletions(-)
>

[0] I was not able to replace everything with Coccinelle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20201013/23350075/attachment-0001.sig>


More information about the libvir-list mailing list