[PATCH 0/4] Appease Clang

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



With its recent update Clang-13.0.1-rc1 started to report spurious
warnings. Basically it is concerned about our use of g_auto*:

  g_autofree char *var = NULL;

  var = func();

It fails to see that @var is there to automatically free retval of
func(). I've reported the bug here:

  https://bugs.llvm.org/show_bug.cgi?id=36627

But there are few places where we can change the code and appease Clang.
The rest, where our code is correct - I'm "fixing" those places in patch
4/4 which exists only to show that after the problematic pattern is
broken the build passes again:

  https://gitlab.com/MichalPrivoznik/libvirt/-/jobs/1532513684

Michal Prívozník (4):
  virpci: Avoid Clang false positive
  test: Drop unused @cfg from qemu*test
  virscsi: Drop @tmp from virSCSIDeviceListDel
  DO NOT MERGE

 src/rpc/virnetclient.c             | 9 ++++++---
 src/util/viridentity.c             | 3 ++-
 src/util/virpci.c                  | 4 ++--
 src/util/virscsi.c                 | 3 +--
 tests/qemumigrationcookiexmltest.c | 2 --
 tests/qemustatusxml2xmltest.c      | 2 --
 6 files changed, 11 insertions(+), 12 deletions(-)

-- 
2.31.1




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux