Fix matrix pointer dereference bug in update.c
authorErik Lindahl <erik@kth.se>
Tue, 12 Aug 2014 18:11:40 +0000 (20:11 +0200)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Wed, 20 Aug 2014 16:21:32 +0000 (18:21 +0200)
The PGI compiler found a bug where an if-statement
checked against a dereferenced pointer rather than
the pointer itself being non-NULL.

Change-Id: If08977b818f28e1c78c35222d0f4efe6ae6aad65

src/gromacs/mdlib/update.c

index e5a095baace0f339528a1f6cf3ccc8e77373cc0b..787135f7158cd12b23bac96a7446d7de8d8e44a7 100644 (file)
@@ -1318,7 +1318,7 @@ static void deform(gmx_update_t upd,
         x[i][YY] = mu[YY][YY]*x[i][YY]+mu[ZZ][YY]*x[i][ZZ];
         x[i][ZZ] = mu[ZZ][ZZ]*x[i][ZZ];
     }
-    if (*scale_tot)
+    if (scale_tot != NULL)
     {
         /* The transposes of the scaling matrices are stored,
          * so we need to do matrix multiplication in the inverse order.