Re: [PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-10-16 Thread Markus Armbruster
Advice for next time... Peter Xu writes: > QEMU will crash if anyone tries to set tls-authz (which is a type > StrOrNull) with 'null' value. Fix it in the easy way by converting it to > qstring just like the other two tls parameters. Include a reproducer whenever practical. Helps backporters.

Re: [PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-10-04 Thread Juan Quintela
Peter Xu wrote: > QEMU will crash if anyone tries to set tls-authz (which is a type > StrOrNull) with 'null' value. Fix it in the easy way by converting it to > qstring just like the other two tls parameters. > > Cc: qemu-sta...@nongnu.org # v4.0+ > Fixes: d2f1d29b95 ("migration: add support for

Re: [PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-09-27 Thread Michael Tokarev
28.09.2023 08:36, Markus Armbruster wrote: Michael Tokarev writes: 05.09.2023 19:23, Peter Xu: QEMU will crash if anyone tries to set tls-authz (which is a type StrOrNull) with 'null' value. Fix it in the easy way by converting it to qstring just like the other two tls parameters. Ping? H

Re: [PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-09-27 Thread Markus Armbruster
Michael Tokarev writes: > 05.09.2023 19:23, Peter Xu: >> QEMU will crash if anyone tries to set tls-authz (which is a type >> StrOrNull) with 'null' value. Fix it in the easy way by converting it to >> qstring just like the other two tls parameters. > > Ping? Has this been forgotten? Obviously

Re: [PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-09-27 Thread Michael Tokarev
05.09.2023 19:23, Peter Xu: QEMU will crash if anyone tries to set tls-authz (which is a type StrOrNull) with 'null' value. Fix it in the easy way by converting it to qstring just like the other two tls parameters. Ping? Has this been forgotten? Thanks, /mjt Cc: qemu-sta...@nongnu.org # v

[PATCH v3 1/4] migration/qmp: Fix crash on setting tls-authz with null

2023-09-05 Thread Peter Xu
QEMU will crash if anyone tries to set tls-authz (which is a type StrOrNull) with 'null' value. Fix it in the easy way by converting it to qstring just like the other two tls parameters. Cc: qemu-sta...@nongnu.org # v4.0+ Fixes: d2f1d29b95 ("migration: add support for a "tls-authz" migration par