Skip to content

Commit 5b91bc9

Browse files
authored
SONARJAVA-5874 Fix bug with varargs on S1319 (#5361)
1 parent 46c2328 commit 5b91bc9

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

java-checks-test-sources/default/src/main/java/checks/CollectionImplementationReferencedCheck.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,9 @@ public void foo18(LinkedList<Integer> list){ // Noncompliant
172172
private void foo19(List<Integer> l){
173173
return;
174174
}
175-
175+
public void foo20(LinkedList<String> list) { // Noncompliant
176+
String.format("", 0, 0);
177+
}
176178
private LinkedList<Integer> getList() {
177179
return null;
178180
}

java-checks/src/main/java/org/sonar/java/checks/CollectionImplementationReferencedCheck.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ public void visitMethodInvocation(MethodInvocationTree tree) {
375375
return;
376376
}
377377
var paramTypes = methodSymbol.parameterTypes();
378-
for (int i = 0; i < arguments.size(); i++) {
378+
for (int i = 0; i < arguments.size() && i < paramTypes.size(); i++) {
379379
processorArgument(arguments.get(i), paramTypes.get(i));
380380
}
381381
}

0 commit comments

Comments
 (0)