diff Modules/mmapmodule.c @ 35821:b652f535201d legacy-trunk

Cleanup the previous checkin. Close the fd in the close method and invalidated it. Don't try to close a second time when deallocating.
author Neal Norwitz <nnorwitz@gmail.com>
date Wed, 02 Nov 2005 05:36:37 +0000
parents 69034a08e5a3
children 9541369b0dca
line wrap: on
line diff
--- a/Modules/mmapmodule.c	Wed Nov 02 05:26:07 2005 +0000
+++ b/Modules/mmapmodule.c	Wed Nov 02 05:36:37 2005 +0000
@@ -99,7 +99,8 @@
 #endif /* MS_WINDOWS */
 
 #ifdef UNIX
-	close(m_obj->fd);
+	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);
@@ -137,6 +138,8 @@
 #endif /* MS_WINDOWS */
 
 #ifdef UNIX
+	(void) close(self->fd);
+	self->fd = -1;
 	if (self->data != NULL) {
 		munmap(self->data, self->size);
 		self->data = NULL;