[PATCH] Re: wrong timestamp in log files

Martin Schwenke martin at meltin.net
Thu Dec 11 21:24:10 MST 2014


On Fri, 12 Dec 2014 11:41:40 +1100, Martin Schwenke <martin at meltin.net>
wrote:

> On Thu, 11 Dec 2014 17:48:36 +0000, Youzhong Yang
> <Youzhong.Yang at mathworks.com> wrote:
> 
> > I saw the timestamp in log files having the wrong month. Looking at the commit, shouldn't the tm->tm_mon be tm->tm_mon + 1?
> > 
> > Thanks,
> > 
> > -Youzhong Yang
> > 
> > ========================================================================
> > commit 1c2ae58509a19fb4590f4c71277c0f0c0a27f2fd
> > Author: Martin Schwenke <martin at meltin.net>
> > Date:   Wed Oct 8 15:52:37 2014 +1100
> > 
> >     lib/util: Use snprintf() instead of strftime() in timeval_str_buf()
> 
> Thanks, Youzhong!  I fixed this once during development but must have
> missed it when the code was refactored into timeval_str_buf().
> 
> Patch attached.  Can someone please review and push?  We need this in
> 4.2.

The logging changes aren't in 4.2, so it isn't needed.

Sorry, I'm having a bad day...  :-(

peace & happiness,
martin


More information about the samba-technical mailing list