Fix NPE when removing non-visible remote JVMs in attach window

Col-E · Col-E · commit b531b7b41d4a · 2024-07-24T05:12:26.000-04:00
diff --git a/recaf-ui/src/main/java/software/coley/recaf/ui/pane/RemoteVirtualMachinesPane.java b/recaf-ui/src/main/java/software/coley/recaf/ui/pane/RemoteVirtualMachinesPane.java
@@ -187,7 +187,7 @@ public void onScanCompleted(@Nonnull Set<VirtualMachineDescriptor> added,
 				vmButtonsList.getChildren().remove(removedButton);
 
 				VmPane cell = vmCellMap.remove(descriptor);
-				cell.setDisable(true);
+				if (cell != null) cell.setDisable(true);
 			}
 
 			// Refresh current cell.

Original file line number	Diff line number	Diff line change
`@@ -187,7 +187,7 @@ public void onScanCompleted(@Nonnull Set<VirtualMachineDescriptor> added,`
`187`	`187`	`vmButtonsList.getChildren().remove(removedButton);`
`188`	`188`
`189`	`189`	`VmPane cell = vmCellMap.remove(descriptor);`
`190`		`- cell.setDisable(true);`
	`190`	`+ if (cell != null) cell.setDisable(true);`
`191`	`191`	`}`
`192`	`192`
`193`	`193`	`// Refresh current cell.`