[Python-checkins] r72888 - in python/branches/release26-maint: Lib/test/test_fcntl.py Misc/NEWS Modules/fcntlmodule.c

Sun May 24 17:41:43 CEST 2009

Author: antoine.pitrou
Date: Sun May 24 17:41:43 2009
New Revision: 72888
Log:
Merged revisions 72887 via svnmerge from 
svn+ssh://pythondev@svn.python.org/python/trunk
........
 r72887 | antoine.pitrou | 2009年05月24日 17:40:09 +0200 (dim., 24 mai 2009) | 6 lines
 
 Issue #1309352: fcntl now converts its third arguments to a C `long` rather
 than an int, which makes some operations possible under 64-bit Linux (e.g.
 DN_MULTISHOT with F_NOTIFY).
........
Modified:
 python/branches/release26-maint/ (props changed)
 python/branches/release26-maint/Lib/test/test_fcntl.py
 python/branches/release26-maint/Misc/NEWS
 python/branches/release26-maint/Modules/fcntlmodule.c
Modified: python/branches/release26-maint/Lib/test/test_fcntl.py
==============================================================================

--- python/branches/release26-maint/Lib/test/test_fcntl.py	(original)
+++ python/branches/release26-maint/Lib/test/test_fcntl.py	Sun May 24 17:41:43 2009
@@ -57,7 +57,7 @@
 self.f = None
 
 def tearDown(self):
- if not self.f.closed:
+ if self.f and not self.f.closed:
 self.f.close()
 unlink(TESTFN)
 
@@ -81,6 +81,21 @@
 rv = fcntl.fcntl(self.f, fcntl.F_SETLKW, lockdata)
 self.f.close()
 
+ def test_fcntl_64_bit(self):
+ # Issue #1309352: fcntl shouldn't fail when the third arg fits in a
+ # C 'long' but not in a C 'int'.
+ try:
+ cmd = fcntl.F_NOTIFY
+ # This flag is larger than 2**31 in 64-bit builds
+ flags = fcntl.DN_MULTISHOT
+ except AttributeError:
+ self.skipTest("F_NOTIFY or DN_MULTISHOT unavailable")
+ fd = os.open(os.path.dirname(os.path.abspath(TESTFN)), os.O_RDONLY)
+ try:
+ fcntl.fcntl(fd, cmd, flags)
+ finally:
+ os.close(fd)
+
 
 def test_main():
 run_unittest(TestFcntl)
Modified: python/branches/release26-maint/Misc/NEWS
==============================================================================
--- python/branches/release26-maint/Misc/NEWS	(original)
+++ python/branches/release26-maint/Misc/NEWS	Sun May 24 17:41:43 2009
@@ -50,6 +50,10 @@
 Library
 -------
 
+- Issue #1309352: fcntl now converts its third arguments to a C `long` rather
+ than an int, which makes some operations possible under 64-bit Linux (e.g.
+ DN_MULTISHOT with F_NOTIFY).
+
 - Issue #1983: Fix functions taking or returning a process identifier to use
 the dedicated C type ``pid_t`` instead of a C ``int``. Some platforms have
 a process identifier type wider than the standard C integer type.
Modified: python/branches/release26-maint/Modules/fcntlmodule.c
==============================================================================
--- python/branches/release26-maint/Modules/fcntlmodule.c	(original)
+++ python/branches/release26-maint/Modules/fcntlmodule.c	Sun May 24 17:41:43 2009
@@ -34,7 +34,7 @@
 {
 	int fd;
 	int code;
-	int arg;
+	long arg;
 	int ret;
 	char *str;
 	Py_ssize_t len;
@@ -61,7 +61,7 @@
 	PyErr_Clear();
 	arg = 0;
 	if (!PyArg_ParseTuple(args,
- "O&i|i;fcntl requires a file or file descriptor,"
+ "O&i|l;fcntl requires a file or file descriptor,"
 " an integer and optionally a third integer or a string", 
 			 conv_descriptor, &fd, &code, &arg)) {
 	 return NULL;
