On Sun, Dec 29, 2002 at 09:17:05PM -0800, David Schultz wrote:
> Thus spake Kris Kennaway <[EMAIL PROTECTED]>:
> > On Mon, Dec 30, 2002 at 03:21:22AM +0000, Mike Barcroft wrote:
> > 
> > > ===> sbin/swapon
> > > cc1: warnings being treated as errors
> > > /tinderbox/sparc64/src/sbin/swapon/swapon.c: In function `swaplist':
> > > /tinderbox/sparc64/src/sbin/swapon/swapon.c:246: warning: field width is not 
>type int (arg 3)
> > 
> > Can someone please just fix this (by backing out the offending commit,
> > if necessary)?
> 
> Eek, given a 64-bit size_t, the present code leaves 32 bits of it
> uninitialized in the usual case.  The following patch ought to fix
> the problem; I can't make sure right now because I'm out of town.


I'm not sure if your patch will solve the problem.
The offending code is here:
    240         if (lflag) { 
    241                 char buf[32];
    242                 snprintf(buf, sizeof(buf), "%ld-blocks", blocksize);
    243                 printf("%-13s %*s %*s\n",
    244                     "Device:",
    245                     hlen, buf,
    246                     hlen, "Used:");
    247         }


Doesn't the printf() statement in question have the wrong number of
arguments?

-- 
Craig Rodrigues        
http://www.gis.net/~craigr    
[EMAIL PROTECTED]

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to