From 34b230919ec4c8f741e48bd52845abb43b6be964 Mon Sep 17 00:00:00 2001 From: Dan <46821332+nsadeveloper789@users.noreply.github.com> Date: Fri, 21 Apr 2023 14:40:21 -0400 Subject: [PATCH] GP-0: Fix some infrequently failing tests. Fixes real issue in impl. --- .../debug/gui/register/DebuggerRegistersProviderTest.java | 2 +- .../ghidra/trace/database/memory/DBTraceMemoryManager.java | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/Ghidra/Debug/Debugger/src/test/java/ghidra/app/plugin/core/debug/gui/register/DebuggerRegistersProviderTest.java b/Ghidra/Debug/Debugger/src/test/java/ghidra/app/plugin/core/debug/gui/register/DebuggerRegistersProviderTest.java index ae2a46cc2c..b9dea66529 100644 --- a/Ghidra/Debug/Debugger/src/test/java/ghidra/app/plugin/core/debug/gui/register/DebuggerRegistersProviderTest.java +++ b/Ghidra/Debug/Debugger/src/test/java/ghidra/app/plugin/core/debug/gui/register/DebuggerRegistersProviderTest.java @@ -15,7 +15,7 @@ */ package ghidra.app.plugin.core.debug.gui.register; -import static ghidra.lifecycle.Unfinished.*; +import static ghidra.lifecycle.Unfinished.TODO; import static org.junit.Assert.*; import java.math.BigInteger; diff --git a/Ghidra/Debug/Framework-TraceModeling/src/main/java/ghidra/trace/database/memory/DBTraceMemoryManager.java b/Ghidra/Debug/Framework-TraceModeling/src/main/java/ghidra/trace/database/memory/DBTraceMemoryManager.java index 59599946b3..06c0b35c25 100644 --- a/Ghidra/Debug/Framework-TraceModeling/src/main/java/ghidra/trace/database/memory/DBTraceMemoryManager.java +++ b/Ghidra/Debug/Framework-TraceModeling/src/main/java/ghidra/trace/database/memory/DBTraceMemoryManager.java @@ -35,8 +35,7 @@ import ghidra.trace.database.map.DBTraceAddressSnapRangePropertyMapTree.TraceAdd import ghidra.trace.database.space.AbstractDBTraceSpaceBasedManager; import ghidra.trace.database.space.DBTraceDelegatingManager; import ghidra.trace.database.thread.DBTraceThreadManager; -import ghidra.trace.model.Lifespan; -import ghidra.trace.model.TraceAddressSnapRange; +import ghidra.trace.model.*; import ghidra.trace.model.memory.*; import ghidra.trace.model.stack.TraceStackFrame; import ghidra.trace.model.thread.TraceThread; @@ -313,7 +312,9 @@ public class DBTraceMemoryManager extends AbstractDBTraceSpaceBasedManager> doGetStates(Lifespan span, AddressRange range) { - return delegateRead(range.getAddressSpace(), m -> m.doGetStates(span, range)); + return delegateReadOr(range.getAddressSpace(), m -> m.doGetStates(span, range), + () -> List.of(Map.entry(new ImmutableTraceAddressSnapRange(range, span), + TraceMemoryState.UNKNOWN))); } @Override