Build Update for libvirt/libvirt
-------------------------------------
Build: #1050
Status: Errored
Duration: 11 minutes and 31 seconds
Commit: 5a81acc (v3.7-maint)
Author: Laine Stump
Message: vbox: fix SEGV during dumpxml of a serial port
commit 77a12987a48 changed the "virDomainChrSourceDef source" inside
virDomainChrDef to "virDomainChrSourceDefPtr source", and started
allocating source inside virDomainChrDefNew(), but vboxDumpSerial()
was allocating a virDomainChrDef with a simple VIR_ALLOC() (i.e. never
calling virDomainChrDefNew()), so source was never initialized,
leading to a SEGV any time a serial port was present. The same problem
was created in vboxDumpParallel().
This patch changes vboxDumpSerial() and vboxDumpParallel() to use
virDomainChrDefNew() instead of VIR_ALLOC(), and changes both of those
functions to return an error if virDomainChrDef() (or any other
allocation) fails.
This resolves:
https://bugzilla.redhat.com/1536649
(cherry picked from commit 9c27e464e3b4603cbe13c00787f4c89e5b1e7a68)
Signed-off-by: Laine Stump <laine(a)laine.org>
View the changeset:
https://github.com/libvirt/libvirt/compare/979a7b3feba1...5a81acc80fc7
View the full build log and details:
https://travis-ci.org/libvirt/libvirt/builds/356027948?utm_source=email&a...
--
You can configure recipients for build notifications in your .travis.yml file. See
https://docs.travis-ci.com/user/notifications
This email was sent to libvirt-ci(a)redhat.com (mailto:libvirt-ci@redhat.com)
unsubscribe from this list
(
http://clicks.travis-ci.com/track/unsub.php?u=14313403&id=c709e36f49a...)