Issue #2643: msync() is not called anymore when deallocating an open mmap

object, only munmap().

diff --git a/Misc/NEWS b/Misc/NEWS
index 139ab0188fe6087395b30196340b095d165081f8..cdd9ef7060c40b9b262a740e6a99b751d4634cec 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -58,6 +58,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #2643: msync() is not called anymore when deallocating an open mmap
+  object, only munmap().
+
 - logging: Changed LoggerAdapter implementation internally, to make it
   easier to subclass in a useful way.
 

diff --git a/Modules/mmapmodule.c b/Modules/mmapmodule.c
index 8c5c8ac80a00b090d3c0eeabcaab95fa9622f391..3413f17ebd3cdb766e10532197e7bee1bd5f55c5 100644 (file)
--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -125,7 +125,6 @@ mmap_object_dealloc(mmap_object *m_obj)
     if (m_obj->fd >= 0)
         (void) close(m_obj->fd);
     if (m_obj->data!=NULL) {
-        msync(m_obj->data, m_obj->size, MS_SYNC);
         munmap(m_obj->data, m_obj->size);
     }
 #endif /* UNIX */