sander/perfix
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Bugfix for illegal field in interface: added extra check that instrumented jdbc impl class is not an interface

Browse source
This commit is contained in:
Sander Hautvast 2019-12-19 16:16:50 +01:00
parent a6f481fc43
commit 5a2ee4e7ec
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/main/java/perfix/instrument/JdbcInstrumentor.java
View file

@ -189,7 +189,7 @@ public class JdbcInstrumentor extends Instrumentor {
} }
boolean isJdbcPreparedStatementImpl(String resource, CtClass ctClass) throws NotFoundException { boolean isJdbcPreparedStatementImpl(String resource, CtClass ctClass) throws NotFoundException {
if (!resource.startsWith("java/sql")) { if (!ctClass.isInterface() && !resource.startsWith("java/sql")) {
return stream(ctClass.getInterfaces()) return stream(ctClass.getInterfaces())
.anyMatch(i -> i.getName().equals("java.sql.PreparedStatement")); .anyMatch(i -> i.getName().equals("java.sql.PreparedStatement"));
} }
@ -197,7 +197,7 @@ public class JdbcInstrumentor extends Instrumentor {
} }
boolean isJdbcConnectionImpl(String resource, CtClass ctClass) throws NotFoundException { boolean isJdbcConnectionImpl(String resource, CtClass ctClass) throws NotFoundException {
if (!resource.startsWith("java/sql")) { if (!ctClass.isInterface() && !resource.startsWith("java/sql")) {
return stream(ctClass.getInterfaces()) return stream(ctClass.getInterfaces())
.anyMatch(i -> i.getName().equals("java.sql.Connection")); .anyMatch(i -> i.getName().equals("java.sql.Connection"));
} }