Bug 533573

Summary:	qemu-img convert can't handle parallels images above 4GiB.
Product:	[Fedora] Fedora	Reporter:	David Woodhouse <dwmw2>
Component:	qemu	Assignee:	Justin M. Forbes <jforbes>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	low
Version:	12	CC:	berrange, dwmw2, gcosta, itamar, jaswinder, jforbes, markmc, quintela, virt-maint
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:	0.11.0-12.fc12	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2009-12-08 08:04:05 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description David Woodhouse 2009-11-07 14:05:32 UTC

See 32-bit arithmetic on 'position' in the seek_to_sector() function in block/parallels.c

No wonder it gives me crap. Making 'position' a uint64_t and ensuring that the arithmetic on it is also suitably cast to uint64_t gives me a more reasonable converted image.

Comment 1 Mark McLoughlin 2009-11-19 10:29:48 UTC

Fix is committed upstream now http://repo.or.cz/w/qemu.git/commitdiff/c34d2451ed

dwmw2: do you need cherry-picked into F-12, or are you happy to mark this as CLOSED UPSTREAM ?

Comment 2 David Woodhouse 2009-11-19 11:01:19 UTC

Turns out I can't get our Parallels images (of Windows XP) to boot in qemu/KVM anyway -- I can't seem to solve the problems described at http://www.virtualbox.org/wiki/Migrate_Windows

So for now I'm not advising people to convert our existing Parallels images -- if I do get it working, I'd probably have to give them a 'fixed' image, which might as well be in qcow2 format already.

So I'm not personally desperate to get it, but it _would_ be nice to drop it into the next update if it's not too much trouble -- a conversion tool which can only handle images smaller than 4GiB isn't really something we should be shipping these days :)

Comment 3 Mark McLoughlin 2009-11-20 18:30:56 UTC

Building this now:

* Fri Nov 20 2009 Mark McLoughlin <markmc> - 2:0.11.0-12
- Fix a use-after-free crasher in the slirp code (#539583)
- Fix overflow in the parallels image format support (#533573)

Comment 4 Fedora Update System 2009-11-20 19:12:59 UTC

qemu-0.11.0-12.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/qemu-0.11.0-12.fc12

Comment 5 Fedora Update System 2009-11-24 07:36:07 UTC

qemu-0.11.0-12.fc12 has been pushed to the Fedora 12 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update qemu'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F12/FEDORA-2009-11901

Comment 6 Fedora Update System 2009-12-08 08:03:53 UTC

qemu-0.11.0-12.fc12 has been pushed to the Fedora 12 stable repository.  If problems still persist, please make note of it in this bug report.