Do not run the unsafe guarded access in prod

Reviewed By: peterogithub

Differential Revision: D3373791

fbshipit-source-id: 810aded
master
Jeremy Dubreil 9 years ago committed by Facebook Github Bot 1
parent 1dc636a971
commit 3a2a6c7c44

@ -59,7 +59,8 @@ ISSUE_TYPES = [
'REGISTERED_OBSERVER_BEING_DEALLOCATED', 'REGISTERED_OBSERVER_BEING_DEALLOCATED',
'ASSIGN_POINTER_WARNING', 'ASSIGN_POINTER_WARNING',
'GLOBAL_VARIABLE_INITIALIZED_WITH_FUNCTION_OR_METHOD_CALL', 'GLOBAL_VARIABLE_INITIALIZED_WITH_FUNCTION_OR_METHOD_CALL',
'UNSAFE_GUARDED_BY_ACCESS', # TODO (t11307776): Turn this back on once some of the FP issues are fixed
# 'UNSAFE_GUARDED_BY_ACCESS',
] ]
NULL_STYLE_ISSUE_TYPES = [ NULL_STYLE_ISSUE_TYPES = [

@ -729,41 +729,6 @@
"file": "codetoanalyze/java/infer/ResourceLeaks.java", "file": "codetoanalyze/java/infer/ResourceLeaks.java",
"procedure": "int ResourceLeaks.readConfigNotCloseStream(String)" "procedure": "int ResourceLeaks.readConfigNotCloseStream(String)"
}, },
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFAfterBlockBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBadWrongAnnotation()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBadWrongLock()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readGFromCopyBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readHBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readHBadSynchronizedMethodShouldntHelp()"
},
{ {
"bug_type": "RESOURCE_LEAK", "bug_type": "RESOURCE_LEAK",
"file": "codetoanalyze/java/infer/ResourceLeaks.java", "file": "codetoanalyze/java/infer/ResourceLeaks.java",
@ -809,11 +774,6 @@
"file": "codetoanalyze/java/infer/CloseableAsResourceExample.java", "file": "codetoanalyze/java/infer/CloseableAsResourceExample.java",
"procedure": "T CloseableAsResourceExample.sourceOfNullWithResourceLeak()" "procedure": "T CloseableAsResourceExample.sourceOfNullWithResourceLeak()"
}, },
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.synchronizedMethodBad()"
},
{ {
"bug_type": "NULL_DEREFERENCE", "bug_type": "NULL_DEREFERENCE",
"file": "codetoanalyze/java/infer/NullPointerExceptions.java", "file": "codetoanalyze/java/infer/NullPointerExceptions.java",

@ -734,41 +734,6 @@
"file": "infer/tests/codetoanalyze/java/infer/ResourceLeaks.java", "file": "infer/tests/codetoanalyze/java/infer/ResourceLeaks.java",
"procedure": "int ResourceLeaks.readConfigNotCloseStream(String)" "procedure": "int ResourceLeaks.readConfigNotCloseStream(String)"
}, },
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFAfterBlockBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBadWrongAnnotation()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readFBadWrongLock()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readGFromCopyBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readHBad()"
},
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.readHBadSynchronizedMethodShouldntHelp()"
},
{ {
"bug_type": "RESOURCE_LEAK", "bug_type": "RESOURCE_LEAK",
"file": "infer/tests/codetoanalyze/java/infer/ResourceLeaks.java", "file": "infer/tests/codetoanalyze/java/infer/ResourceLeaks.java",
@ -814,11 +779,6 @@
"file": "infer/tests/codetoanalyze/java/infer/CloseableAsResourceExample.java", "file": "infer/tests/codetoanalyze/java/infer/CloseableAsResourceExample.java",
"procedure": "T CloseableAsResourceExample.sourceOfNullWithResourceLeak()" "procedure": "T CloseableAsResourceExample.sourceOfNullWithResourceLeak()"
}, },
{
"bug_type": "UNSAFE_GUARDED_BY_ACCESS",
"file": "infer/tests/codetoanalyze/java/infer/GuardedByExample.java",
"procedure": "void GuardedByExample.synchronizedMethodBad()"
},
{ {
"bug_type": "NULL_DEREFERENCE", "bug_type": "NULL_DEREFERENCE",
"file": "infer/tests/codetoanalyze/java/infer/NullPointerExceptions.java", "file": "infer/tests/codetoanalyze/java/infer/NullPointerExceptions.java",

Loading…
Cancel
Save