--- a/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/blackListCalls.py Tue Sep 13 19:46:19 2022 +0200
+++ b/src/eric7/Plugins/CheckerPlugins/CodeStyleChecker/Security/Checks/blackListCalls.py Tue Sep 13 20:00:55 2022 +0200
@@ -17,6 +17,7 @@
import ast
import fnmatch
+import sys
import AstUtilities
@@ -38,9 +39,29 @@
"M",
),
"S302": (["marshal.load", "marshal.loads"], "M"),
- "S303": (
+}
+if sys.version_info >= (3, 9):
+ _blacklists["S303"] = (
[
+ "Crypto.Hash.MD2.new",
+ "Crypto.Hash.MD4.new",
+ "Crypto.Hash.MD5.new",
+ "Crypto.Hash.SHA.new",
+ "Cryptodome.Hash.MD2.new",
+ "Cryptodome.Hash.MD4.new",
+ "Cryptodome.Hash.MD5.new",
+ "Cryptodome.Hash.SHA.new",
+ "cryptography.hazmat.primitives.hashes.MD5",
+ "cryptography.hazmat.primitives.hashes.SHA1",
+ ],
+ "M",
+ )
+else:
+ _blacklists["S303"] = (
+ [
+ "hashlib.md4",
"hashlib.md5",
+ "hashlib.sha",
"hashlib.sha1",
"Crypto.Hash.MD2.new",
"Crypto.Hash.MD4.new",
@@ -54,107 +75,115 @@
"cryptography.hazmat.primitives.hashes.SHA1",
],
"M",
- ),
- "S304": (
- [
- "Crypto.Cipher.ARC2.new",
- "Crypto.Cipher.ARC4.new",
- "Crypto.Cipher.Blowfish.new",
- "Crypto.Cipher.DES.new",
- "Crypto.Cipher.XOR.new",
- "Cryptodome.Cipher.ARC2.new",
- "Cryptodome.Cipher.ARC4.new",
- "Cryptodome.Cipher.Blowfish.new",
- "Cryptodome.Cipher.DES.new",
- "Cryptodome.Cipher.XOR.new",
- "cryptography.hazmat.primitives.ciphers.algorithms.ARC4",
- "cryptography.hazmat.primitives.ciphers.algorithms.Blowfish",
- "cryptography.hazmat.primitives.ciphers.algorithms.IDEA",
- ],
- "H",
- ),
- "S305": (["cryptography.hazmat.primitives.ciphers.modes.ECB"], "M"),
- "S306": (["tempfile.mktemp"], "M"),
- "S307": (["eval"], "M"),
- "S308": (["django.utils.safestring.mark_safe"], "M"),
- "S309": (
- [
- "httplib.HTTPSConnection",
- "http.client.HTTPSConnection",
- "six.moves.http_client.HTTPSConnection",
- ],
- "M",
- ),
- "S310": (
- [
- "urllib.urlopen",
- "urllib.request.urlopen",
- "urllib.urlretrieve",
- "urllib.request.urlretrieve",
- "urllib.URLopener",
- "urllib.request.URLopener",
- "urllib.FancyURLopener",
- "urllib.request.FancyURLopener",
- "urllib2.urlopen",
- "urllib2.Request",
- "six.moves.urllib.request.urlopen",
- "six.moves.urllib.request.urlretrieve",
- "six.moves.urllib.request.URLopener",
- "six.moves.urllib.request.FancyURLopener",
- ],
- "",
- ),
- "S311": (
- [
- "random.random",
- "random.randrange",
- "random.randint",
- "random.choice",
- "random.uniform",
- "random.triangular",
- ],
- "L",
- ),
- "S312": (["telnetlib.*"], "H"),
- "S313": (
- [
- "xml.etree.cElementTree.parse",
- "xml.etree.cElementTree.iterparse",
- "xml.etree.cElementTree.fromstring",
- "xml.etree.cElementTree.XMLParser",
- ],
- "M",
- ),
- "S314": (
- [
- "xml.etree.ElementTree.parse",
- "xml.etree.ElementTree.iterparse",
- "xml.etree.ElementTree.fromstring",
- "xml.etree.ElementTree.XMLParser",
- ],
- "M",
- ),
- "S315": (["xml.sax.expatreader.create_parser"], "M"),
- "S316": (["xml.dom.expatbuilder.parse", "xml.dom.expatbuilder.parseString"], "M"),
- "S317": (["xml.sax.parse", "xml.sax.parseString", "xml.sax.make_parser"], "M"),
- "S318": (["xml.dom.minidom.parse", "xml.dom.minidom.parseString"], "M"),
- "S319": (["xml.dom.pulldom.parse", "xml.dom.pulldom.parseString"], "M"),
- "S320": (
- [
- "lxml.etree.parse",
- "lxml.etree.fromstring",
- "lxml.etree.RestrictedElement",
- "lxml.etree.GlobalParserTLS",
- "lxml.etree.getDefaultParser",
- "lxml.etree.check_docinfo",
- ],
- "M",
- ),
- "S321": (["ftplib.*"], "H"),
- "S322": (["input"], "H"),
- "S323": (["ssl._create_unverified_context"], "M"),
- "S324": (["os.tempnam", "os.tmpnam"], "M"),
-}
+ )
+
+_blacklists.update(
+ {
+ "S304": (
+ [
+ "Crypto.Cipher.ARC2.new",
+ "Crypto.Cipher.ARC4.new",
+ "Crypto.Cipher.Blowfish.new",
+ "Crypto.Cipher.DES.new",
+ "Crypto.Cipher.XOR.new",
+ "Cryptodome.Cipher.ARC2.new",
+ "Cryptodome.Cipher.ARC4.new",
+ "Cryptodome.Cipher.Blowfish.new",
+ "Cryptodome.Cipher.DES.new",
+ "Cryptodome.Cipher.XOR.new",
+ "cryptography.hazmat.primitives.ciphers.algorithms.ARC4",
+ "cryptography.hazmat.primitives.ciphers.algorithms.Blowfish",
+ "cryptography.hazmat.primitives.ciphers.algorithms.IDEA",
+ ],
+ "H",
+ ),
+ "S305": (["cryptography.hazmat.primitives.ciphers.modes.ECB"], "M"),
+ "S306": (["tempfile.mktemp"], "M"),
+ "S307": (["eval"], "M"),
+ "S308": (["django.utils.safestring.mark_safe"], "M"),
+ "S309": (
+ [
+ "httplib.HTTPSConnection",
+ "http.client.HTTPSConnection",
+ "six.moves.http_client.HTTPSConnection",
+ ],
+ "M",
+ ),
+ "S310": (
+ [
+ "urllib.urlopen",
+ "urllib.request.urlopen",
+ "urllib.urlretrieve",
+ "urllib.request.urlretrieve",
+ "urllib.URLopener",
+ "urllib.request.URLopener",
+ "urllib.FancyURLopener",
+ "urllib.request.FancyURLopener",
+ "urllib2.urlopen",
+ "urllib2.Request",
+ "six.moves.urllib.request.urlopen",
+ "six.moves.urllib.request.urlretrieve",
+ "six.moves.urllib.request.URLopener",
+ "six.moves.urllib.request.FancyURLopener",
+ ],
+ "",
+ ),
+ "S311": (
+ [
+ "random.random",
+ "random.randrange",
+ "random.randint",
+ "random.choice",
+ "random.choices",
+ "random.uniform",
+ "random.triangular",
+ ],
+ "L",
+ ),
+ "S312": (["telnetlib.*"], "H"),
+ "S313": (
+ [
+ "xml.etree.cElementTree.parse",
+ "xml.etree.cElementTree.iterparse",
+ "xml.etree.cElementTree.fromstring",
+ "xml.etree.cElementTree.XMLParser",
+ ],
+ "M",
+ ),
+ "S314": (
+ [
+ "xml.etree.ElementTree.parse",
+ "xml.etree.ElementTree.iterparse",
+ "xml.etree.ElementTree.fromstring",
+ "xml.etree.ElementTree.XMLParser",
+ ],
+ "M",
+ ),
+ "S315": (["xml.sax.expatreader.create_parser"], "M"),
+ "S316": (
+ ["xml.dom.expatbuilder.parse", "xml.dom.expatbuilder.parseString"],
+ "M",
+ ),
+ "S317": (["xml.sax.parse", "xml.sax.parseString", "xml.sax.make_parser"], "M"),
+ "S318": (["xml.dom.minidom.parse", "xml.dom.minidom.parseString"], "M"),
+ "S319": (["xml.dom.pulldom.parse", "xml.dom.pulldom.parseString"], "M"),
+ "S320": (
+ [
+ "lxml.etree.parse",
+ "lxml.etree.fromstring",
+ "lxml.etree.RestrictedElement",
+ "lxml.etree.GlobalParserTLS",
+ "lxml.etree.getDefaultParser",
+ "lxml.etree.check_docinfo",
+ ],
+ "M",
+ ),
+ "S321": (["ftplib.*"], "H"),
+ "S322": (["input"], "H"),
+ "S323": (["ssl._create_unverified_context"], "M"),
+ "S324": (["os.tempnam", "os.tmpnam"], "M"),
+ }
+)
def getChecks():
