Bug 813972

Summary: libvirt should reject invalid memory values in xml
Product: Red Hat Enterprise Linux 6 Reporter: Eric Blake <eblake>
Component: libvirtAssignee: Eric Blake <eblake>
Status: CLOSED ERRATA QA Contact: Virtualization Bugs <virt-bugs>
Severity: low Docs Contact:
Priority: medium    
Version: 6.3CC: acathrow, ajia, dallan, dyasny, dyuan, eblake, gsun, mzhan, rwu, veillard
Target Milestone: rc   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: libvirt-0.9.10-14.el6 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 796526 Environment:
Last Closed: 2012-06-20 06:54:31 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:
Bug Depends On: 617711, 770683, 796526    
Bug Blocks: 808522    

Comment 4 dyuan 2012-04-25 07:36:09 UTC
Verified PASS with libvirt-0.9.10-14.el6.

virsh will reject the following xml:
<memory>1G</memory>
<currentMemory>1G</currentMemory>

# virsh edit xp
error: XML error: could not parse memory element ./currentMemory[1]

# virsh define /tmp/xp.xml 
error: Failed to define domain from /tmp/xp.xml
error: XML error: could not parse memory element ./currentMemory[1]

# virsh create /tmp/xp.xml 
error: Failed to create domain from /tmp/xp.xml
error: XML error: could not parse memory element ./currentMemory[1]

Reproduced this bug with libvirt-0.9.10-12.el6.
libvirt silently use <memory unit='KiB'>1</memory> instead of 1G.

Comment 6 errata-xmlrpc 2012-06-20 06:54:31 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHSA-2012-0748.html