Re: [libvirt-glib PATCHv2] Fix *_new_from_xml

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



ping?

Christophe

On Wed, Jul 18, 2012 at 03:39:39PM +0200, Christophe Fergeau wrote:
> For objects with a subtype 'type' attribute, when the _new_from_xml
> function was called, the 'type' attribute was forcefully set to the
> right value rather than checking that the passed-in value matches
> the type of the subclass we are trying to instantiate. This commit
> changes this, and returns NULL when the value of the 'type' attribute
> of the passed-in XML document does not match the expected type.
> ---
>  libvirt-gconfig/libvirt-gconfig-domain-address-pci.c             |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-address-usb.c             |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-chardev-source-pty.c      |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-chardev-source-spicevmc.c |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-controller-usb.c          |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-graphics-sdl.c            |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-graphics-spice.c          |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-graphics-vnc.c            |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-interface-bridge.c        |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-interface-network.c       |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-interface-user.c          |    5 +++--
>  libvirt-gconfig/libvirt-gconfig-domain-timer-pit.c               |    5 ++++-
>  libvirt-gconfig/libvirt-gconfig-domain-timer-rtc.c               |    5 ++++-
>  13 files changed, 46 insertions(+), 19 deletions(-)

Attachment: pgp9GFTbpqFd4.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]