Check if job is set to hopefully fix potential null pointer exception

J12934 · J12934 · commit 23f17d2a096d · 2020-04-02T13:19:43.000+02:00
diff --git a/operator/controllers/execution/scan_controller.go b/operator/controllers/execution/scan_controller.go
@@ -145,7 +145,7 @@ func (r *ScanReconciler) checkIfJobIsCompleted(name, namespace string) (bool, er
 	if err != nil {
 		return false, err
 	}
-	return (job.Status.Succeeded != 0), nil
+	return (job != nil && job.Status.Succeeded != 0), nil
 }
 
 func (r *ScanReconciler) startScan(scan *executionv1.Scan) error {

Original file line number	Diff line number	Diff line change
`@@ -145,7 +145,7 @@ func (r *ScanReconciler) checkIfJobIsCompleted(name, namespace string) (bool, er`
`145`	`145`	`if err != nil {`
`146`	`146`	`return false, err`
`147`	`147`	`}`
`148`		`- return (job.Status.Succeeded != 0), nil`
	`148`	`+ return (job != nil && job.Status.Succeeded != 0), nil`
`149`	`149`	`}`
`150`	`150`
`151`	`151`	`func (r ScanReconciler) startScan(scan executionv1.Scan) error {`