On a Sunday in 2020, Laine Stump wrote:
There are several calls to virBufferFreeAndReset() when functions
encounter an error, but the caller never uses the virBuffer once an
error has been encountered (all callers detect error by looking at the
function return value, not the contents of the virBuffer being
operated on), and now that all virBuffers are auto-freed there is no
reason for the lower level functions like these to spend time freeing
a buffer that is guaranteed to be freed momentarily anyway.
Signed-off-by: Laine Stump <laine(a)redhat.com>
---
src/conf/checkpoint_conf.c | 1 -
src/conf/domain_conf.c | 1 -
src/conf/snapshot_conf.c | 1 -
src/libxl/xen_xl.c | 1 -
src/util/virsysinfo.c | 1 -
src/util/virxml.c | 1 -
6 files changed, 6 deletions(-)
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano