On a Tuesday in 2020, Ján Tomko wrote: > On 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 <tu.guoyi@xxxxxxx> > > Should there be a space separating your name(s)? It can be changed to: Signed-off-by: Guoyi Tu<tu.guoyi@xxxxxxx>