Bug 46144 - QuietWriter.write should ignore all Exceptions
QuietWriter.write should ignore all Exceptions
Product: Log4j
Classification: Unclassified
Component: Appender
PC Mac OS X 10.4
: P2 normal
: ---
Assigned To: log4j-dev
Depends on:
  Show dependency tree
Reported: 2008-11-04 09:52 UTC by Stepan Koltsov
Modified: 2009-01-08 10:14 UTC (History)
0 users


Note You need to log in before you can comment on or make changes to this bug.
Description Stepan Koltsov 2008-11-04 09:52:42 UTC
QuietWriter.write should ignore all Exception's, not only IOException.

Actually, it should ignore NullPointerException or just check input string for null.


throws NullPointerException if passed string is null, and if null string accidentally appeared in QuietWriter.write, it should be ignored, rather then NullPointerException thrown.

Currently we have rare NullPointerException in logs somewhere inside log4j, and we cannot repeat this situation:

        at java.io.Writer.write(Writer.java:140)
        at org.apache.log4j.helpers.QuietWriter.write(QuietWriter.java:48)
        at org.apache.log4j.WriterAppender.subAppend(WriterAppender.java:309)
        at org.apache.log4j.WriterAppender.append(WriterAppender.java:160)
        at ru.yandex.video.converter.convert.ConfiguredFileAppender.append(ConfiguredFileAppender.java:23)
        at org.apache.log4j.AppenderSkeleton.doAppend(AppenderSkeleton.java:251)

log4j is 1.2.15. Proposed solution avoids NullPointerException (though probably causes some information loss).
Comment 1 Curt Arnold 2009-01-08 10:14:39 UTC
Added check for null argument in rev 732774.