Prevent NPE in TcpSyslogMessageSender#close (#83)

AmirWiener · Amir Wiener · web-flow · commit 2753e8d0a05e · 2025-07-10T08:41:46.000-07:00
Co-authored-by: Amir Wiener &lt;awiener@fortinet.com&gt;
diff --git a/src/main/java/com/cloudbees/syslog/sender/TcpSyslogMessageSender.java b/src/main/java/com/cloudbees/syslog/sender/TcpSyslogMessageSender.java
@@ -288,6 +288,8 @@ public String toString() {
 
     @Override
     public void close() throws IOException {
-        this.socket.close();
+        if (socket != null) {
+            this.socket.close();
+        }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -288,6 +288,8 @@ public String toString() {`
`288`	`288`
`289`	`289`	`@Override`
`290`	`290`	`public void close() throws IOException {`
`291`		`- this.socket.close();`
	`291`	`+ if (socket != null) {`
	`292`	`+ this.socket.close();`
	`293`	`+ }`
`292`	`294`	`}`
`293`	`295`	`}`