Author: markt Date: Sat Dec 27 11:47:42 2008 New Revision: 729665 URL: http://svn.apache.org/viewvc?rev=729665&view=rev Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46408 Correct potential invalid cast
Modified: tomcat/tc6.0.x/trunk/ (props changed) tomcat/tc6.0.x/trunk/STATUS.txt tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityUtil.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Propchange: tomcat/tc6.0.x/trunk/ ------------------------------------------------------------------------------ --- svn:mergeinfo (original) +++ svn:mergeinfo Sat Dec 27 11:47:42 2008 @@ -1 +1 @@ -/tomcat/trunk:601180,606992,612607,630314,652744,653247,673796,673820,683982,684001,684081,684234,684269-684270,687503,687645,690781,691392,691805,692748,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,709294,709811,709816,710063,710125,710205,711126,711600,712461,718360,726052 +/tomcat/trunk:601180,606992,612607,630314,652744,653247,673796,673820,683982,684001,684081,684234,684269-684270,687503,687645,690781,691392,691805,692748,695053,695311,696780,696782,698012,698227,698236,698613,699427,699634,709294,709811,709816,710063,710125,710205,711126,711600,712461,718360,726052,728032 Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=729665&r1=729664&r2=729665&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS.txt (original) +++ tomcat/tc6.0.x/trunk/STATUS.txt Sat Dec 27 11:47:42 2008 @@ -250,12 +250,6 @@ +1: markt, fhanik -1: -* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46408 - Correct potential invalid cast - http://svn.apache.org/viewvc?rev=728032&view=rev - +1: markt, fhanik, billbarker - -1: - * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46304 Excludes performance issues Use Principal when calling doFilterEvent() Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityUtil.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityUtil.java?rev=729665&r1=729664&r2=729665&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityUtil.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/security/SecurityUtil.java Sat Dec 27 11:47:42 2008 @@ -299,9 +299,14 @@ } Subject.doAsPrivileged(subject, pea, null); - } catch( PrivilegedActionException pe) { - Throwable e = ((InvocationTargetException)pe.getException()) + } catch( PrivilegedActionException pe) { + Throwable e; + if (pe.getException() instanceof InvocationTargetException) { + e = ((InvocationTargetException)pe.getException()) .getTargetException(); + } else { + e = pe; + } if (log.isDebugEnabled()){ log.debug(sm.getString("SecurityUtil.doAsPrivilege"), e); Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=729665&r1=729664&r2=729665&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Sat Dec 27 11:47:42 2008 @@ -202,6 +202,9 @@ Remove unnecessary reference to commons-logging from the bootstrap JAR manifest. (markt) </fix> + <fix> + <bug>46408</bug>: Fix invalid cast in security utility package. (markt) + </fix> </changelog> </subsection> <subsection name="Coyote"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org