On Fri, Dec 13, 2013 at 5:06 AM, Daniel Lescohier <daniel.lescoh...@cbsi.com
> wrote:

> Here is my draft replacement:
>
> static const char *log_request_time_custom(request_rec *r, char *a,
>                                            apr_time_exp_t *xt)
> {
>     static const apr_size_t buf_len = 256;
>     apr_size_t tstr_len;
>     char *tstr = apr_palloc(r->pool, buf_len);
>
> 
> apr_strftime(tstr, &tstr_len, buf_len, a, xt);
>     return tstr_len ? tstr : "-";
> }
>

Shouldn't [MAX_STRING_LEN > buf_len > strlen(a)] for apr_strftime() not
to truncate the result (for any resonnable 'a')?

Reply via email to