From 57b30e333b70ffc624e958ebbd4b18838dd2afec Mon Sep 17 00:00:00 2001
From: Sebastian Bazley <sebb@apache.org>
Date: Wed, 18 Jul 2012 15:19:22 +0000
Subject: [PATCH] LOGGING-130 - Potential missing privileged block for class
 loader

git-svn-id: https://svn.apache.org/repos/asf/commons/proper/logging/trunk@1362978 13f79535-47bb-0310-9956-ffa450edef68
---
 RELEASE-NOTES.txt                                           | 3 +++
 .../org/apache/commons/logging/impl/LogFactoryImpl.java     | 6 ++++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/RELEASE-NOTES.txt b/RELEASE-NOTES.txt
index 61d91fa..834c5db 100644
--- a/RELEASE-NOTES.txt
+++ b/RELEASE-NOTES.txt
@@ -17,6 +17,9 @@
 
 -->
 
+TODO - rework notes to incorporate: Changes since 1.1.1
+LOGGING-130 - Potential missing privileged block for class loader
+
 $Id$
 
                            Commons Logging Package
diff --git a/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java b/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
index 7144dd8..44d003c 100644
--- a/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
+++ b/src/java/org/apache/commons/logging/impl/LogFactoryImpl.java
@@ -1322,7 +1322,8 @@ public class LogFactoryImpl extends LogFactory {
         while (current != null) {
             if (current == c2)
                 return c1;
-            current = current.getParent();
+            // current = current.getParent();
+            current = getParentClassLoader(current);
         }
        
         // scan c2's ancestors to find c1
@@ -1330,7 +1331,8 @@ public class LogFactoryImpl extends LogFactory {
         while (current != null) {
             if (current == c1)
                 return c2;
-            current = current.getParent();
+            // current = current.getParent();
+            current = getParentClassLoader(current);
         }
 
         return null;