[Gnucash-changes] r13282 - gnucash/trunk - Don't pass NULL to ferror() (from bug #143720).

[Derek Atkins]

Author: warlord
Date: 2006-02-17 16:31:29 -0500 (Fri, 17 Feb 2006)
New Revision: 13282
Trac: http://svn.gnucash.org/trac/changeset/13282

Modified:
   gnucash/trunk/ChangeLog
   gnucash/trunk/src/import-export/log-replay/gnc-log-replay.c
Log:
Don't pass NULL to ferror() (from bug #143720).



Modified: gnucash/trunk/ChangeLog
===================================================================
--- gnucash/trunk/ChangeLog	2006-02-17 20:22:54 UTC (rev 13281)
+++ gnucash/trunk/ChangeLog	2006-02-17 21:31:29 UTC (rev 13282)
@@ -33,6 +33,9 @@
           so it's possible to run from it.
 	  Also fix "fin.scm installed twice", bug #168629.
 
+	* src/import-export/log-replay/gnc-log-replay.c:
+	  Don't pass NULL to ferror() (from bug #143720).
+
 2006-02-15  Derek Atkins <derek at ihtfp.com>
 
 	* configure.in:  get --disable-gui to actually do what we want.

Modified: gnucash/trunk/src/import-export/log-replay/gnc-log-replay.c
===================================================================
--- gnucash/trunk/src/import-export/log-replay/gnc-log-replay.c	2006-02-17 20:22:54 UTC (rev 13281)
+++ gnucash/trunk/src/import-export/log-replay/gnc-log-replay.c	2006-02-17 21:31:29 UTC (rev 13282)
@@ -523,7 +523,7 @@
 
       DEBUG("Opening selected file");
       log_file = fopen(selected_filename, "r");
-      if(ferror(log_file)!=0)
+      if(!log_file || ferror(log_file)!=0)
 	{
 	  perror("File open failed");
 	}