From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9]) by lore.proxmox.com (Postfix) with ESMTPS id 88D371FF136 for ; Mon, 09 Mar 2026 14:07:16 +0100 (CET) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id 70769335AB; Mon, 9 Mar 2026 14:07:02 +0100 (CET) From: Fiona Ebner To: pve-devel@lists.proxmox.com Subject: [PATCH qemu-server v2 1/4] qmp client: encode JSON as UTF-8 to fix PBS backup when password contains multi-byte UTF-8 Date: Mon, 9 Mar 2026 14:06:40 +0100 Message-ID: <20260309130653.90674-2-f.ebner@proxmox.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <20260309130653.90674-1-f.ebner@proxmox.com> References: <20260309130653.90674-1-f.ebner@proxmox.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Bm-Milter-Handled: 55990f41-d878-4baa-be0a-ee34c49e34d2 X-Bm-Transport-Timestamp: 1773061584089 X-SPAM-LEVEL: Spam detection results: 0 AWL -2.072 Adjusted score from AWL reputation of From: address BAYES_00 -1.9 Bayes spam probability is 0 to 1% DMARC_MISSING 0.1 Missing DMARC policy KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment KAM_MAILER 2 Automated Mailer Tag Left in Email RCVD_IN_MSPIKE_H2 0.001 Average reputation (+2) RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.408 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_RPBL_BLOCKED 0.819 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. RCVD_IN_VALIDITY_SAFE_BLOCKED 0.903 ADMINISTRATOR NOTICE: The query to Validity was blocked. See https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more information. SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record Message-ID-Hash: SFHXK6XHANHCOLHOZKN7F2STROQBZJ3F X-Message-ID-Hash: SFHXK6XHANHCOLHOZKN7F2STROQBZJ3F X-MailFrom: f.ebner@proxmox.com X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; emergency; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.10 Precedence: list List-Id: Proxmox VE development discussion List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: As reported in the community forum, PBS backup of VMs would not work when the password contained a multi-byte UTF-8 character [0]. The reason is that when writing a string in Perl's internal representation with character values >= 128 to the QMP socket, QEMU seems to have a bug where it reads too few characters (one less for each char value >= 128). In such a case, the QMP client will just time out, as the command is never completely read on the QEMU side. Stuffing with additional characters would actually lead to QEMU reading enough characters and executing the command, but Perl's internal representation should not be submitted to QMP in the first place. Encode the JSON properly as UTF-8 to avoid the issue. In particular, this makes backing up a VM to PBS possible when the PBS password contains multi-byte UTF-8 characters. This also fixes future similar issues, for example when a QMP command is passed a filesystem path with multi-byte UTF-8 characters. [0]: https://forum.proxmox.com/threads/172871/post-804921 Signed-off-by: Fiona Ebner --- Unfortunately, doing the same for QGA is not yet enough to fix issue #6609. src/PVE/QMPClient.pm | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/src/PVE/QMPClient.pm b/src/PVE/QMPClient.pm index 723123a6..3cc5c3c8 100644 --- a/src/PVE/QMPClient.pm +++ b/src/PVE/QMPClient.pm @@ -295,12 +295,14 @@ my $check_queue = sub { . "\n"; } else { - - $qmpcmd = to_json({ - execute => $cmd->{execute}, - arguments => $cmd->{arguments}, - id => $cmd->{id}, - }); + $qmpcmd = to_json( + { + execute => $cmd->{execute}, + arguments => $cmd->{arguments}, + id => $cmd->{id}, + }, + { utf8 => 1 }, + ); } if ($fd >= 0) { -- 2.47.3