Warnings fixed:

Unneded imports deleted
Never read variables deleted

git-svn-id: http://webgoat.googlecode.com/svn/trunk@323 4033779f-a91e-0410-96ef-6bf7bf53c507
This commit is contained in:
wirth.marcel 2008-04-13 15:15:09 +00:00
parent 89e67715fd
commit 1fca79e494
4 changed files with 4 additions and 26 deletions

View File

@ -4,6 +4,7 @@ package org.owasp.webgoat.lessons;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import java.util.regex.Pattern; import java.util.regex.Pattern;
import org.apache.ecs.Element; import org.apache.ecs.Element;
import org.apache.ecs.ElementContainer; import org.apache.ecs.ElementContainer;
import org.apache.ecs.html.A; import org.apache.ecs.html.A;
@ -18,7 +19,6 @@ import org.apache.ecs.html.TH;
import org.apache.ecs.html.TR; import org.apache.ecs.html.TR;
import org.apache.ecs.html.Table; import org.apache.ecs.html.Table;
import org.owasp.webgoat.session.WebSession; import org.owasp.webgoat.session.WebSession;
import org.owasp.webgoat.util.HtmlEncoder;
/*************************************************************************************************** /***************************************************************************************************

View File

@ -49,7 +49,6 @@ import org.owasp.webgoat.session.WebSession;
public class PasswordStrength extends LessonAdapter public class PasswordStrength extends LessonAdapter
{ {
private final static String USER = "user";
/** /**
* Description of the Method * Description of the Method
@ -62,7 +61,6 @@ public class PasswordStrength extends LessonAdapter
{ {
ElementContainer ec = new ElementContainer(); ElementContainer ec = new ElementContainer();
StringBuffer person = null;
try try
{ {

View File

@ -3,20 +3,19 @@ package org.owasp.webgoat.lessons;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import org.apache.ecs.Element; import org.apache.ecs.Element;
import org.apache.ecs.ElementContainer; import org.apache.ecs.ElementContainer;
import org.apache.ecs.StringElement; import org.apache.ecs.StringElement;
import org.apache.ecs.html.A; import org.apache.ecs.html.A;
import org.apache.ecs.html.BR; import org.apache.ecs.html.BR;
import org.apache.ecs.html.H2;
import org.apache.ecs.html.H3; import org.apache.ecs.html.H3;
import org.apache.ecs.html.IMG; import org.apache.ecs.html.IMG;
import org.apache.ecs.html.Input; import org.apache.ecs.html.Input;
import org.apache.ecs.html.Script; import org.apache.ecs.html.Script;
import org.apache.ecs.html.TextArea; import org.apache.ecs.html.TextArea;
import org.apache.ecs.xhtml.button; import org.apache.ecs.xhtml.button;
import org.apache.ecs.xhtml.link; import org.owasp.webgoat.session.WebSession;
import org.owasp.webgoat.session.*;
public class SameOriginPolicyProtection extends LessonAdapter public class SameOriginPolicyProtection extends LessonAdapter

View File

@ -7,6 +7,7 @@ import java.sql.ResultSet;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import java.util.Random; import java.util.Random;
import org.apache.ecs.Element; import org.apache.ecs.Element;
import org.apache.ecs.ElementContainer; import org.apache.ecs.ElementContainer;
import org.apache.ecs.StringElement; import org.apache.ecs.StringElement;
@ -140,10 +141,6 @@ public class SessionFixation extends SequentialLessonAdapter
getLessonTracker(s).setStage(3); getLessonTracker(s).setStage(3);
s.setMessage("You completed stage 2!"); s.setMessage("You completed stage 2!");
} }
// else
// {
// createStage2Content(s);
// }
} }
String mailContent = s.getParser().getRawParameter(MAILCONTENTNAME, ""); String mailContent = s.getParser().getRawParameter(MAILCONTENTNAME, "");
@ -442,22 +439,6 @@ public class SessionFixation extends SequentialLessonAdapter
return ec; return ec;
} }
/**
* See if the user has logged in correctly
*
* @param s
* @return true if loggedIn
*/
private boolean loggedIn(WebSession s)
{
try
{
return s.get(LOGGEDIN).equals("true");
} catch (Exception e)
{
return false;
}
}
/** /**
* See if the password and corresponding user is valid * See if the password and corresponding user is valid