mfs: avoid flushing/syncing closed files.

Technically, the sync could fail. If it does, we don't want to allow syncing *again* without the write lock held. License: MIT Signed-off-by: Steven Allen <steven@stebalien.com>
ipfs · Dec 21, 2017 · 763d804 · 763d804
1 parent dadddcd
commit 763d804
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/mfs/fd.go b/mfs/fd.go
@@ -119,10 +119,16 @@ func (fi *fileDescriptor) Close() error {
 }
 
 func (fi *fileDescriptor) Sync() error {
+	if fi.state == stateClosed {
+		return ErrClosed
+	}
 	return fi.flushUp(false)
 }
 
 func (fi *fileDescriptor) Flush() error {
+	if fi.state == stateClosed {
+		return ErrClosed
+	}
 	return fi.flushUp(true)
 }