phoedos/pmd
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'pr-181'

Browse Source
This commit is contained in:
Juan Martín Sotuyo Dodero
2017-01-12 22:02:26 -03:00
parent e4c353ff83 608c9139be
commit ac0b873f08
2 changed files with 339 additions and 57 deletions
Download Patch File Download Diff File Expand all files Collapse all files

225
pmd-apex/src/main/java/net/sourceforge/pmd/lang/apex/rule/security/ApexCRUDViolationRule.java
View File

File diff suppressed because it is too large Load Diff

165
pmd-apex/src/test/resources/net/sourceforge/pmd/lang/apex/rule/security/xml/ApexCRUDViolation.xml
View File

@ -480,4 +480,169 @@ public class FooTest {
} }
]]></code> ]]></code>
</test-code> </test-code>
<test-code>
<description>Control flow based CRUD,FLS check</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
public void foo(String newName, String tempID) {
doChecks();
upsert new Contact(FirstName = 'First', LastName = 'Last', Phone = '414-414-4414');
}
public void doChecks() {
if (!Contact.sObjectType.getDescribe().isCreateable() && !Contact.sObjectType.getDescribe().isUpdateable()) {
throw new NoAccessException();
}
}
} ]]></code>
</test-code>
<test-code>
<description>Control flow based CRUD,FLS check recursive</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
public void foo(String newName, String tempID) {
doChecks();
insert new Contact(FirstName = 'First', LastName = 'Last', Phone = '414-414-4414');
}
public void doChecks() {
anotherLevelHere("yolo");
}
private void anotherLevelHere(String s) {
if (!Contact.sObjectType.getDescribe().isCreateable()) {
throw new NoAccessException();
}
}
} ]]></code>
</test-code>
<test-code>
<description>Control flow constructor based CRUD,FLS check</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
public void foo(String newName, String tempID) {
upsert new Contact(FirstName = 'First', LastName = 'Last', Phone = '414-414-4414');
}
public Foo() {
if (!Contact.sObjectType.getDescribe().isCreateable() && !Contact.sObjectType.getDescribe().isUpdateable()) {
throw new NoAccessException();
}
}
} ]]></code>
</test-code>
<test-code>
<description>Control flow accessibility CRUD check</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
public Contact justGiveMeFoo() {
checkPerms();
String tempID = 'someID';
return [SELECT Name FROM Contact WHERE Id=:tempID];
}
private void checkPerms() {
if (!Contact.sObjectType.getDescribe().isAccessible()) {
throw new NoAccessException();
}
}
}
]]></code>
</test-code>
<test-code>
<description>Control flow substitute CRUD check</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
public Contact justGiveMeFoo() {
checkPerms();
String tempID = 'someID';
return [SELECT Name FROM Contact WHERE Id=:tempID];
}
private void checkPerms() {
if (!Contact.sObjectType.getDescribe().isCreateable()) {
throw new NoAccessException();
}
}
}
]]></code>
</test-code>
<test-code>
<description>Forgot to call the CRUD check</description>
<expected-problems>1</expected-problems>
<code><![CDATA[
public class Foo {
public Contact justGiveMeFoo() {
String tempID = 'someID';
return [SELECT Name FROM Contact WHERE Id=:tempID];
}
private void checkPerms() {
if (!Contact.sObjectType.getDescribe().isCreateable()) {
throw new NoAccessException();
}
}
}
]]></code>
</test-code>
<test-code>
<description>Control flow substitute CRUD check should fail when check follows SOQL</description>
<expected-problems>1</expected-problems>
<code><![CDATA[
public class Foo {
private void bar() {
List<Profile> profiles = [SELECT Id FROM Profile WHERE Name = 'System Administrator'];
checkPerms();
}
private void checkPerms() {
if (!Profile.sObjectType.getDescribe().isCreateable()) {
throw new NoAccessException();
}
}
}
]]></code>
</test-code>
<test-code>
<description>Control flow with nested statementsL</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class Foo {
private void bar() {
if (whatever()) {
checkPerms();
if (something()) {
List<Profile> profiles = [SELECT Id FROM Profile WHERE Name = 'System Administrator'];
}
}
}
private void checkPerms() {
if (!Profile.sObjectType.getDescribe().isCreateable()) {
throw new NoAccessException();
}
}
}
]]></code>
</test-code>
</test-data> </test-data>