Re: [libvirt] [PATCH] build: use correct limit for unsigned long long
On Fri, Jun 29, 2012 at 03:11:41PM -0600, Eric Blake wrote: Reported by Jason Helfman as a build-breaker on FreeBSD. * src/conf/domain_conf.c (virDomainFSDefParseXML): Use POSIX spelling. * src/openvz/openvz_conf.c (openvzReadFSConf): Likewise. --- Pushing under the build-breaker rule. src/conf/domain_conf.c |8 +++- src/openvz/openvz_conf.c |4 ++-- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 4086dac..3fb90db 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -4248,14 +4248,12 @@ virDomainFSDefParseXML(xmlNodePtr node, if (virDomainParseScaledValue(./space_hard_limit[1], ctxt, def-space_hard_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; if (virDomainParseScaledValue(./space_soft_limit[1], ctxt, def-space_soft_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; cur = node-children; @@ -4335,7 +4333,7 @@ virDomainFSDefParseXML(xmlNodePtr node, } if (unit virScaleInteger(def-usage, unit, -1024, ULONG_LONG_MAX) 0) +1024, ULLONG_MAX) 0) goto error; } diff --git a/src/openvz/openvz_conf.c b/src/openvz/openvz_conf.c index bb61b13..ad27d37 100644 --- a/src/openvz/openvz_conf.c +++ b/src/openvz/openvz_conf.c @@ -455,8 +455,8 @@ openvzReadFSConf(virDomainDefPtr def, goto error; } else { /* Ensure that we can multiply by 1024 without overflowing. */ -if (barrier ULONG_LONG_MAX / 1024 || -limit ULONG_LONG_MAX / 1024 ) { +if (barrier ULLONG_MAX / 1024 || +limit ULLONG_MAX / 1024 ) { virReportSystemError(VIR_ERR_OVERFLOW, _(%s), Unable to parse quota); Probably worth blacklisting the non-portable spelling with syntax-check Daniel -- |: http://berrange.com -o-http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] build: use correct limit for unsigned long long
On Fri, Jun 29, 2012 at 03:11:41PM -0600, Eric Blake wrote: Reported by Jason Helfman as a build-breaker on FreeBSD. * src/conf/domain_conf.c (virDomainFSDefParseXML): Use POSIX spelling. * src/openvz/openvz_conf.c (openvzReadFSConf): Likewise. --- Pushing under the build-breaker rule. src/conf/domain_conf.c |8 +++- src/openvz/openvz_conf.c |4 ++-- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 4086dac..3fb90db 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -4248,14 +4248,12 @@ virDomainFSDefParseXML(xmlNodePtr node, if (virDomainParseScaledValue(./space_hard_limit[1], ctxt, def-space_hard_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; if (virDomainParseScaledValue(./space_soft_limit[1], ctxt, def-space_soft_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; cur = node-children; @@ -4335,7 +4333,7 @@ virDomainFSDefParseXML(xmlNodePtr node, } if (unit virScaleInteger(def-usage, unit, -1024, ULONG_LONG_MAX) 0) +1024, ULLONG_MAX) 0) goto error; } diff --git a/src/openvz/openvz_conf.c b/src/openvz/openvz_conf.c index bb61b13..ad27d37 100644 --- a/src/openvz/openvz_conf.c +++ b/src/openvz/openvz_conf.c @@ -455,8 +455,8 @@ openvzReadFSConf(virDomainDefPtr def, goto error; } else { /* Ensure that we can multiply by 1024 without overflowing. */ -if (barrier ULONG_LONG_MAX / 1024 || -limit ULONG_LONG_MAX / 1024 ) { +if (barrier ULLONG_MAX / 1024 || +limit ULLONG_MAX / 1024 ) { virReportSystemError(VIR_ERR_OVERFLOW, _(%s), Unable to parse quota); Ack. -- Guido -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH] build: use correct limit for unsigned long long
Reported by Jason Helfman as a build-breaker on FreeBSD. * src/conf/domain_conf.c (virDomainFSDefParseXML): Use POSIX spelling. * src/openvz/openvz_conf.c (openvzReadFSConf): Likewise. --- Pushing under the build-breaker rule. src/conf/domain_conf.c |8 +++- src/openvz/openvz_conf.c |4 ++-- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 4086dac..3fb90db 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -4248,14 +4248,12 @@ virDomainFSDefParseXML(xmlNodePtr node, if (virDomainParseScaledValue(./space_hard_limit[1], ctxt, def-space_hard_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; if (virDomainParseScaledValue(./space_soft_limit[1], ctxt, def-space_soft_limit, 1, - ULONG_LONG_MAX, - false) 0) + ULLONG_MAX, false) 0) goto error; cur = node-children; @@ -4335,7 +4333,7 @@ virDomainFSDefParseXML(xmlNodePtr node, } if (unit virScaleInteger(def-usage, unit, -1024, ULONG_LONG_MAX) 0) +1024, ULLONG_MAX) 0) goto error; } diff --git a/src/openvz/openvz_conf.c b/src/openvz/openvz_conf.c index bb61b13..ad27d37 100644 --- a/src/openvz/openvz_conf.c +++ b/src/openvz/openvz_conf.c @@ -455,8 +455,8 @@ openvzReadFSConf(virDomainDefPtr def, goto error; } else { /* Ensure that we can multiply by 1024 without overflowing. */ -if (barrier ULONG_LONG_MAX / 1024 || -limit ULONG_LONG_MAX / 1024 ) { +if (barrier ULLONG_MAX / 1024 || +limit ULLONG_MAX / 1024 ) { virReportSystemError(VIR_ERR_OVERFLOW, _(%s), Unable to parse quota); -- 1.7.10.4 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] build: use correct limit for unsigned long long
On Fri, Jun 29, 2012 at 03:11:41PM -0600, Eric Blake thus spake: Reported by Jason Helfman as a build-breaker on FreeBSD. Patch is good. It builds, now. Thanks! -jgh -- Jason Helfman System Administrator experts-exchange.com http://www.experts-exchange.com/M_4830110.html E4AD 7CF1 1396 27F6 79DD 4342 5E92 AD66 8C8C FBA5 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list