mirror of
https://git.yoctoproject.org/poky
synced 2026-02-23 09:59:39 +01:00
016a0b830e
Source: OpenEmbedded.org MR: 98320, 98319 Type: Security Fix Disposition: Backport from https://git.openembedded.org/openembedded-core/commit/meta/recipes-devtools/python/python_2.7.16.bb?id=9d23b982fa4e0290761b3d15f6959779fed72ad6 ChangeID: e79b6fe3b7b4253bf0d76b029070ae869d5234bd Description: Fixes: CVE-2019-9948 CVE-2019-9636 CVE-2019-9940 is a dup of 9948 per python.org CVE-2019-9947 appears to be a dup of 9940 per https://bugs.python.org/issue30458#msg295067 (From OE-Core rev: e7bdff05da6075efc21c5ac9492b06e481e5a239) Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org> [Minor clean up for thud] Signed-off-by: Armin Kuster <akuster@mvista.com> Signed-off-by: Armin Kuster <akuster808@gmail.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
56 lines
2.1 KiB
Diff
56 lines
2.1 KiB
Diff
From 8f99cc799e4393bf1112b9395b2342f81b3f45ef Mon Sep 17 00:00:00 2001
|
|
From: push0ebp <push0ebp@shl-MacBook-Pro.local>
|
|
Date: Thu, 14 Feb 2019 02:05:46 +0900
|
|
Subject: [PATCH] bpo-35907: Avoid file reading as disallowing the unnecessary
|
|
URL scheme in urllib
|
|
|
|
Upstream-Status: Submitted https://github.com/python/cpython/pull/11842
|
|
|
|
CVE: CVE-2019-9948
|
|
|
|
Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
|
|
---
|
|
Lib/test/test_urllib.py | 12 ++++++++++++
|
|
Lib/urllib.py | 5 ++++-
|
|
2 files changed, 16 insertions(+), 1 deletion(-)
|
|
|
|
diff --git a/Lib/test/test_urllib.py b/Lib/test/test_urllib.py
|
|
index 1ce9201c0693..e5f210e62a18 100644
|
|
--- a/Lib/test/test_urllib.py
|
|
+++ b/Lib/test/test_urllib.py
|
|
@@ -1023,6 +1023,18 @@ def open_spam(self, url):
|
|
"spam://c:|windows%/:=&?~#+!$,;'@()*[]|/path/"),
|
|
"//c:|windows%/:=&?~#+!$,;'@()*[]|/path/")
|
|
|
|
+ def test_local_file_open(self):
|
|
+ class DummyURLopener(urllib.URLopener):
|
|
+ def open_local_file(self, url):
|
|
+ return url
|
|
+ self.assertEqual(DummyURLopener().open(
|
|
+ 'local-file://example'), '//example')
|
|
+ self.assertEqual(DummyURLopener().open(
|
|
+ 'local_file://example'), '//example')
|
|
+ self.assertRaises(IOError, urllib.urlopen,
|
|
+ 'local-file://example')
|
|
+ self.assertRaises(IOError, urllib.urlopen,
|
|
+ 'local_file://example')
|
|
|
|
# Just commented them out.
|
|
# Can't really tell why keep failing in windows and sparc.
|
|
diff --git a/Lib/urllib.py b/Lib/urllib.py
|
|
index d85504a5cb7e..a24e9a5c68fb 100644
|
|
--- a/Lib/urllib.py
|
|
+++ b/Lib/urllib.py
|
|
@@ -203,7 +203,10 @@ def open(self, fullurl, data=None):
|
|
name = 'open_' + urltype
|
|
self.type = urltype
|
|
name = name.replace('-', '_')
|
|
- if not hasattr(self, name):
|
|
+
|
|
+ # bpo-35907: # disallow the file reading with the type not allowed
|
|
+ if not hasattr(self, name) or \
|
|
+ (self == _urlopener and name == 'open_local_file'):
|
|
if proxy:
|
|
return self.open_unknown_proxy(proxy, fullurl, data)
|
|
else:
|