commit 134a67c1051e9c640ccbc396665a9a875f93defd
author Jaroslav Škarvada <jskarvad@redhat.com> Wed Nov 28 16:42:41 2012 +0100
committer Chris E Ferron <chris.e.ferron@linux.intel.com> Thu Nov 29 07:55:47 2012 -0800
tree bdeaf60f720eab6a85d1c31771d1c1e5ebcf1d71
parent 78956bdcd91c1e1c49e0462b64dda9c11a8ae53d

report: fixed crash when writing report to RO filesystem

Signed-off-by: Jaroslav Škarvada <jskarvad@redhat.com>

src/report/report.cpp

diff --git a/src/report/report.cpp b/src/report/report.cpp
index 51bc6a7..c018bb1 100644
--- a/src/report/report.cpp
+++ b/src/report/report.cpp
@@ -183,13 +183,16 @@
 
 void finish_report_output(void)
 {
-	fprintf(stderr, _("PowerTOP outputing using base filename %s\n"), reportout.filename);
 	if (reporttype == REPORT_OFF)
 		return;
 
 	report.finish_report();
-	fputs(report.get_result(), reportout.report_file);
-	fdatasync(fileno(reportout.report_file));
-	fclose(reportout.report_file);
+	if (reportout.report_file)
+	{
+		fprintf(stderr, _("PowerTOP outputing using base filename %s\n"), reportout.filename);
+		fputs(report.get_result(), reportout.report_file);
+		fdatasync(fileno(reportout.report_file));
+		fclose(reportout.report_file);
+	}
 	report.clear_result();
 }