> Index: usr.sbin/mtree/spec.c
> ===================================================================
> RCS file: /data/src/openbsd/src/usr.sbin/mtree/spec.c,v
> retrieving revision 1.26
> diff -u -p -r1.26 spec.c
> --- usr.sbin/mtree/spec.c     8 Jul 2012 21:19:42 -0000       1.26
> +++ usr.sbin/mtree/spec.c     14 Aug 2016 07:38:44 -0000
> @@ -241,7 +241,7 @@ set(char *t, NODE *ip)
>                               error("%s", strerror(errno));
>                       break;
>               case F_SIZE:
> -                     ip->st_size = strtouq(val, &ep, 10);
> +                     ip->st_size = strtoull(val, &ep, 10);
>                       if (*ep)
>                               error("invalid size %s", val);
>                       break;

st_size is an off_t, so shouldn't that rather be strtoll? Also, mtree
uses %q in a printf format in two places. Otherwise the diff makes sense
to me.

natano

Reply via email to