summaryrefslogtreecommitdiffstats
path: root/hw/ide/atapi.c
diff options
context:
space:
mode:
authorKevin Wolf2015-07-27 05:42:53 +0200
committerJohn Snow2015-07-27 05:42:53 +0200
commit03441c3a4a42beb25460dd11592539030337d0f8 (patch)
tree5c1d4295d5644656ede50367180519b930431cba /hw/ide/atapi.c
parentide: Check array bounds before writing to io_buffer (CVE-2015-5154) (diff)
downloadqemu-03441c3a4a42beb25460dd11592539030337d0f8.tar.gz
qemu-03441c3a4a42beb25460dd11592539030337d0f8.tar.xz
qemu-03441c3a4a42beb25460dd11592539030337d0f8.zip
ide/atapi: Fix START STOP UNIT command completion
The command must be completed on all code paths. START STOP UNIT with pwrcnd set should succeed without doing anything. Signed-off-by: Kevin Wolf <kwolf@redhat.com> Reviewed-by: John Snow <jsnow@redhat.com>
Diffstat (limited to 'hw/ide/atapi.c')
-rw-r--r--hw/ide/atapi.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c
index 950e311d31..79dd167107 100644
--- a/hw/ide/atapi.c
+++ b/hw/ide/atapi.c
@@ -983,6 +983,7 @@ static void cmd_start_stop_unit(IDEState *s, uint8_t* buf)
if (pwrcnd) {
/* eject/load only happens for power condition == 0 */
+ ide_atapi_cmd_ok(s);
return;
}