From 124ee8b1abb8f9d5b03eae20abb9405ce3fa5d7b Mon Sep 17 00:00:00 2001
From: Antoine Pitrou <solipsis@pitrou.net>
Date: Sun, 16 Dec 2012 13:55:47 +0100
Subject: [PATCH] Issue #16626: Fix infinite recursion in glob.glob() on
 Windows when the pattern contains a wildcard in the drive or UNC path. Patch
 by Serhiy Storchaka.

---
 Lib/glob.py           | 5 ++++-
 Lib/test/test_glob.py | 9 +++++++++
 Misc/NEWS             | 4 ++++
 3 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/Lib/glob.py b/Lib/glob.py
index d6ddaadc90d..0aee60561fa 100644
--- a/Lib/glob.py
+++ b/Lib/glob.py
@@ -38,7 +38,10 @@ def iglob(pathname):
         for name in glob1(os.curdir, basename):
             yield name
         return
-    if has_magic(dirname):
+    # `os.path.split()` returns the argument itself as a dirname if it is a
+    # drive or UNC path.  Prevent an infinite recursion if a drive or UNC path
+    # contains magic characters (i.e. r'\\?\C:').
+    if dirname != pathname and has_magic(dirname):
         dirs = iglob(dirname)
     else:
         dirs = [dirname]
diff --git a/Lib/test/test_glob.py b/Lib/test/test_glob.py
index 285c1690c42..fee3a60a7e6 100644
--- a/Lib/test/test_glob.py
+++ b/Lib/test/test_glob.py
@@ -3,6 +3,7 @@ from test.test_support import run_unittest, TESTFN
 import glob
 import os
 import shutil
+import sys
 
 
 class GlobTests(unittest.TestCase):
@@ -110,6 +111,14 @@ class GlobTests(unittest.TestCase):
             eq(self.glob('sym1'), [self.norm('sym1')])
             eq(self.glob('sym2'), [self.norm('sym2')])
 
+    @unittest.skipUnless(sys.platform == "win32", "Win32 specific test")
+    def test_glob_magic_in_drive(self):
+        eq = self.assertSequencesEqual_noorder
+        eq(glob.glob('*:'), [])
+        eq(glob.glob(u'*:'), [])
+        eq(glob.glob('?:'), [])
+        eq(glob.glob(u'?:'), [])
+
 
 def test_main():
     run_unittest(GlobTests)
diff --git a/Misc/NEWS b/Misc/NEWS
index 44b9b06676b..605a7a8f0fb 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -160,6 +160,10 @@ Core and Builtins
 Library
 -------
 
+- Issue #16626: Fix infinite recursion in glob.glob() on Windows when the
+  pattern contains a wildcard in the drive or UNC path.  Patch by Serhiy
+  Storchaka.
+
 - Issue #16298: In HTTPResponse.read(), close the socket when there is no
   Content-Length and the incoming stream is finished.  Patch by Eran
   Rundstein.