qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] monitor: Add whitelist support for QMP commands


From: Julia Suvorova
Subject: Re: [Qemu-devel] [PATCH] monitor: Add whitelist support for QMP commands
Date: Mon, 11 Feb 2019 18:42:24 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1



On 01.02.2019 12:14, Markus Armbruster wrote:
Julia Suvorova via Qemu-devel <address@hidden> writes:

The whitelist option allows to run a reduced monitor with a subset of
QMP commands. This allows the monitor to run in secure mode, which is

For a value of "secure".  I'm not saying this can't be useful, just
tempering expecations.  I guess you intend to use this to restrict the
monitor to sufficiently harmless commands, such as commands that merely
return information without changing anything.  However, even such
commands can be abused for denial of service.  Whether that's an issue
depends on your use case.

convenient for sending commands via the WebSocket monitor using the
web UI. This is planned to be done on micro:bit board.

The list of allowed commands should be written to a file, one per line.
The command line will look like this:
     -mon chardev_name,mode=control,whitelist=path_to_file

Signed-off-by: Julia Suvorova <address@hidden>

Please describe your intended use case in more detail, and provide at
least a rough security analysis that includes the denial of service
aspect.

It is planned to use the web interface for micro:bit board, e.g. send a
button press as a QMP command and send a LED display changing as a QMP
event, and send them via WebSocket protocol from QEMU to a web page.
The monitor will also send commands from some other sensors, for
example, an accelerometer/magnetometer. Therefore, it is convenient to
limit the monitor so that it can send only these commands of
buttons/sensors.

Best regards, Julia Suvorova.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]