Merge #16: [LevelDB] Do no crash if filesystem can't fsync

laanwj · laanwj · commit 0d969fd5708c · 2017-03-21T11:48:39.000+01:00
c8c029b [LevelDB] Do no crash if filesystem can't fsync (Nicolas Dorier)

Tree-SHA512: 26657326cef26cac28401f0c770e201dcb65cd1cf867f3f6e2149999c96881d80604b90caf3506d8425c46ebc746716473d4a2e8ade8638a08e02d31f6184b48
diff --git a/util/env_posix.cc b/util/env_posix.cc
@@ -231,7 +231,7 @@ class PosixWritableFile : public WritableFile {
       if (fd < 0) {
         s = IOError(dir, errno);
       } else {
-        if (fsync(fd) < 0) {
+        if (fsync(fd) < 0 && errno != EINVAL) {
           s = IOError(dir, errno);
         }
         close(fd);

Original file line number	Diff line number	Diff line change
`@@ -231,7 +231,7 @@ class PosixWritableFile : public WritableFile {`
`231`	`231`	`if (fd < 0) {`
`232`	`232`	`s = IOError(dir, errno);`
`233`	`233`	`} else {`
`234`		`- if (fsync(fd) < 0) {`
	`234`	`+ if (fsync(fd) < 0 && errno != EINVAL) {`
`235`	`235`	`s = IOError(dir, errno);`
`236`	`236`	`}`
`237`	`237`	`close(fd);`