Author: carnold
Date: Thu Jan 17 10:18:59 2008
New Revision: 612911

URL: http://svn.apache.org/viewvc?rev=612911&view=rev
Log:
Bug 44109: TelnetAppender throws null pointers at log time when socket couldn't 
be opened

Modified:
    logging/log4j/trunk/src/main/java/org/apache/log4j/net/TelnetAppender.java

Modified: 
logging/log4j/trunk/src/main/java/org/apache/log4j/net/TelnetAppender.java
URL: 
http://svn.apache.org/viewvc/logging/log4j/trunk/src/main/java/org/apache/log4j/net/TelnetAppender.java?rev=612911&r1=612910&r2=612911&view=diff
==============================================================================
--- logging/log4j/trunk/src/main/java/org/apache/log4j/net/TelnetAppender.java 
(original)
+++ logging/log4j/trunk/src/main/java/org/apache/log4j/net/TelnetAppender.java 
Thu Jan 17 10:18:59 2008
@@ -103,17 +103,19 @@
   /** Handles a log event.  For this appender, that means writing the
     message to each connected client.  */
   protected void append(LoggingEvent event) {
-    sh.send(this.layout.format(event));
-    if(layout.ignoresThrowable()) {
-      String[] s = event.getThrowableStrRep();
-      if (s != null) {
-       int len = s.length;
-       for(int i = 0; i < len; i++) {
-         sh.send(s[i]);
-         sh.send(Layout.LINE_SEP);
-       }
+      if(sh != null) {
+        sh.send(this.layout.format(event));
+        if(layout.ignoresThrowable()) {
+            String[] s = event.getThrowableStrRep();
+            if (s != null) {
+                   int len = s.length;
+                   for(int i = 0; i < len; i++) {
+                       sh.send(s[i]);
+                       sh.send(Layout.LINE_SEP);
+                   }
+            }
+        }
       }
-    }
   }
 
   //---------------------------------------------------------- SocketHandler:



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to