[Sheepdog] [PATCH 8/9] sizeof problem: sizeof was applied to a global definition, rather then a local

[MORITA Kazutaka]

At Tue, 18 May 2010 09:53:19 +0900,
FUJITA Tomonori wrote:
> 
> On Mon, 17 May 2010 09:37:16 -0700
> Steven Dake <sdake at redhat.com> wrote:
> 
> > Have sizeof apply to proper local variable.
> > 
> > Signed-off-by: Steven Dake <sdake at redhat.com>
> > ---
> >  collie/collie.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/collie/collie.c b/collie/collie.c
> > index ce3d086..d9f26fc 100644
> > --- a/collie/collie.c
> > +++ b/collie/collie.c
> > @@ -919,7 +919,7 @@ static int cluster_info(int argc, char **argv)
> >  
> >  		ti = logs[i].ctime >> 32;
> >  		localtime_r(&ti, &tm);
> > -		strftime(time_str, sizeof(time), "%y-%m-%d %H:%M:%S", &tm);
> > +		strftime(time_str, sizeof(time_str), "%y-%m-%d %H:%M:%S", &tm);
> 
> I think that this patch needs to be fold into 6th patch (otherwise,
> 6th patch breaks 'cluster info' output).
> 

I've folded this patch into the 6th patch, and applied.

Thanks,

Kazutaka