From 75964f2f50dda20eaf20639ddffc8947e643e1d8 Mon Sep 17 00:00:00 2001 From: Ryan Kurtz Date: Mon, 31 Aug 2026 11:16:42 -0400 Subject: [PATCH] GP-7176: Checking for unsupported paths during library loading --- .../app/util/opinion/AbstractLibrarySupportLoader.java | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/Ghidra/Features/Base/src/main/java/ghidra/app/util/opinion/AbstractLibrarySupportLoader.java b/Ghidra/Features/Base/src/main/java/ghidra/app/util/opinion/AbstractLibrarySupportLoader.java index 95ea18face..4c70362c12 100644 --- a/Ghidra/Features/Base/src/main/java/ghidra/app/util/opinion/AbstractLibrarySupportLoader.java +++ b/Ghidra/Features/Base/src/main/java/ghidra/app/util/opinion/AbstractLibrarySupportLoader.java @@ -841,6 +841,15 @@ public abstract class AbstractLibrarySupportLoader extends AbstractProgramLoader } if (results.isEmpty() && isAbsoluteLibraryPath(library)) { + if (library.startsWith("\\\\")) { + log.appendMsg("Skipping library '%s' with UNC path".formatted(library)); + return results; + } + if (library.chars().anyMatch(Character::isISOControl)) { + log.appendMsg("Skipping library '%s' that contains control characters" + .formatted(library)); + return results; + } LocalFileSystem localFS = FileSystemService.getInstance().getLocalFS(); GFile file = lookupLibraryInFs(library, localFS); Optional.ofNullable(file).ifPresent(results::add);