On Sat, Nov 12, 2011 at 12:16:23PM -0800, Garrett Cooper wrote: > On Sat, Nov 12, 2011 at 12:01 PM, Alexander Motin <m...@freebsd.org> wrote: > > Author: mav > > Date: Sat Nov 12 20:01:30 2011 > > New Revision: 227473 > > URL: http://svn.freebsd.org/changeset/base/227473 > > > > Log: > > Fix build on some archs after r227464. > > > > Modified: > > head/sbin/geom/class/multipath/geom_multipath.c > > > > Modified: head/sbin/geom/class/multipath/geom_multipath.c > > ============================================================================== > > --- head/sbin/geom/class/multipath/geom_multipath.c Sat Nov 12 19:55:48 > > 2011 (r227472) > > +++ head/sbin/geom/class/multipath/geom_multipath.c Sat Nov 12 20:01:30 > > 2011 (r227473) > > @@ -133,7 +133,8 @@ mp_label(struct gctl_req *req) > > uint8_t *sector, *rsector; > > char *ptr; > > uuid_t uuid; > > - uint32_t secsize = 0, ssize, status; > > + ssize_t secsize = 0, ssize; > > + uint32_t status; > > const char *name, *name2, *mpname; > > int error, i, nargs, fd; > > > > @@ -161,8 +162,8 @@ mp_label(struct gctl_req *req) > > disksize = msize; > > } else { > > if (secsize != ssize) { > > - gctl_error(req, "%s sector size %u > > different.", > > - name, ssize); > > + gctl_error(req, "%s sector size %ju > > different.", > > + name, (intmax_t)ssize); > > Shouldn't that be uintmax_t, not intmax_t ?
No, ssize_t is signed. Although the best would be to use %zd for ssize_t. -- Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://yomoli.com
pgpmWxqeYa6WI.pgp
Description: PGP signature
