diff options
author | Peter Stephenson <pws@users.sourceforge.net> | 2004-05-04 16:43:29 +0000 |
---|---|---|
committer | Peter Stephenson <pws@users.sourceforge.net> | 2004-05-04 16:43:29 +0000 |
commit | f57f65282b8b7de7db638eaabd06867ea8b1162f (patch) | |
tree | 64bb58c0fe67fc88c212f6f7fc1b7260400da2e2 /Src/utils.c | |
parent | c98b9dc800171365eee34a3adbf5a741dbef69ca (diff) | |
download | zsh-f57f65282b8b7de7db638eaabd06867ea8b1162f.tar.gz zsh-f57f65282b8b7de7db638eaabd06867ea8b1162f.tar.xz zsh-f57f65282b8b7de7db638eaabd06867ea8b1162f.zip |
19877: improved test for empty strftime strings
Diffstat (limited to 'Src/utils.c')
-rw-r--r-- | Src/utils.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/Src/utils.c b/Src/utils.c index c0a96a5b4..ee78665c1 100644 --- a/Src/utils.c +++ b/Src/utils.c @@ -1719,6 +1719,11 @@ ztrftimebuf(int *bufsizeptr, int decr) * Like the system function, this returns the number of characters * copied, not including the terminating NUL. This may be zero * if the string didn't fit. + * + * As an extension, try to detect an error in strftime --- typically + * not enough memory --- and return -1. Not guaranteed to be portable, + * since the strftime() interface doesn't make any guarantees about + * the state of the buffer if it returns zero. */ /**/ @@ -1831,9 +1836,14 @@ ztrftime(char *buf, int bufsize, char *fmt, struct tm *tm) */ *buf = '\0'; tmp[1] = fmt[-1]; - if (!strftime(buf, bufsize + 2, tmp, tm) && - tmp[1]!='p' && tmp[1]!='P') + if (!strftime(buf, bufsize + 2, tmp, tm)) + { + if (*buf) { + buf[0] = '\0'; + return -1; + } return 0; + } decr = strlen(buf); buf += decr; bufsize -= decr - 2; |