gui: Fix theoretical comparison against garbage pointer.
authorBen Pfaff <blp@gnu.org>
Thu, 11 Feb 2010 04:26:15 +0000 (20:26 -0800)
committerBen Pfaff <blp@gnu.org>
Sat, 20 Feb 2010 01:16:14 +0000 (17:16 -0800)
Presumably gtk_tree_model_iter_n_children() never returns a negative value,
but in case it ever could, this eliminates a comparison against an
uninitialized 'v1' value.

Found by Clang (http://clang-analyzer.llvm.org).

src/ui/gui/t-test-paired-samples.c

index 95e212aff7e21e12e95912a9ab46f9fd3824b9a0..56de59fcc449562702ec6c5aa1b05e2206d88a9c 100644 (file)
@@ -1,5 +1,5 @@
 /* PSPPIRE - a graphical user interface for PSPP.
-   Copyright (C) 2008  Free Software Foundation
+   Copyright (C) 2008, 2010  Free Software Foundation
 
    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
@@ -133,6 +133,8 @@ select_as_pair_member (GtkTreeIter source_iter,
 
       gtk_tree_model_get (tt_d->list_store, &dest_iter, 1, &v1, -1);
     }
+  else
+    v1 = NULL;
 
   if ( n_rows == 0 || v1 != NULL)
     {