|
 |
76daa3 |
From bbd4931e233557c34ec216ed4784282a16b4f3e4 Mon Sep 17 00:00:00 2001
|
|
|
76daa3 |
From: Kevin Wolf <kwolf@redhat.com>
|
|
|
76daa3 |
Date: Wed, 31 May 2017 11:15:12 +0200
|
|
|
76daa3 |
Subject: [PATCH 05/13] qemu-iotests: Test streaming with missing job ID
|
|
|
76daa3 |
|
|
|
76daa3 |
RH-Author: Kevin Wolf <kwolf@redhat.com>
|
|
|
76daa3 |
Message-id: <1496229312-8649-3-git-send-email-kwolf@redhat.com>
|
|
|
76daa3 |
Patchwork-id: 75444
|
|
|
76daa3 |
O-Subject: [RHEL-7.4 qemu-kvm-rhev PATCH 2/2] qemu-iotests: Test streaming with missing job ID
|
|
|
76daa3 |
Bugzilla: 1456424
|
|
|
76daa3 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Jeffrey Cody <jcody@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: Max Reitz <mreitz@redhat.com>
|
|
|
76daa3 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
76daa3 |
|
|
|
76daa3 |
This adds a small test for the image streaming error path for failing
|
|
|
76daa3 |
block_job_create(), which would have found the null pointer dereference
|
|
|
76daa3 |
in commit a170a91f.
|
|
|
76daa3 |
|
|
|
76daa3 |
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Alberto Garcia <berto@igalia.com>
|
|
|
76daa3 |
Reviewed-by: Kashyap Chamarthy <kchamart@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
76daa3 |
Reviewed-by: Jeff Cody <jcody@redhat.com>
|
|
|
76daa3 |
(cherry picked from commit 0bb0aea4bac2332389a9b481f650a2a9df80ea0c)
|
|
|
76daa3 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
76daa3 |
---
|
|
|
76daa3 |
tests/qemu-iotests/030 | 4 ++++
|
|
|
76daa3 |
tests/qemu-iotests/030.out | 4 ++--
|
|
|
76daa3 |
2 files changed, 6 insertions(+), 2 deletions(-)
|
|
|
76daa3 |
|
|
|
76daa3 |
diff --git a/tests/qemu-iotests/030 b/tests/qemu-iotests/030
|
|
|
76daa3 |
index 0d472d5..e59bee2 100755
|
|
|
76daa3 |
--- a/tests/qemu-iotests/030
|
|
|
76daa3 |
+++ b/tests/qemu-iotests/030
|
|
|
76daa3 |
@@ -147,6 +147,10 @@ class TestSingleDrive(iotests.QMPTestCase):
|
|
|
76daa3 |
result = self.vm.qmp('block-stream', device='nonexistent')
|
|
|
76daa3 |
self.assert_qmp(result, 'error/class', 'GenericError')
|
|
|
76daa3 |
|
|
|
76daa3 |
+ def test_job_id_missing(self):
|
|
|
76daa3 |
+ result = self.vm.qmp('block-stream', device='mid')
|
|
|
76daa3 |
+ self.assert_qmp(result, 'error/class', 'GenericError')
|
|
|
76daa3 |
+
|
|
|
76daa3 |
|
|
|
76daa3 |
class TestParallelOps(iotests.QMPTestCase):
|
|
|
76daa3 |
num_ops = 4 # Number of parallel block-stream operations
|
|
|
76daa3 |
diff --git a/tests/qemu-iotests/030.out b/tests/qemu-iotests/030.out
|
|
|
76daa3 |
index 84bfd63..391c857 100644
|
|
|
76daa3 |
--- a/tests/qemu-iotests/030.out
|
|
|
76daa3 |
+++ b/tests/qemu-iotests/030.out
|
|
|
76daa3 |
@@ -1,5 +1,5 @@
|
|
|
76daa3 |
-......................
|
|
|
76daa3 |
+.......................
|
|
|
76daa3 |
----------------------------------------------------------------------
|
|
|
76daa3 |
-Ran 22 tests
|
|
|
76daa3 |
+Ran 23 tests
|
|
|
76daa3 |
|
|
|
76daa3 |
OK
|
|
|
76daa3 |
--
|
|
|
76daa3 |
1.8.3.1
|
|
|
76daa3 |
|