On Tue, Jul 13, 2021 at 2:42 PM Stefan Berger
wrote:
>
> Allow swtpm (0.7.0 or later) to fsync on the directory where it writes
> its state files into so that "the entry in the directory containing the
> file has also reached disk" (fsync(2)).
>
> Signed-off-by: Stefan Berger
> ---
> src/securit
Allow swtpm (0.7.0 or later) to fsync on the directory where it writes
its state files into so that "the entry in the directory containing the
file has also reached disk" (fsync(2)).
Signed-off-by: Stefan Berger
---
src/security/virt-aa-helper.c | 5 -
1 file changed, 4 insertions(+), 1 dele
Signed-off-by: Tim Wiederhake
---
src/qemu/qemu_monitor.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 9b1a3ec3eb..0b5da8b71f 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -4214,7 +4214,7 @@ i
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
tests/nwfilterxml2firewalltest.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/tests/nwfilterxml2firewalltest.c b/tests/nwfilterxml2firewalltest.c
index 6709cc15fd..3b7190b5cd 100644
--- a/tests/nwfilterxml2firewalltest.c
+++ b/t
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
src/util/viriptables.c | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/util/viriptables.c b/src/util/viriptables.c
index 847af9b9d7..721e1eeae7 100644
--- a/src/util/viriptables.c
+++ b/src/util/viriptab
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
src/qemu/qemu_monitor.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index 0b5da8b71f..f66a3457c1 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_moni
Signed-off-by: Tim Wiederhake
---
src/qemu/qemu_monitor.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
index a2df1a6ec3..9b1a3ec3eb 100644
--- a/src/qemu/qemu_monitor.c
+++ b/src/qemu/qemu_monitor.c
@@ -4214,6 +42
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
src/util/viriptables.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/util/viriptables.c b/src/util/viriptables.c
index 198ece3d71..847af9b9d7 100644
--- a/src/util/viriptables.c
+++ b/src/util/viriptables
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
tests/nwfilterxml2firewalltest.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/tests/nwfilterxml2firewalltest.c b/tests/nwfilterxml2firewalltest.c
index 26d4a936ad..bdfe858185 100644
--- a/tests/nwfilterxml2
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
tests/nwfilterxml2firewalltest.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/tests/nwfilterxml2firewalltest.c b/tests/nwfilterxml2firewalltest.c
index 3b7190b5cd..26d4a936ad 100644
--- a/tests/nwfilterxml2fi
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
src/util/viriptables.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/src/util/viriptables.c b/src/util/viriptables.c
index 4189578245..198ece3d71 100644
--- a/src/util/viriptables.c
+++ b/src/util/viriptab
Signed-off-by: Tim Wiederhake
Reviewed-by: Peter Krempa
---
tests/nwfilterxml2firewalltest.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/nwfilterxml2firewalltest.c b/tests/nwfilterxml2firewalltest.c
index bdfe858185..75a70e9972 100644
--- a/tests/nwfilterxml2fir
"virHashNew" cannot return NULL, yet we check for NULL in various places.
V1: https://listman.redhat.com/archives/libvir-list/2021-July/msg00188.html
Changes since V1:
* Moved the inversion of the `if` condition in patch #9 to patch #8
* Only patches #8 and #9 are missing review
Tim Wiederhake (
On libvirtd startup, the list of priority worker threads is uninitialized
(`pool->prioWorkers` is NULL), and then "expanded" to zero (`prioWorkers`)
entries.
This causes `virThreadPoolExpand` to call `VIR_EXPAND_N` on a null pointer
and an increment of zero. The zero increment triggers `virRealloc
V1: https://listman.redhat.com/archives/libvir-list/2021-July/msg00217.html
Changes since V1:
* Fix this not in virThreadPoolExpand, but in virThreadPoolNewFull.
* Expanded the commit message.
Tim Wiederhake (1):
virThreadPoolNewFull: Prevent expanding worker pool by zero
src/util/virthreadpo
Signed-off-by: Tim Wiederhake
---
src/conf/domain_nwfilter.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/conf/domain_nwfilter.c b/src/conf/domain_nwfilter.c
index cb35221a59..0a67b6765e 100644
--- a/src/conf/domain_nwfilter.c
+++ b/src/conf/domain_nwfilter.c
@@ -61
Signed-off-by: Tim Wiederhake
---
src/conf/domain_nwfilter.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/conf/domain_nwfilter.c b/src/conf/domain_nwfilter.c
index 1c62353fa1..5024d5fa03 100644
--- a/src/conf/domain_nwfilter.c
+++ b/src/conf/domain_nwfilter.c
@@ -59,8
Signed-off-by: Tim Wiederhake
---
src/nwfilter/nwfilter_dhcpsnoop.c | 17 -
1 file changed, 17 deletions(-)
diff --git a/src/nwfilter/nwfilter_dhcpsnoop.c
b/src/nwfilter/nwfilter_dhcpsnoop.c
index 2481ea5371..4b62a7b661 100644
--- a/src/nwfilter/nwfilter_dhcpsnoop.c
+++ b/src/nw
Signed-off-by: Tim Wiederhake
---
src/conf/virnwfilterbindingobjlist.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/conf/virnwfilterbindingobjlist.c
b/src/conf/virnwfilterbindingobjlist.c
index 1f19e27eb0..470a30ca90 100644
--- a/src/conf/virnwfilterbindingobjlist.
Signed-off-by: Tim Wiederhake
---
src/nwfilter/nwfilter_gentech_driver.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/nwfilter/nwfilter_gentech_driver.c
b/src/nwfilter/nwfilter_gentech_driver.c
index 8aa1db23d3..da4f71daf1 100644
--- a/src/nwfilter/nwfilter_gentech_d
Signed-off-by: Tim Wiederhake
---
src/conf/virnwfilterbindingdef.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/conf/virnwfilterbindingdef.c b/src/conf/virnwfilterbindingdef.c
index 98df1f750a..9704e1bebb 100644
--- a/src/conf/virnwfilterbindingdef.c
+++ b/src/conf/vi
Signed-off-by: Tim Wiederhake
---
src/conf/virnwfilterbindingdef.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/conf/virnwfilterbindingdef.c b/src/conf/virnwfilterbindingdef.c
index 4ed3763efd..22ecf7b828 100644
--- a/src/conf/virnwfilterbindingdef.c
+++ b/src/conf/
Signed-off-by: Tim Wiederhake
---
src/conf/virnwfilterbindingdef.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/conf/virnwfilterbindingdef.h b/src/conf/virnwfilterbindingdef.h
index 0e52789332..68d531b75d 100644
--- a/src/conf/virnwfilterbindingdef.h
+++ b/src/conf/virnwfilterbinding
Signed-off-by: Tim Wiederhake
---
src/conf/virnwfilterbindingdef.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/conf/virnwfilterbindingdef.c b/src/conf/virnwfilterbindingdef.c
index 9704e1bebb..4ed3763efd 100644
--- a/src/conf/virnwfilterbindingdef.c
+++ b/src/co
Signed-off-by: Tim Wiederhake
---
src/conf/domain_nwfilter.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/conf/domain_nwfilter.c b/src/conf/domain_nwfilter.c
index 5024d5fa03..cb35221a59 100644
--- a/src/conf/domain_nwfilter.c
+++ b/src/conf/domain_nwfilter.c
@@
"virHashNew" cannot return NULL, yet we check for NULL in various places.
See https://listman.redhat.com/archives/libvir-list/2021-July/msg00074.html.
Tim Wiederhake (10):
conf: Add AUTOPTR_CLEANUP_FUNC for virNWFilterBindingDef
virNWFilterBindingDefCopy: `virHashNew` cannot return NULL
vir
This is a followup for commit e906c4d02bdcddf141b4d124afd68c8ee10134fe
("apparmor: Allow /usr/libexec for libxl-save-helper and pygrub"):
In recent rpm versions --libexecdir changed from /usr/lib64 to
/usr/libexec. A plain rpmbuild %configure in xen.git will install all
files, including the privat
On 7/13/21 9:36 AM, Michal Prívozník wrote:
> On 7/6/21 2:37 PM, Tim Wiederhake wrote:
>> "virHashNew" cannot return NULL, yet we check for NULL in various places.
>>
>> This series is the first of several that remove these checks. Where
>> applicable, the functions are refactored to use automatic
As test driver won't have real background job running, in order to get
all possible states, the time is used here to decide which state to be
returned. The default time will get `ok` as return value.
Note that using `virsh domtime fc4 200` won't take effect for the test
driver, to get other states
On 7/13/21 11:11 AM, Peter Krempa wrote:
> On Tue, Jul 13, 2021 at 08:44:06 +0200, Michal Privoznik wrote:
>> We document that our coding style is to use postfix increment in
>> for() loops rather than prefix. This change was generated by the
>> following sed script:
>>
>> for i in $(git grep -l
On 7/13/21 11:25 AM, Peter Krempa wrote:
> On Tue, Jul 13, 2021 at 08:57:30 +0200, Michal Privoznik wrote:
>> In not so distant past (v6.5.0~3) I've updated the private key we
>> use for virnettls* tests. Back then I was driven by Fedora 33
>> change which deprecated RSA-1024 which we used back the
Hi all,
I have recently started a virt-tuner tool using the python3 binding.
The goal of this tool is to help users tune their domains using a few
templates.
https://github.com/SUSE/virt-tuner
As of today I only wrote a template for a giant VM taking almost all
the host resources and mapping the
On Mon, Jul 12, 2021 at 11:34:07 +0200, Tim Wiederhake wrote:
> "virHashNew" cannot return NULL, yet we check for NULL in various places.
>
> See https://listman.redhat.com/archives/libvir-list/2021-July/msg00074.html.
>
Series:
Reviewed-by: Peter Krempa
The function name in the subject is unfortunately ambiguous. Please
prepend it with the test file name or something.
On Mon, Jul 12, 2021 at 11:34:08 +0200, Tim Wiederhake wrote:
> Signed-off-by: Tim Wiederhake
> ---
> tests/nwfilterxml2firewalltest.c | 3 ---
> 1 file changed, 3 deletions(-)
On Fri, Jul 09, 2021 at 10:27:29 +0200, Tim Wiederhake wrote:
> "virHashNew" cannot return NULL, yet we check for NULL in various places.
>
> See https://listman.redhat.com/archives/libvir-list/2021-July/msg00074.html.
>
> Tim Wiederhake (10):
> virNWFilterCreateVarsFrom: `virHashNew` cannot re
On Fri, Jul 09, 2021 at 10:27:38 +0200, Tim Wiederhake wrote:
> Signed-off-by: Tim Wiederhake
> ---
> src/qemu/qemu_monitor.c | 7 +++
> 1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
> index 8646efe9c4..3a56ed8ef9 100644
>
On Tue, 2021-07-13 at 09:36 +0200, Michal Prívozník wrote:
> On 7/6/21 2:37 PM, Tim Wiederhake wrote:
> > Signed-off-by: Tim Wiederhake
> > ---
> > src/qemu/qemu_monitor.c | 16 ++--
> > 1 file changed, 6 insertions(+), 10 deletions(-)
> >
> > diff --git a/src/qemu/qemu_monitor.c b/s
On Tue, Jul 13, 2021 at 08:57:30 +0200, Michal Privoznik wrote:
> In not so distant past (v6.5.0~3) I've updated the private key we
> use for virnettls* tests. Back then I was driven by Fedora 33
> change which deprecated RSA-1024 which we used back then. I
> generated an EC-384 key which was fine
On Tue, Jul 13, 2021 at 08:44:06 +0200, Michal Privoznik wrote:
> We document that our coding style is to use postfix increment in
> for() loops rather than prefix. This change was generated by the
> following sed script:
>
> for i in $(git grep -l "for (.*; ++.*)"); do \
> sed -i 's/\(for (
On Tue, Jul 13, 2021 at 16:04:12 +0800, Han Han wrote:
> According to definition of virDomainCoreDumpFormat, the "elf" should be
> the first argument in VIR_ENUM_*.
>
> Fixes: 84cc4543be
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1981625
>
> Signed-off-by: Han Han
> ---
> tools/
According to definition of virDomainCoreDumpFormat, the "elf" should be
the first argument in VIR_ENUM_*.
Fixes: 84cc4543be
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1981625
Signed-off-by: Han Han
---
tools/virsh-domain.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On 7/6/21 2:37 PM, Tim Wiederhake wrote:
> Signed-off-by: Tim Wiederhake
> ---
> src/qemu/qemu_monitor.c | 16 ++--
> 1 file changed, 6 insertions(+), 10 deletions(-)
>
> diff --git a/src/qemu/qemu_monitor.c b/src/qemu/qemu_monitor.c
> index cb59fc7b7b..4489b809f4 100644
> --- a/src/
On 7/6/21 2:37 PM, Tim Wiederhake wrote:
> "virHashNew" cannot return NULL, yet we check for NULL in various places.
>
> This series is the first of several that remove these checks. Where
> applicable, the functions are refactored to use automatic memory management
> by means of g_autoptr etc. as
43 matches
Mail list logo