Skip to content

Revert "ComputeEnginerInstance terminate instance without checking if…

ci.jenkins.io / SpotBugs failed Dec 11, 2024 in 0s

2 new issues

Total New Outstanding Fixed Trend
2 2 0 0 👎

Reference build: Plugins » google-compute-engine-plugin » develop #106

Details

Severity distribution of new issues

Error Warning High Warning Normal Warning Low
0 1 1 0

Annotations

Check warning on line 236 in src/main/java/com/google/jenkins/plugins/computeengine/ComputeEngineComputerLauncher.java

See this annotation in the file changed.

@ci-jenkins-io ci-jenkins-io / SpotBugs

NP_ALWAYS_NULL

HIGH:
Null pointer dereference of node in com.google.jenkins.plugins.computeengine.ComputeEngineComputerLauncher.terminateNode(SlaveComputer, TaskListener)
Raw output
<p> A null pointer is dereferenced here.&nbsp; This will lead to a <code>NullPointerException</code> when the code is executed.</p>

Check warning on line 236 in src/main/java/com/google/jenkins/plugins/computeengine/ComputeEngineComputerLauncher.java

See this annotation in the file changed.

@ci-jenkins-io ci-jenkins-io / SpotBugs

NP_LOAD_OF_KNOWN_NULL_VALUE

NORMAL:
Load of known null value in com.google.jenkins.plugins.computeengine.ComputeEngineComputerLauncher.terminateNode(SlaveComputer, TaskListener)
Raw output
<p> The variable referenced at this point is known to be null due to an earlier check against null. Although this is valid, it might be a mistake (perhaps you intended to refer to a different variable, or perhaps the earlier check to see if the variable is null should have been a check to see if it was non-null). </p>