On Tue, Aug 11, 2015 at 05:37:30PM -0400, Cole Robinson wrote:
On 08/11/2015 05:32 PM, John Ferlan wrote:On 08/11/2015 05:28 PM, Cole Robinson wrote:On 08/11/2015 05:25 PM, John Ferlan wrote:On 08/10/2015 07:33 PM, Cole Robinson wrote:If you pass <disk><serial> XML to UpdateDevice, and the original device didn't have a <serial> block, libvirtd crashes trying to read the original NULL serial string. Use _NULLABLE string comparisons to avoid the crash. A couple other properties needed the change too. --- src/conf/domain_conf.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)Using STRNEQ_NULLABLE means you probably don't have to check the disk->* value either, right? ACK - with that adjustment...It would change the semantic a bit; if the orig disk has a <serial>, but not the XML passed to UpdateDevice, libvirt would now error where previously it wouldn't. It might make sense but I didn't want to change behavior - ColeOK - fair enough... right - time to quit for the day I guess ;-) ACK to what's on listThanks, pushed now
Just a short info. Thanks for finding this out, sorry I meesed it up. And you still have to check for the disk->value because that's there on purpose. Only check changed values, no value means no check to be done, so you chose wisely :) Have a nice day, Martin
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list