Merge remote-tracking branch 'origin/patch'

Author: ryanmkurtz

Ghidra/Features/Base/src/main/java/ghidra/app/util/bin/format/macho/commands/SymbolTableCommand.java

@@ -42,7 +42,7 @@ public class SymbolTableCommand extends LoadCommand {
 	private long stroff;
 	private long strsize;
 
-	private List<NList> symbols = new ArrayList<NList>();
+	private List<NList> symbols = new ArrayList<>();
 
 	/**
 	 * Creates and parses a new {@link SymbolTableCommand}
@@ -144,7 +144,7 @@ public NList getSymbolAt(int index) {
 			(index & DynamicSymbolTableConstants.INDIRECT_SYMBOL_ABS) != 0) {
 			return null;
 		}
-		if (index > symbols.size()) {
+		if (index >= symbols.size()) {
 			return null;
 		}
 		return symbols.get(index);

Ghidra/Features/Base/src/main/java/ghidra/app/util/opinion/MachoProgramBuilder.java

@@ -684,7 +684,8 @@ protected void processStubs() throws Exception {
 			for (int i = indirectSymbolTableIndex; i < indirectSymbolTableIndex + nSymbols; ++i) {
 				monitor.increment();
 				int symbolIndex = indirectSymbols.get(i);
-				NList symbol = symbolTableCommand.getSymbolAt(symbolIndex);
+				NList symbol =
+					symbolTableCommand != null ? symbolTableCommand.getSymbolAt(symbolIndex) : null;
 				String name = null;
 				if (symbol != null && !symbol.getString().isBlank()) {
 					name = SymbolUtilities.replaceInvalidChars(symbol.getString(), true);