diff options
author | Michael Adam <obnox@samba.org> | 2013-02-13 16:51:54 +0100 |
---|---|---|
committer | Michael Adam <obnox@samba.org> | 2013-02-19 13:58:08 +0100 |
commit | 762dd3cf29788a9f8ab66fd96ad2b9cc20f621f3 (patch) | |
tree | bb4cf8fe357c68addcbb24b65c5adc88d0fb9869 | |
parent | 4d3928a8f494661764f4a3367b2f1b94772dedf9 (diff) | |
download | samba-762dd3cf29788a9f8ab66fd96ad2b9cc20f621f3.tar.gz samba-762dd3cf29788a9f8ab66fd96ad2b9cc20f621f3.tar.xz samba-762dd3cf29788a9f8ab66fd96ad2b9cc20f621f3.zip |
lib/util/time: strip a potential trailing newline in the asctime case.
If strftime() is not available, asctime() is used, and this usually
appends a newline character to the result. This is not desired for
timestamp().
Signed-off-by: Michael Adam <obnox@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
-rw-r--r-- | lib/util/time.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/lib/util/time.c b/lib/util/time.c index d5a429af94..56b2ec50de 100644 --- a/lib/util/time.c +++ b/lib/util/time.c @@ -450,6 +450,15 @@ _PUBLIC_ char *timestring(TALLOC_CTX *mem_ctx, time_t t) TimeBuf = talloc_strdup(mem_ctx, tempTime); #else TimeBuf = talloc_strdup(mem_ctx, asctime(tm)); + if (TimeBuf == NULL) { + return NULL; + } + if (TimeBuf[0] != '\0') { + size_t len = strlen(TimeBuf); + if (TimeBuf[len - 1] == '\n') { + TimeBuf[len - 1] = '\0'; + } + } #endif return TimeBuf; |