405484b29f
With VNC, a client can send a non-extended VNC_MSG_CLIENT_CUT_TEXT message with len=0. In qemu_clipboard_set_data(), the clipboard info will be updated setting data to NULL (because g_memdup(data, size) returns NULL when size is 0). If the client does not set the VNC_ENCODING_CLIPBOARD_EXT feature when setting up the encodings, then the 'request' callback for the clipboard peer is not initialized. Later, because data is NULL, qemu_clipboard_request() can be reached via vdagent_chr_write() and vdagent_clipboard_recv_request() and there, the clipboard owner's 'request' callback will be attempted to be called, but that is a NULL pointer. In particular, this can happen when using the KRDC (22.12.3) VNC client. Another scenario leading to the same issue is with two clients (say noVNC and KRDC): The noVNC client sets the extension VNC_FEATURE_CLIPBOARD_EXT and initializes its cbpeer. The KRDC client does not, but triggers a vnc_client_cut_text() (note it's not the _ext variant)). There, a new clipboard info with it as the 'owner' is created and via qemu_clipboard_set_data() is called, which in turn calls qemu_clipboard_update() with that info. In qemu_clipboard_update(), the notifier for the noVNC client will be called, i.e. vnc_clipboard_notify() and also set vs->cbinfo for the noVNC client. The 'owner' in that clipboard info is the clipboard peer for the KRDC client, which did not initialize the 'request' function. That sounds correct to me, it is the owner of that clipboard info. Then when noVNC sends a VNC_MSG_CLIENT_CUT_TEXT message (it did set the VNC_FEATURE_CLIPBOARD_EXT feature correctly, so a check for it passes), that clipboard info is passed to qemu_clipboard_request() and the original segfault still happens. Fix the issue by handling updates with size 0 differently. In particular, mark in the clipboard info that the type is not available. While at it, switch to g_memdup2(), because g_memdup() is deprecated. Cc: qemu-stable@nongnu.org Fixes: CVE-2023-6683 Reported-by: Markus Frank <m.frank@proxmox.com> Suggested-by: Marc-André Lureau <marcandre.lureau@redhat.com> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com> Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com> Tested-by: Markus Frank <m.frank@proxmox.com> Message-ID: <20240124105749.204610-1-f.ebner@proxmox.com>
180 lines
4.5 KiB
C
180 lines
4.5 KiB
C
#include "qemu/osdep.h"
|
|
#include "ui/clipboard.h"
|
|
#include "trace.h"
|
|
|
|
static NotifierList clipboard_notifiers =
|
|
NOTIFIER_LIST_INITIALIZER(clipboard_notifiers);
|
|
|
|
static QemuClipboardInfo *cbinfo[QEMU_CLIPBOARD_SELECTION__COUNT];
|
|
|
|
void qemu_clipboard_peer_register(QemuClipboardPeer *peer)
|
|
{
|
|
notifier_list_add(&clipboard_notifiers, &peer->notifier);
|
|
}
|
|
|
|
void qemu_clipboard_peer_unregister(QemuClipboardPeer *peer)
|
|
{
|
|
int i;
|
|
|
|
for (i = 0; i < QEMU_CLIPBOARD_SELECTION__COUNT; i++) {
|
|
qemu_clipboard_peer_release(peer, i);
|
|
}
|
|
notifier_remove(&peer->notifier);
|
|
}
|
|
|
|
bool qemu_clipboard_peer_owns(QemuClipboardPeer *peer,
|
|
QemuClipboardSelection selection)
|
|
{
|
|
QemuClipboardInfo *info = qemu_clipboard_info(selection);
|
|
|
|
return info && info->owner == peer;
|
|
}
|
|
|
|
void qemu_clipboard_peer_release(QemuClipboardPeer *peer,
|
|
QemuClipboardSelection selection)
|
|
{
|
|
g_autoptr(QemuClipboardInfo) info = NULL;
|
|
|
|
if (qemu_clipboard_peer_owns(peer, selection)) {
|
|
/* set empty clipboard info */
|
|
info = qemu_clipboard_info_new(NULL, selection);
|
|
qemu_clipboard_update(info);
|
|
}
|
|
}
|
|
|
|
bool qemu_clipboard_check_serial(QemuClipboardInfo *info, bool client)
|
|
{
|
|
bool ok;
|
|
|
|
if (!info->has_serial ||
|
|
!cbinfo[info->selection] ||
|
|
!cbinfo[info->selection]->has_serial) {
|
|
trace_clipboard_check_serial(-1, -1, true);
|
|
return true;
|
|
}
|
|
|
|
if (client) {
|
|
ok = info->serial >= cbinfo[info->selection]->serial;
|
|
} else {
|
|
ok = info->serial > cbinfo[info->selection]->serial;
|
|
}
|
|
|
|
trace_clipboard_check_serial(cbinfo[info->selection]->serial, info->serial, ok);
|
|
return ok;
|
|
}
|
|
|
|
void qemu_clipboard_update(QemuClipboardInfo *info)
|
|
{
|
|
QemuClipboardNotify notify = {
|
|
.type = QEMU_CLIPBOARD_UPDATE_INFO,
|
|
.info = info,
|
|
};
|
|
assert(info->selection < QEMU_CLIPBOARD_SELECTION__COUNT);
|
|
|
|
notifier_list_notify(&clipboard_notifiers, ¬ify);
|
|
|
|
if (cbinfo[info->selection] != info) {
|
|
qemu_clipboard_info_unref(cbinfo[info->selection]);
|
|
cbinfo[info->selection] = qemu_clipboard_info_ref(info);
|
|
}
|
|
}
|
|
|
|
QemuClipboardInfo *qemu_clipboard_info(QemuClipboardSelection selection)
|
|
{
|
|
assert(selection < QEMU_CLIPBOARD_SELECTION__COUNT);
|
|
|
|
return cbinfo[selection];
|
|
}
|
|
|
|
QemuClipboardInfo *qemu_clipboard_info_new(QemuClipboardPeer *owner,
|
|
QemuClipboardSelection selection)
|
|
{
|
|
QemuClipboardInfo *info = g_new0(QemuClipboardInfo, 1);
|
|
|
|
info->owner = owner;
|
|
info->selection = selection;
|
|
info->refcount = 1;
|
|
|
|
return info;
|
|
}
|
|
|
|
QemuClipboardInfo *qemu_clipboard_info_ref(QemuClipboardInfo *info)
|
|
{
|
|
info->refcount++;
|
|
return info;
|
|
}
|
|
|
|
void qemu_clipboard_info_unref(QemuClipboardInfo *info)
|
|
{
|
|
uint32_t type;
|
|
|
|
if (!info) {
|
|
return;
|
|
}
|
|
|
|
info->refcount--;
|
|
if (info->refcount > 0) {
|
|
return;
|
|
}
|
|
|
|
for (type = 0; type < QEMU_CLIPBOARD_TYPE__COUNT; type++) {
|
|
g_free(info->types[type].data);
|
|
}
|
|
g_free(info);
|
|
}
|
|
|
|
void qemu_clipboard_request(QemuClipboardInfo *info,
|
|
QemuClipboardType type)
|
|
{
|
|
if (info->types[type].data ||
|
|
info->types[type].requested ||
|
|
!info->types[type].available ||
|
|
!info->owner)
|
|
return;
|
|
|
|
info->types[type].requested = true;
|
|
info->owner->request(info, type);
|
|
}
|
|
|
|
void qemu_clipboard_reset_serial(void)
|
|
{
|
|
QemuClipboardNotify notify = { .type = QEMU_CLIPBOARD_RESET_SERIAL };
|
|
int i;
|
|
|
|
for (i = 0; i < QEMU_CLIPBOARD_SELECTION__COUNT; i++) {
|
|
QemuClipboardInfo *info = qemu_clipboard_info(i);
|
|
if (info) {
|
|
info->serial = 0;
|
|
}
|
|
}
|
|
notifier_list_notify(&clipboard_notifiers, ¬ify);
|
|
}
|
|
|
|
void qemu_clipboard_set_data(QemuClipboardPeer *peer,
|
|
QemuClipboardInfo *info,
|
|
QemuClipboardType type,
|
|
uint32_t size,
|
|
const void *data,
|
|
bool update)
|
|
{
|
|
if (!info ||
|
|
info->owner != peer) {
|
|
return;
|
|
}
|
|
|
|
g_free(info->types[type].data);
|
|
if (size) {
|
|
info->types[type].data = g_memdup2(data, size);
|
|
info->types[type].size = size;
|
|
info->types[type].available = true;
|
|
} else {
|
|
info->types[type].data = NULL;
|
|
info->types[type].size = 0;
|
|
info->types[type].available = false;
|
|
}
|
|
|
|
if (update) {
|
|
qemu_clipboard_update(info);
|
|
}
|
|
}
|