From: Benjamin Peterson <benjamin@python.org>
Date: Mon, 25 Jan 2010 03:31:13 +0000 (+0000)
Subject: fix an UnboundLocalError when the release file is empty #7773
X-Git-Tag: v2.7a3~92
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a43f34cc2a7a55cea794ecb9bda2eea5c685d1f3;p=python

fix an UnboundLocalError when the release file is empty #7773
---

diff --git a/Lib/platform.py b/Lib/platform.py
index 5df1e2305e..829d5f0fd5 100755
--- a/Lib/platform.py
+++ b/Lib/platform.py
@@ -263,6 +263,12 @@ _supported_dists = (
 
 def _parse_release_file(firstline):
 
+    # Default to empty 'version' and 'id' strings.  Both defaults are used
+    # when 'firstline' is empty.  'id' defaults to empty when an id can not
+    # be deduced.
+    version = ''
+    id = ''
+
     # Parse the first line
     m = _lsb_release_version.match(firstline)
     if m is not None:
@@ -280,8 +286,6 @@ def _parse_release_file(firstline):
         version = l[0]
         if len(l) > 1:
             id = l[1]
-        else:
-            id = ''
     return '', version, id
 
 def linux_distribution(distname='', version='', id='',
diff --git a/Lib/test/test_platform.py b/Lib/test/test_platform.py
index bd3b234b8e..b51225e616 100644
--- a/Lib/test/test_platform.py
+++ b/Lib/test/test_platform.py
@@ -198,6 +198,7 @@ class PlatformTest(unittest.TestCase):
             ('Red Hat Enterprise Linux release 4 (Nahant)', ('Red Hat Enterprise Linux', '4', 'Nahant')),
             ('CentOS release 4', ('CentOS', '4', None)),
             ('Rocks release 4.2.1 (Cydonia)', ('Rocks', '4.2.1', 'Cydonia')),
+            ('', ('', '', '')), # If there's nothing there.
             ):
             self.assertEqual(platform._parse_release_file(input), output)
 
diff --git a/Misc/ACKS b/Misc/ACKS
index f6df409b85..7c98382b7e 100644
--- a/Misc/ACKS
+++ b/Misc/ACKS
@@ -357,6 +357,7 @@ Jeremy Hylton
 Gerhard Häring
 Mihai Ibanescu
 Lars Immisch
+Meador Inge
 Tony Ingraldi
 John Interrante
 Bob Ippolito
diff --git a/Misc/NEWS b/Misc/NEWS
index f1ab6880b8..3dac0ea087 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -42,6 +42,9 @@ Core and Builtins
 Library
 -------
 
+- Issue #7773: Fix an UnboundLocalError in platform.linux_distribution() when
+  the release file is empty.
+
 - Issue #7748: Since unicode values are supported for some metadata options
   in Distutils, the DistributionMetadata get_* methods will now return an utf-8
   encoded string for them. This ensure that the upload and register commands