
com.h3xstream.findsecbugs.ReDosDetector Maven / Gradle / Ivy
Show all versions of plugin Show documentation
/**
* Find Security Bugs
* Copyright (c) 2013, Philippe Arteau, All rights reserved.
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 3.0 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library.
*/
package com.h3xstream.findsecbugs;
import com.h3xstream.findsecbugs.common.StringTracer;
import edu.umd.cs.findbugs.BugInstance;
import edu.umd.cs.findbugs.BugReporter;
import edu.umd.cs.findbugs.OpcodeStack;
import edu.umd.cs.findbugs.Priorities;
import edu.umd.cs.findbugs.bcel.OpcodeStackDetector;
import edu.umd.cs.findbugs.classfile.FieldDescriptor;
import edu.umd.cs.findbugs.classfile.MethodDescriptor;
import org.apache.bcel.Constants;
import java.util.concurrent.atomic.AtomicInteger;
/**
* This detector does minimal effort to find potential REDOS.
*
* It will identify pattern similar to : (( )+)+
*
* It will not identify pattern of equivalence (such as(aa|a)
).
* It is far more complex to identify.
*
*
* For more advanced Regex analysis: Safe Regex
*/
public class ReDosDetector extends OpcodeStackDetector {
private static final boolean DEBUG = true;
private static final String REDOS_TYPE = "REDOS";
private static final char[] OPENING_CHAR = {'(','['};
private static final char[] CLOSING_CHAR = {')',']'};
private static final char[] PLUS_CHAR = {'+','*','?'};
private BugReporter bugReporter;
public ReDosDetector(BugReporter bugReporter) {
this.bugReporter = bugReporter;
}
@Override
public void sawOpcode(int seen) {
//printOpCode(seen);
if (seen == Constants.INVOKESTATIC && getClassConstantOperand().equals("java/util/regex/Pattern")
&& getNameConstantOperand().equals("compile")
&& getSigConstantOperand().equals("(Ljava/lang/String;)Ljava/util/regex/Pattern;")) {
OpcodeStack.Item item = stack.getStackItem(0);
if (!StringTracer.isVariableString(item)) {
String value = (String) item.getConstant();
analyseRegexString(value);
}
} else if (seen == Constants.INVOKEVIRTUAL && getClassConstantOperand().equals("java/lang/String")
&& getNameConstantOperand().equals("matches")
&& getSigConstantOperand().equals("(Ljava/lang/String;)Z")) {
OpcodeStack.Item item = stack.getStackItem(0);
if (!StringTracer.isVariableString(item)) {
String value = (String) item.getConstant();
analyseRegexString(value);
}
}
}
public void analyseRegexString(String regex) {
if(regex.length()>0) {
recurAnalyseRegex(regex,regex.length()-1,0);
}
}
private int recurAnalyseRegex(String regex, int startPosition, int level) {
// print(level, "level = " + level);
if(level == 2) {
MethodDescriptor md = this.getMethodDescriptor();
FieldDescriptor fd = this.getFieldDescriptor();
BugInstance bug = new BugInstance(this, REDOS_TYPE, Priorities.NORMAL_PRIORITY) //
.addString(regex).addClass(this);
if(md != null)
bug.addMethod(md);
if(fd != null)
bug.addField(fd);
try {
bug.addSourceLine(this);
}
catch (IllegalStateException e) {}
bugReporter.reportBug(bug);
return 0;
}
// print(level, "Analysing " + regex.substring(0, startPosition + 1));
boolean openingMode = false;
for(int i=startPosition;i>=0;i--) {
// print(level, "[" + i + "] = '" + regex.charAt(i) + "'");
if(isChar(regex,i,OPENING_CHAR) ) {
// print(level, "<<<<");
return i;
}
if(isChar(regex,i,CLOSING_CHAR) ) {
int newLevel = level;
if(i+1 < regex.length() && isChar(regex,i+1,PLUS_CHAR)) {
newLevel += 1;
}
// print(level, ">>>>");
openingMode=true;
i = recurAnalyseRegex(regex,i-1,newLevel);
if(i==-1) {
return 0;
}
// print(level, "Restarting at " + i);
}
}
// print(level, "END!");
return 0;
}
/**
*
* @param value
* @param position
* @param charToTest
* @return
*/
private boolean isChar(String value, int position,char[] charToTest) {
char actualChar = value.charAt(position);
boolean oneCharFound=false;
for(char ch : charToTest) {
if(actualChar == ch) {
oneCharFound = true;
break;
}
}
return oneCharFound && (position == 0 || value.charAt(position-1) != '\\');
}
//Debug method
// private void print(int level,Object obj) {
// System.out.println(lvl(level) + "> "+ obj);
// }
//
// private String lvl(int level) {
// StringBuilder str = new StringBuilder();
// for(int i=0;i