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

#1409 NullPointerException in InsufficientStringBufferRule

Browse Source
This commit is contained in:
Andreas Dangel
2015-09-20 20:51:09 +02:00
parent 365e7a1f68
commit d8d8a9539f
3 changed files with 26 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
pmd-java/src/main/java/net/sourceforge/pmd/lang/java/rule/strings/InsufficientStringBufferDeclarationRule.java
View File

@ -305,7 +305,9 @@ public class InsufficientStringBufferDeclarationRule extends AbstractJavaRule {
int size = 0; int size = 0;
if (parent != null) { if (parent != null) {
final Node initializer = parent.getFirstChildOfType(ASTVariableInitializer.class); final Node initializer = parent.getFirstChildOfType(ASTVariableInitializer.class);
if (initializer != null) {
final Node primExp = initializer.getFirstDescendantOfType(ASTPrimaryExpression.class); final Node primExp = initializer.getFirstDescendantOfType(ASTPrimaryExpression.class);
if (primExp != null) {
for (int i = 0; i < primExp.jjtGetNumChildren(); i++) { for (int i = 0; i < primExp.jjtGetNumChildren(); i++) {
final Node sn = primExp.jjtGetChild(i); final Node sn = primExp.jjtGetChild(i);
if (!(sn instanceof ASTPrimarySuffix) || sn.getImage() != null) { if (!(sn instanceof ASTPrimarySuffix) || sn.getImage() != null) {
@ -314,6 +316,8 @@ public class InsufficientStringBufferDeclarationRule extends AbstractJavaRule {
size += processNode(sn); size += processNode(sn);
} }
} }
}
}
return size; return size;
} }

13
pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/strings/xml/InsufficientStringBufferDeclaration.xml
View File

@ -1023,6 +1023,19 @@ public class StringBuilderWithMessageRetrieval {
} }
} }
}
]]></code>
</test-code>
<test-code>
<description>#1409 NullPointerException in InsufficientStringBufferRule</description>
<expected-problems>0</expected-problems>
<code><![CDATA[
public class NullPointer {
public void run(String[] strings) {
StringBuilder builder;
builder = new StringBuilder();
}
} }
]]></code> ]]></code>
</test-code> </test-code>

2
src/site/markdown/overview/changelog.md
View File

@ -18,6 +18,8 @@
* java-controversial/DefaultPackage: * java-controversial/DefaultPackage:
[#1410](https://sourceforge.net/p/pmd/bugs/1410/): DefaultPackage triggers on field annotated with @VisibleForTesting [#1410](https://sourceforge.net/p/pmd/bugs/1410/): DefaultPackage triggers on field annotated with @VisibleForTesting
* java-strings/InsufficientStringBufferDeclaration:
[#1409](https://sourceforge.net/p/pmd/bugs/1409/): NullPointerException in InsufficientStringBufferRule
**API Changes:** **API Changes:**