[java] Merged with 3665 (https://github.com/github/codeql/pull/3665)#3674
[java] Merged with 3665 (https://github.com/github/codeql/pull/3665)#3674haby0 wants to merge 1 commit intogithub:mainfrom haby0:master
Conversation
|
@haby0 This query as it stands now does not actually detect potential Fastjson RCE vulnerabilities. Besides parsing JSON, Autotyping needs to be enabled (either globally or locally). |
|
@p- When writing codeql rules, it is considered to add local detection of autoType, but autoType may be bypassed. |
|
@p- I think I can increase the detection of autoType in the codeql rules, but there will be some underreporting, because autoType has the risk of being bypassed. And I began to try to detect the set and get methods of dependent packages. |
|
@haby0 Oh wow, that's interesting... Makes it harder to write a good query for this. |
|
The additions here related to Spring are by now supported out-of-the-box - comprehensive Spring support was added in #3653. |
[java] Merged with 3665 (#3665)