]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
virConfGetValueSSizeT: Fix build on 32 bits
authorMichal Privoznik <mprivozn@redhat.com>
Thu, 14 Jul 2016 21:37:56 +0000 (23:37 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Fri, 15 Jul 2016 03:58:42 +0000 (05:58 +0200)
This function tries to get a ssize_t value from a config file.
But before returning it, it checks whether the value would fit in
ssize_t and if not an error is printed out among with the range
for the ssize_t type. However, on some platforms SSIZE_MAX may
actually be a signed long type:

util/virconf.c: In function 'virConfGetValueSSizeT':
util/virconf.c:1268:9: error: format '%zd' expects argument of type 'signed size_t', but argument 9 has type 'long int' [-Werror=format=]
         virReportError(VIR_ERR_INTERNAL_ERROR,
         ^
$ grep -r SSIZE_MAX /usr/include/
/usr/include/bits/posix1_lim.h:#ifndef  SSIZE_MAX
/usr/include/bits/posix1_lim.h:# define SSIZE_MAX       LONG_MAX

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/util/virconf.c

index 5085768eb8716a6040acadc232cab4911b825c9e..4dc82ec66adb3c2a6445dc867b0168d4764e23b4 100644 (file)
@@ -1267,7 +1267,8 @@ int virConfGetValueSSizeT(virConfPtr conf,
     if (cval->l > SSIZE_MAX || cval->l < (-SSIZE_MAX - 1)) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("%s: value for '%s' parameter must be in range %zd:%zd"),
-                       conf->filename, setting, -SSIZE_MAX - 1, SSIZE_MAX);
+                       conf->filename, setting,
+                       (ssize_t) -SSIZE_MAX - 1, (ssize_t) SSIZE_MAX);
         return -1;
     }