Browse Source

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.

David Cole 19 years ago
parent
commit
76d05cd5fa
1 changed files with 2 additions and 2 deletions
  1. 2 2
      Source/kwsys/SystemTools.cxx

+ 2 - 2
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;
     }