Merge lp:~ev/apport/fsync-upstart-crashes into lp:~apport-hackers/apport/trunk

[Evan]

As discussed in the "Apport/Whoopsie and Upstart" email, we should fsync crashes of upstart.

[James Hunt (jamesodhunt)]

Looks good to me.

review: Approve

[Martin Pitt (pitti)]

Thanks, looks fine!

review: Approve

=== modified file 'data/apport'
--- data/apport	2012-07-12 15:11:48 +0000
+++ data/apport	2012-11-20 12:16:25 +0000
@@ -422,6 +422,17 @@
     try:
         info.write(reportfile)
         if reportfile != sys.stderr:
+            # Ensure that the file gets written to disk in the event of an
+            # Upstart crash.
+            if info.get('ExectuablePath', '') == '/sbin/init':
+                reportfile.flush()
+                os.fsync(reportfile.fileno())
+                parent_directory = os.path.dirname(report)
+                try:
+                    fd = os.open(parent_directory, os.O_RDONLY)
+                    os.fsync(fd)
+                finally:
+                    os.close(fd)
             reportfile.close()
     except IOError:
         if reportfile != sys.stderr: