676ca2f0f6
bugfix patches list: d677de remote: fix driver name check for libxl driver 075641 systemd: start libvirtd after firewalld iptables services 06fc99 qemuDomainCleanupRun: Actually run cleanup callbacks in reverse order 01626c virDevMapperGetTargetsImpl: quit early if device is not a devmapper target 006782 qemu: only stop external devices after the domain 42a415 qemuDomainStorageSourcePrivateDispose: Free httpcookie cc8c29 Don't require secdrivers to implement .domainMoveImageMetadata 0230e3 qemuProcessStop: Use XATTRs to restore seclabels on disks a domain is mirroring into 8fd274 qemuProcessStop: Reattach NVMe disks a domain is mirroring into 55029d security: don't fail if built without attr support a5a297 qemu: Skip pre creation of NVMe disks 2a372 Fix some wrong usage of ATTRIBUTE_NONNULL e728ff conf: Increase cpuset length limit for CPU pinning 8f58a4 virQEMUDriverConfigNew: Add slash to cfg defaultTLSx509certdir for non-embeded driver 1b22dd qemuDomainSetNumaParamsLive: set nodeset for root cgroup bdb8f2 qemu: do not add model when actual iface type is hostdev Signed-off-by: Jin Yan <jinyan12@huawei.com>
57 lines
2.0 KiB
Diff
57 lines
2.0 KiB
Diff
From 42e4b74e1bbd08aad3afa46d741e46c40a8af73d Mon Sep 17 00:00:00 2001
|
|
From: Bihong Yu <yubihong@huawei.com>
|
|
Date: Sat, 6 Jun 2020 18:52:35 +0800
|
|
Subject: [PATCH] Fix some wrong usage of ATTRIBUTE_NONNULL()
|
|
|
|
The virStateInitialize() function has ATTRIBUTE_NONNULL()
|
|
referring to @root argument (incorrectly anyway) but in
|
|
daemonRunStateInit() NULL is passed in anyway.
|
|
|
|
Then there is virCommandAddArgPair() which also has
|
|
ATTRIBUTE_NONNULL() for one of its arguments and then checks the
|
|
argument for being NULL anyways.
|
|
|
|
cherry-pick from commit: 2a372a5ad5fab3bf26fb9bea019d38fa04ba8b34
|
|
|
|
Signed-off-by:Bihong Yu <yubihong@huawei.com>
|
|
Reviewed-by:Chuan Zheng <zhengchuan@huawei.com>
|
|
Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
|
|
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
|
|
Signed-off-by: Jin Yan <jinyan12@huawei.com>
|
|
---
|
|
src/libvirt_internal.h | 3 +--
|
|
src/util/vircommand.h | 3 +--
|
|
2 files changed, 2 insertions(+), 4 deletions(-)
|
|
|
|
diff --git a/src/libvirt_internal.h b/src/libvirt_internal.h
|
|
index 00ef7aa..72c6127 100644
|
|
--- a/src/libvirt_internal.h
|
|
+++ b/src/libvirt_internal.h
|
|
@@ -33,8 +33,7 @@ int virStateInitialize(bool privileged,
|
|
bool mandatory,
|
|
const char *root,
|
|
virStateInhibitCallback inhibit,
|
|
- void *opaque)
|
|
- ATTRIBUTE_NONNULL(2);
|
|
+ void *opaque);
|
|
int virStateCleanup(void);
|
|
int virStateReload(void);
|
|
int virStateStop(void);
|
|
diff --git a/src/util/vircommand.h b/src/util/vircommand.h
|
|
index 9086f9a..4e6cb0a 100644
|
|
--- a/src/util/vircommand.h
|
|
+++ b/src/util/vircommand.h
|
|
@@ -128,8 +128,7 @@ void virCommandAddArgFormat(virCommandPtr cmd,
|
|
|
|
void virCommandAddArgPair(virCommandPtr cmd,
|
|
const char *name,
|
|
- const char *val)
|
|
- ATTRIBUTE_NONNULL(2) ATTRIBUTE_NONNULL(3);
|
|
+ const char *val);
|
|
|
|
void virCommandAddArgSet(virCommandPtr cmd,
|
|
const char *const*vals) ATTRIBUTE_NONNULL(2);
|
|
--
|
|
1.8.3.1
|
|
|