[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] e5b815: qemu-nbd: regression with arguments p
From: |
Richard Henderson |
Subject: |
[Qemu-commits] [qemu/qemu] e5b815: qemu-nbd: regression with arguments passing into n... |
Date: |
Fri, 28 Jul 2023 09:57:46 -0700 |
Branch: refs/heads/staging
Home: https://github.com/qemu/qemu
Commit: e5b815b0defcc3617f473ba70c3e675ef0ee69c2
https://github.com/qemu/qemu/commit/e5b815b0defcc3617f473ba70c3e675ef0ee69c2
Author: Denis V. Lunev <den@openvz.org>
Date: 2023-07-27 (Thu, 27 Jul 2023)
Changed paths:
M qemu-nbd.c
Log Message:
-----------
qemu-nbd: regression with arguments passing into nbd_client_thread()
Unfortunately
commit 03b67621445d601c9cdc7dfe25812e9f19b81488
Author: Denis V. Lunev <den@openvz.org>
Date: Mon Jul 17 16:55:40 2023 +0200
qemu-nbd: pass structure into nbd_client_thread instead of plain char*
has introduced a regression. struct NbdClientOpts resides on stack inside
'if' block. This specifically means that this stack space could be reused
once the execution will leave that block of the code.
This means that parameters passed into nbd_client_thread could be
overwritten at any moment.
The patch moves the data to the namespace of main() function effectively
preserving it for the whole process lifetime.
Signed-off-by: Denis V. Lunev <den@openvz.org>
CC: Eric Blake <eblake@redhat.com>
CC: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
CC: <qemu-stable@nongnu.org>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-ID: <20230727105828.324314-1-den@openvz.org>
Signed-off-by: Eric Blake <eblake@redhat.com>
Commit: ccb86f079a9e4d94918086a9df18c1844347aff8
https://github.com/qemu/qemu/commit/ccb86f079a9e4d94918086a9df18c1844347aff8
Author: Richard Henderson <richard.henderson@linaro.org>
Date: 2023-07-28 (Fri, 28 Jul 2023)
Changed paths:
M qemu-nbd.c
Log Message:
-----------
Merge tag 'pull-nbd-2023-07-28' of https://repo.or.cz/qemu/ericb into staging
NBD patches for 2023-07-28
- Denis V. Lunev: Fix regression in 'qemu-nbd -c /dev/nbdN'
# -----BEGIN PGP SIGNATURE-----
#
# iQEzBAABCAAdFiEEccLMIrHEYCkn0vOqp6FrSiUnQ2oFAmTD1ncACgkQp6FrSiUn
# Q2pTJwf/ajrWlu3O6VF2xeavAbeBNpEtGkGRuuWAcY9l3XubKA6CYVOWXsXaqKiB
# CzqaiiMpAU4EeRroOjM0REWbVr1VyHtxGV3neCxcbXoXvZXaYDaBU5KAalZv++os
# +e0tit93LnJ+MYYx6r9z2MmD+A1yXqHo+K4lvI2hd3royYC0zn/1lFJxGqk8is8O
# EHI9o40zMaQ2l+zrsC5tMuxQf6EGjtHm9rtRRASDnyf8V4zGEwWMQ30Xd31nFVGS
# V9+XnIIWRNhY1DDNBnNrn6If+MiXeBfV7UPof7gMjAv7v4QdCdPELi7UBbkQU2d6
# y87w3KxgVlgDQ7IskA+Y1ykEFXCKAw==
# =N00U
# -----END PGP SIGNATURE-----
# gpg: Signature made Fri 28 Jul 2023 07:53:43 AM PDT
# gpg: using RSA key 71C2CC22B1C4602927D2F3AAA7A16B4A2527436A
# gpg: Good signature from "Eric Blake <eblake@redhat.com>" [full]
# gpg: aka "Eric Blake (Free Software Programmer)
<ebb9@byu.net>" [full]
# gpg: aka "[jpeg image of size 6874]" [full]
* tag 'pull-nbd-2023-07-28' of https://repo.or.cz/qemu/ericb:
qemu-nbd: regression with arguments passing into nbd_client_thread()
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Compare: https://github.com/qemu/qemu/compare/f33c74576425...ccb86f079a9e
- [Qemu-commits] [qemu/qemu] e5b815: qemu-nbd: regression with arguments passing into n...,
Richard Henderson <=