Index: src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java =================================================================== --- src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java (revision 1414836) +++ src/main/java/org/apache/logging/log4j/spi/AbstractLogger.java (working copy) @@ -1157,7 +1157,7 @@ */ public void error(Marker marker, Message msg) { if (isEnabled(Level.ERROR, marker, msg, null)) { - log(null, FQCN, Level.ERROR, msg, null); + log(marker, FQCN, Level.ERROR, msg, null); } } Index: src/test/java/org/apache/logging/log4j/AbstractLoggerTest.java =================================================================== --- src/test/java/org/apache/logging/log4j/AbstractLoggerTest.java (revision 1414836) +++ src/test/java/org/apache/logging/log4j/AbstractLoggerTest.java (working copy) @@ -66,6 +66,7 @@ new LogEvent(null, simple, t), new LogEvent(marker, simple, null), new LogEvent(marker, simple, t), + new LogEvent(marker, simple, null), }; @@ -107,6 +108,8 @@ trace(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; trace(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + trace(MarkerManager.getMarker("TEST"), simple); } @Test @@ -147,6 +150,8 @@ debug(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; debug(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + debug(MarkerManager.getMarker("TEST"), simple); } @Test @@ -187,6 +192,8 @@ info(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; info(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + info(MarkerManager.getMarker("TEST"), simple); } @Test @@ -227,6 +234,8 @@ warn(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; warn(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + warn(MarkerManager.getMarker("TEST"), simple); } @Test @@ -267,6 +276,8 @@ error(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; error(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + error(MarkerManager.getMarker("TEST"), simple); } @Test @@ -307,6 +318,8 @@ fatal(MarkerManager.getMarker("TEST"), simple, null); currentEvent = events[13]; fatal(MarkerManager.getMarker("TEST"), simple, t); + currentEvent = events[14]; + fatal(MarkerManager.getMarker("TEST"), simple); } @Override