Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding JNDI gadgets based on JDBC connection pooling classes. #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

benny-sec
Copy link

These gadgets trigger an RCE by abusing the JDBC connection string to a vulnerable DB.

These gadgets trigger an RCE by abusing the JDBC connection string to a vulnerable DB.
@benny-sec
Copy link
Author

Hi

Since the forceString in the org.apache.naming.factory.BeanFactory is replaced with a String setter lookup from Tomcat 9.0.63 / 8.5.79 onwards adding in a JDBC route as mentioned in the research of 浅蓝’s blogpost seems to be helpful with the newer Tomcat versions. Please note, my proficiency with of Java is quite limited and hence these additions may not have an optimal implementation / clean code, kindly let me know your input and I'll rework it as needed. Thank you

Kind Regards
SnowyOwl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant