Siphalor/tweed5-commons-logging
1
0
Fork 0
Code Activity

LOGGING-147 - SimpleLog.log - unsafe update of shortLogName

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/logging/trunk@1363173 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Sebastian Bazley
2012-07-18 23:30:03 +00:00
parent e8c77e524e
commit 7eada4cf49
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
RELEASE-NOTES.txt
View File

@@ -22,6 +22,7 @@ LOGGING-130 - Potential missing privileged block for class loader
LOGGING-145 - LogFactoryImpl.setAttribute - possible NPE LOGGING-145 - LogFactoryImpl.setAttribute - possible NPE
LOGGING-142 - Log4JLogger uses deprecated static members of Priority such as INFO LOGGING-142 - Log4JLogger uses deprecated static members of Priority such as INFO
LOGGING-128 - Static analysis suggests a number of potential improvements LOGGING-128 - Static analysis suggests a number of potential improvements
LOGGING-147 - SimpleLog.log - unsafe update of shortLogName
$Id$ $Id$

5
src/java/org/apache/commons/logging/impl/SimpleLog.java
View File

@@ -311,9 +311,8 @@ public class SimpleLog implements Log, Serializable {
if( showShortName) { if( showShortName) {
if( shortLogName==null ) { if( shortLogName==null ) {
// Cut all but the last component of the name for both styles // Cut all but the last component of the name for both styles
shortLogName = logName.substring(logName.lastIndexOf(".") + 1); String slName = logName.substring(logName.lastIndexOf(".") + 1);
shortLogName = shortLogName = slName.substring(slName.lastIndexOf("/") + 1);
shortLogName.substring(shortLogName.lastIndexOf("/") + 1);
} }
buf.append(String.valueOf(shortLogName)).append(" - "); buf.append(String.valueOf(shortLogName)).append(" - ");
} else if(showLogName) { } else if(showLogName) {