On a Tuesday in 2020, Michal Privoznik wrote:
On 12/1/20 2:50 AM, Tuguoyi wrote:-----Original Message----- From: Ján Tomko [mailto:[email protected]] Sent: Tuesday, November 24, 2020 6:57 PM To: tuguoyi (Cloud) <[email protected]> Cc: [email protected] Subject: Re: [PATCH] qemu_conf: Fix double free problem for cfg->firmwaresOn a Tuesday in 2020, Tuguoyi wrote:cfg->firmwares still points to the original memory address after being freed by virFirmwareFreeList(). As cfg get freed, it will be freed again even if cfg->nfirmwares=0 which eventually lead to crash. The patch fix it by setting cfg->firmwares to NULL explicitly after virFirmwareFreeList() returns Signed-off-by: Tuguoyi <[email protected]>Should there be a space separating your name(s)?--- src/qemu/qemu_conf.c | 1 + 1 file changed, 1 insertion(+)Reviewed-by: Ján Tomko <[email protected]> JanoHi there, It's my first time to submit patch to libvirt, so I'm wondering will this patch be applied to the upstream?Oh yeah, sorry. I've pushed it now:
Thank you, Jano
https://gitlab.com/libvirt/libvirt/-/commit/c4f4e195a14c86b7daff2c45f1cbfd23ac16aaa8 Congratulations on your first libvirt contribution! Michal
signature.asc
Description: PGP signature
