[LOG4NET-460] log4net.Repository.Hierarchy.Logger ignores overridden virtual properties

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Assignee: Unassigned
• Reporter: Joseph Edwards
• Created: 31/Mar/15 21:12
• Updated: 01/Apr/15 04:29
• Resolved: 01/Apr/15 04:29

Description

log4net.Repository.Hierarchy.Logger declares Hierarchy property as virtual, however, when reading the value it uses the private field m_hierarchy, which is not the correct value (and probably null) if a derived class has overridden the Hierarchy property.
This leads to a NullReferenceException when invoking the IsEnabledFor method, which is used in the Log method.

Expected: overriding the Hierarchy property works
Actual: overriding the Hierarchy property leaves the private field m_hierarchy unset, which causes NullReferenceExceptions to be thrown

This issue also appears to apply for Level and m_level. The EffectiveLevel property accesses the private m_level field instead of the virtual Level property, which may also lead to NullReferenceException.