Re: [Qemu-devel] [PATCH] vmdk: refuse to open higher version than supported
On Sun, Jun 09, 2013 at 09:44:15AM +0800, Fam Zheng wrote:
Although we try to be compatible with published VMDK spec, VMware has
newer version from ESXi 5.1 exported OVF/OVA, which we have no knowledge
what's changed in it. And it is very likely to have more new versions in
the future, so it's not safe to open them blindly.
The best I could find was this high-level overview:
http://myvirtualcloud.net/?p=3829
Signed-off-by: Fam Zheng f...@redhat.com
---
block/vmdk.c | 4
1 file changed, 4 insertions(+)
diff --git a/block/vmdk.c b/block/vmdk.c
index 608daaf..d9c2368 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@ -558,6 +558,10 @@ static int vmdk_open_vmdk4(BlockDriverState *bs,
header = footer.header;
}
+if (le32_to_cpu(header.version) = 3) {
+return -EINVAL;
+}
+
Looks fine, the VMDK 5.0 spec says header.version may be 1 or 2.
Reviewed-by: Stefan Hajnoczi stefa...@redhat.com
Re: [Qemu-devel] [PATCH] vmdk: refuse to open higher version than supported
Am 09.06.2013 um 03:44 hat Fam Zheng geschrieben:
Refuse to open higher version for safety.
Although we try to be compatible with published VMDK spec, VMware has
newer version from ESXi 5.1 exported OVF/OVA, which we have no knowledge
what's changed in it. And it is very likely to have more new versions in
the future, so it's not safe to open them blindly.
Signed-off-by: Fam Zheng f...@redhat.com
Yes, it's definitely a good idea to add a check.
@@ -558,6 +558,10 @@ static int vmdk_open_vmdk4(BlockDriverState *bs,
header = footer.header;
}
+if (le32_to_cpu(header.version) = 3) {
+return -EINVAL;
+}
+
Other block drivers return -ENOTSUP for this case, and also call
qerror_report(QERR_UNKNOWN_BLOCK_FORMAT_FEATURE, ...) so that you get a
meaningful error message. Can you model the VMDK code after them?
Kevin
[Qemu-devel] [PATCH] vmdk: refuse to open higher version than supported
Refuse to open higher version for safety.
Although we try to be compatible with published VMDK spec, VMware has
newer version from ESXi 5.1 exported OVF/OVA, which we have no knowledge
what's changed in it. And it is very likely to have more new versions in
the future, so it's not safe to open them blindly.
Signed-off-by: Fam Zheng f...@redhat.com
---
block/vmdk.c | 4
1 file changed, 4 insertions(+)
diff --git a/block/vmdk.c b/block/vmdk.c
index 608daaf..d9c2368 100644
--- a/block/vmdk.c
+++ b/block/vmdk.c
@@ -558,6 +558,10 @@ static int vmdk_open_vmdk4(BlockDriverState *bs,
header = footer.header;
}
+if (le32_to_cpu(header.version) = 3) {
+return -EINVAL;
+}
+
l1_entry_sectors = le32_to_cpu(header.num_gtes_per_gte)
* le64_to_cpu(header.granularity);
if (l1_entry_sectors == 0) {
--
1.8.3
