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 <[email protected]>
ipfs · Dec 21, 2017 · ed9a405 · ed9a405
1 parent feafd11
commit ed9a405
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)
 }