Fix: NullPointerException triggered when http error in job allocation (…

…#274)
laurent-clouet · Jul 7, 2020 · 674a375 · 674a375
1 parent 081c20f
commit 674a375
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/com/sheepit/client/Server.java b/src/com/sheepit/client/Server.java
@@ -475,11 +475,16 @@ else if (this.client.getRenderingJob().isUserBlockJob()) {
 		}
 		finally {
 			try {
-				output.flush();
-				output.close();
-				is.close();
+				if (output != null) {
+					output.flush();
+					output.close();
+				}
+
+				if (is != null) {
+					is.close();
+				}
 			}
-			catch (IOException e) {
+			catch (Exception e) {
 				this.log.debug(String.format("Server::HTTPGetFile Error trying to close the open streams (%s)", e.getMessage()));
 			}
 		}