yetangitu/ghidra
1
0
Fork 0
mirror of https://github.com/NationalSecurityAgency/ghidra.git synced 2025-10-05 10:49:34 +02:00
Code Issues Releases Wiki Activity

Merge remote-tracking branch 'origin/GP-1484_Dan_fixAutoSaveTracesNameConflict'

Browse source
This commit is contained in:
Ryan Kurtz 2023-01-19 13:10:46 -05:00
commit a1dbb25260
1 changed files with 10 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

18
Ghidra/Debug/Debugger/src/main/java/ghidra/app/plugin/core/debug/service/tracemgr/DebuggerTraceManagerServicePlugin.java
View file

@ -953,13 +953,7 @@ public class DebuggerTraceManagerServicePlugin extends Plugin
});
}
else {
String filename = trace.getName();
DomainFolder root = tool.getProject().getProjectData().getRootFolder();
DomainFile existing = root.getFile(filename);
for (int i = 1; existing != null; i++) {
filename = trace.getName() + "." + i;
existing = root.getFile(filename);
}
DomainFolder traces;
try {
traces = createOrGetFolder(tool, "Save New Trace", root, NEW_TRACES_FOLDER_NAME);
@ -968,12 +962,20 @@ public class DebuggerTraceManagerServicePlugin extends Plugin
throw new AssertionError(e);
}
final String finalFilename = filename;
new TaskLauncher(new Task("Save New Trace", true, true, true) {
@Override
public void run(TaskMonitor monitor) throws CancelledException {
String filename = trace.getName();
try (DomainObjectLockHold hold = maybeLock(trace, force)) {
traces.createFile(finalFilename, trace, monitor);
for (int i = 1;; i++) {
try {
traces.createFile(filename, trace, monitor);
break;
}
catch (DuplicateFileException e) {
filename = trace.getName() + "." + i;
}
}
trace.save("Initial save", monitor);
future.complete(null);
}