Bug #1344508, Fix UNIX mmap leaking file descriptors. Will backport.

7b631793 · Neal Norwitz · 245a5ab3 · 7b631793 · 7b631793
Kaydet (Commit) 7b631793 authored Kas 02, 2005 tarafından Neal Norwitz
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

NEWS Misc/NEWS +2 -0

mmapmodule.c Modules/mmapmodule.c +1 -0

No files found.
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -177,6 +177,8 @@ Core and builtins
 Extension Modules
 -----------------

+- Bug #1344508, Fix UNIX mmap leaking file descriptors
+
 - Patch #1338314, Bug #1336623: fix tarfile so it can extract
  REGTYPE directories from tarfiles written by old programs.


--- a/Modules/mmapmodule.c
+++ b/Modules/mmapmodule.c
@@ -99,6 +99,7 @@ mmap_object_dealloc(mmap_object *m_obj)
 #endif /* MS_WINDOWS */

 #ifdef UNIX
+	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);