[LOG4J2-2751] StackLocator.getCallerClass throws and catches unnecessarily internally in some cases

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.13.0
• Fix Version/s: 2.13.1
• Assignee: Carter Kozak
• Reporter: Carter Kozak
• Created: 30/Dec/19 18:59
• Updated: 30/Dec/19 19:05
• Resolved: 30/Dec/19 19:05

Description

In cases when 'WARNING: sun.reflect.Reflection.getCallerClass is not supported. This will impact performance.' is logged, invocations of 'getCallerClass' internally attempt a dynamic method invocation using a null Method reference, but catch the null pointer exception and return null.
We should do a null check first to avoid unnecessary work.