From 76d05cd5fa34e2a54f91f3388fe2849064c05b5e Mon Sep 17 00:00:00 2001 From: David Cole Date: Thu, 10 Aug 2006 15:17:53 -0400 Subject: [PATCH] BUG: strlen logic was backwards resulting in function body never actually executing... when called with valid strings, it was always doing nothing and returning false... now it works as expected. --- Source/kwsys/SystemTools.cxx | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Source/kwsys/SystemTools.cxx b/Source/kwsys/SystemTools.cxx index bd824d37e..7c0242c5c 100644 --- a/Source/kwsys/SystemTools.cxx +++ b/Source/kwsys/SystemTools.cxx @@ -1764,7 +1764,7 @@ long int SystemTools::CreationTime(const char* filename) bool SystemTools::ConvertDateMacroString(const char *str, time_t *tmt) { - if (!str || !tmt || strlen(str) < 12) + if (!str || !tmt || strlen(str) > 11) { return false; } @@ -1812,7 +1812,7 @@ bool SystemTools::ConvertDateMacroString(const char *str, time_t *tmt) bool SystemTools::ConvertTimeStampMacroString(const char *str, time_t *tmt) { - if (!str || !tmt || strlen(str) < 27) + if (!str || !tmt || strlen(str) > 26) { return false; }