Skip to content

Commit 43c6e55

Browse files
committed
change array for list
1 parent 676ba9a commit 43c6e55

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

dd-java-agent/agent-iast/src/main/java/com/datadog/iast/securitycontrol/SecurityControlMethodClassVisitor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,12 +56,12 @@ private boolean shouldBeAdapted(SecurityControl securityControl, String name, St
5656
}
5757

5858
Type[] types = Type.getArgumentTypes(desc);
59-
if (types.length != securityControl.getParameterTypes().length) {
59+
if (types.length != securityControl.getParameterTypes().size()) {
6060
return false;
6161
}
6262

6363
for (int i = 0; i < types.length; i++) {
64-
if (!types[i].getClassName().equals(securityControl.getParameterTypes()[i])) {
64+
if (!types[i].getClassName().equals(securityControl.getParameterTypes().get(i))) {
6565
return false;
6666
}
6767
}

internal-api/src/main/java/datadog/trace/api/iast/securitycontrol/SecurityControl.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
package datadog.trace.api.iast.securitycontrol;
22

3+
import java.util.List;
34
import java.util.Set;
45
import javax.annotation.Nonnull;
56
import javax.annotation.Nullable;
@@ -14,7 +15,7 @@ public class SecurityControl {
1415

1516
@Nonnull private String method;
1617

17-
@Nullable private String[] parameterTypes;
18+
@Nullable private List<String> parameterTypes;
1819

1920
@Nullable private Set<Integer> parametersToMark;
2021

@@ -23,7 +24,7 @@ public SecurityControl(
2324
int marks,
2425
@Nonnull String className,
2526
@Nonnull String method,
26-
@Nullable String[] parameterTypes,
27+
@Nullable List<String> parameterTypes,
2728
@Nullable Set<Integer> parametersToMark) {
2829
this.type = type;
2930
this.marks = marks;
@@ -54,7 +55,7 @@ public String getMethod() {
5455
}
5556

5657
@Nullable
57-
public String[] getParameterTypes() {
58+
public List<String> getParameterTypes() {
5859
return parameterTypes;
5960
}
6061

internal-api/src/main/java/datadog/trace/api/iast/securitycontrol/SecurityControlFormatter.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ private static SecurityControl getSecurityControl(@Nonnull final String config)
7070
String className = split[2].replaceAll("\\.", "/");
7171
String method = split[3];
7272

73-
String[] parameterTypes = null;
73+
List<String> parameterTypes = null;
7474
Set<Integer> parametersToMark = null;
7575

7676
if (split.length > 4) {
@@ -83,7 +83,7 @@ private static SecurityControl getSecurityControl(@Nonnull final String config)
8383
}
8484
parametersToMark = getParametersToMark(elements);
8585
} else {
86-
parameterTypes = elements;
86+
parameterTypes = Arrays.asList(elements);
8787
}
8888
}
8989
}

0 commit comments

Comments
 (0)