Skip to content

Commit

Permalink
[build/cpython-defs] Add marshal.load() to the blacklist too
Browse files Browse the repository at this point in the history
We're using yajl.load() now.
  • Loading branch information
Andy Chu committed Dec 6, 2019
1 parent 5eeb91f commit e5c2cf5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions build/cpython_defs.py
Expand Up @@ -302,8 +302,8 @@ def __call__(self, rel_path, def_name, method_name):
# - Do we need __sizeof__? Is that for sys.getsizeof()?

# NOTE: LoadOilGrammar needs marshal.loads().
# False positive for yajl.dumps()
if basename == 'marshal.c' and method_name in ('dump', 'dumps'):
# False positive for yajl.dumps() and load()
if basename == 'marshal.c' and method_name in ('dump', 'dumps', 'load'):
return False

# Auto-filtering gave false-positives here.
Expand Down

0 comments on commit e5c2cf5

Please sign in to comment.