[REVIEW 3-5]: don't loose data on disk failure (fsync) fdo#40607

Michael Meeks michael.meeks at suse.com
Mon Mar 12 08:06:12 PDT 2012


On Mon, 2012-03-12 at 09:43 +0100, Stephan Bergmann wrote:
> As discussed on IRC, the patch breaks smoketest/adding extensions (and 
> my attempt at a quick fix was futile).

	Ah yes - it turns out that:

-                ::ucbhelper::Content ucbStamp( stampURL, xCmdEnv );
-                ::rtl::OString stamp(
-                    RTL_CONSTASCII_STRINGPARAM(CURRENT_STAMP) );
-                Reference<io::XInputStream> xData(
-                    ::xmlscript::createInputStream(
-                        ::rtl::ByteSequence(
-                            reinterpret_cast<sal_Int8 const *>(stamp.getStr()),
-                            stamp.getLength() ) ) );

	Creates a parent directory if it is not present, something not entirely
obvious when reading it ;-) Fixed on master.

	I attach a unified patch:
	
fdo#40607 - osl_syncFile having written, and avoid doing that on start

	Thanks,

		Michael.

-- 
michael.meeks at suse.com  <><, Pseudo Engineer, itinerant idiot
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fdo-40607-syncfile.patch
Type: text/x-patch
Size: 6506 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120312/1b684f8e/attachment.bin>


More information about the LibreOffice mailing list