X-Recipient:	archive-cygwin AT delorie DOT com
X-SWARE-Spam-Status:	No, hits=-1.7 required=5.0 tests=AWL,BAYES_00,T_RP_MATCHES_RCVD
X-Spam-Check-By:	sourceware.org
Date:	Wed, 15 Jun 2011 14:43:03 -0400 (EDT)
From:	Steve Thompson <smt AT vgersoft DOT com>
To:	cygwin AT cygwin DOT com
Subject:	Re: localtime
In-Reply-To:	<4DF8F426.7040705@gmail.com>
Message-ID:	<alpine.LRH.0.9999.1106151440190.8672@helios.vgersoft.com>
References:	<4DF8CF3D DOT 40104 AT gmail DOT com> <4DF8EA79 DOT 2060404 AT gmail DOT com> <20110615173847 DOT GB23078 AT ednor DOT casa DOT cgf DOT cx> <4DF8EF9F DOT 4090906 AT gmail DOT com> <20110615174805 DOT GC23078 AT ednor DOT casa DOT cgf DOT cx> <4DF8F426 DOT 7040705 AT gmail DOT com>
MIME-Version:	1.0
X-IsSubscribed:	yes
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Unsubscribe:	<mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com

On Wed, 15 Jun 2011, Tod wrote:

> I'm passing a 128 byte char array.  I allocated it to provide enough room for 
> the date/time stamp this function is returning.  strlen(tout) will resolve to 
> the length of the tout string.
>
> You said above that I shouldn't be using strlen(tout) and instead I should be 
> passing 128.  Would I be better off using sizeof(tout) instead?
>
> Also, the code has always worked.  I just recompiled it recently.  Now the 
> date works but the time isn't appearing.  What could be causing that?

tout is a char *, so sizeof(tout) in getTime() will be 4 or 8. You will 
need to change the calling sequence of getTime() to pass the length of 
tout, which will depend on how you allocated it:

 	char tout[128];
 	getTime(tout,sizeof(tout));

or
 	char *tout = malloc(128);
 	getTime(tout,128);

etc.

Steve

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

- Raw text -