Merge remote-tracking branch 'origin/GP-2636_d-millar_update_on_writeRegisters'

Author: ryanmkurtz

Ghidra/Debug/Debugger-agent-dbgeng/src/main/java/agent/dbgeng/manager/cmd/DbgLaunchProcessCommand.java

@@ -95,7 +95,7 @@ public void invoke() {
 	}
 
 	private String fixPath(String input) {
-		if (input.equals("")) {
+		if (input == null || input.equals("")) {
 			return null;
 		}
 		String output = input;

Ghidra/Debug/Debugger-agent-dbgeng/src/main/java/agent/dbgeng/model/iface2/DbgModelTargetRegisterBank.java

@@ -21,7 +21,7 @@
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.atomic.AtomicReference;
 
-import agent.dbgeng.manager.DbgThread;
+import agent.dbgeng.manager.*;
 import agent.dbgeng.manager.impl.*;
 import ghidra.async.AsyncUtils;
 import ghidra.async.TypeSpec;
@@ -101,6 +101,7 @@ public default CompletableFuture<Void> writeRegistersNamed(Map<String, byte[]> v
 	}
 
 	public default CompletableFuture<Void> doWriteRegistersNamed(Map<String, byte[]> values) {
+		DbgManagerImpl manager = getManager();
 		DbgThread thread = getParentThread().getThread();
 		return AsyncUtils.sequence(TypeSpec.VOID).then(seq -> {
 			requestNativeElements().handle(seq::nextIgnore);
@@ -122,6 +123,8 @@ public default CompletableFuture<Void> doWriteRegistersNamed(Map<String, byte[]>
 			getParentThread().getThread().writeRegisters(toWrite).handle(seq::next);
 			// TODO: Should probably filter only effective and normalized writes in the callback
 		}).then(seq -> {
+			manager.getEventListeners().fire.threadStateChanged(thread, thread.getState(),
+				DbgCause.Causes.UNCLAIMED, DbgReason.Reasons.NONE);
 			broadcast().registersUpdated(getProxy(), values);
 			seq.exit();
 		}).finish();

Ghidra/Debug/Debugger-agent-dbgeng/src/main/java/agent/dbgeng/model/impl/DbgModelTargetRegisterContainerImpl.java

@@ -21,6 +21,7 @@
 import java.util.stream.Collectors;
 
 import agent.dbgeng.manager.*;
+import agent.dbgeng.manager.impl.DbgManagerImpl;
 import agent.dbgeng.manager.impl.DbgRegister;
 import agent.dbgeng.model.iface2.*;
 import ghidra.async.AsyncUtils;
@@ -141,6 +142,7 @@ public CompletableFuture<? extends Map<String, byte[]>> readRegistersNamed(
 
 	@Override
 	public CompletableFuture<Void> writeRegistersNamed(Map<String, byte[]> values) {
+		DbgManagerImpl manager = getManager();
 		return model.gateFuture(thread.listRegisters().thenCompose(regs -> {
 			return requestElements(false);
 		}).thenCompose(__ -> {
@@ -159,6 +161,8 @@ public CompletableFuture<Void> writeRegistersNamed(Map<String, byte[]> values) {
 			return thread.writeRegisters(toWrite);
 			// TODO: Should probably filter only effective and normalized writes in the callback
 		}).thenAccept(__ -> {
+			manager.getEventListeners().fire.threadStateChanged(thread, thread.getState(),
+				DbgCause.Causes.UNCLAIMED, DbgReason.Reasons.NONE);
 			broadcast().registersUpdated(getProxy(), values);
 		}));
 	}