moments: Fix null pointer dereference in calc_moments().
authorBen Pfaff <blp@gnu.org>
Thu, 11 Feb 2010 04:19:44 +0000 (20:19 -0800)
committerBen Pfaff <blp@gnu.org>
Sat, 20 Feb 2010 01:16:13 +0000 (17:16 -0800)
Found by Clang (http://clang-analyzer.llvm.org).

src/math/moments.c

index d129f6ab038b3eee06104e716b15fbcf432c61c5..545c9644399a9501765455cfc99505be30e8d460 100644 (file)
@@ -1,5 +1,5 @@
 /* PSPP - a program for statistical analysis.
-   Copyright (C) 1997-9, 2000 Free Software Foundation, Inc.
+   Copyright (C) 1997-9, 2000, 2010 Free Software Foundation, Inc.
 
    This program is free software: you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -49,7 +49,7 @@ calc_moments (enum moment max_moment,
 
       /* From _SPSS Statistical Algorithms, 2nd ed.,
          0-918469-89-9, section "DESCRIPTIVES". */
-      if (fabs (*variance) >= 1e-20)
+      if (fabs (s2) >= 1e-20)
         {
           if (max_moment >= MOMENT_SKEWNESS && skewness != NULL && w > 2.)
             {