Fix memory error in asc format IO
authorRoland Schulz <roland@utk.edu>
Tue, 22 Apr 2014 19:51:29 +0000 (15:51 -0400)
committerRoland Schulz <roland@utk.edu>
Tue, 22 Apr 2014 19:51:29 +0000 (15:51 -0400)
Found with cppcheck 1.64

Change-Id: I72e80cd2b11559be47d449d0a8d444857843979b

src/gmxlib/gmxfio_asc.c

index ef9870dd6c2f3e63fb50d92f12d72d9a6952c937..0e38211c45c3744037ad381eefa446b2aa1503c4 100644 (file)
@@ -108,7 +108,7 @@ static void encode_string(int maxlen, char dst[], const char src[])
 {
     int i;
 
-    for (i = 0; (src[i] != '\0') && (i < maxlen - 1); i++)
+    for (i = 0; (i < maxlen - 1) && (src[i] != '\0'); i++)
     {
         if ((src[i] == ' ') || (src[i] == '\t'))
         {
@@ -131,7 +131,7 @@ static void decode_string(int maxlen, char dst[], const char src[])
 {
     int i;
 
-    for (i = 0; (src[i] != '\0') && (i < maxlen - 1); i++)
+    for (i = 0; (i < maxlen - 1) && (src[i] != '\0'); i++)
     {
         if (src[i] == '_')
         {