Fix underflowing SECTION FRAGMENT fixed address in RGBLINK (#2122)

src/link/section.cpp

@@ -56,6 +56,18 @@ static void checkPieceCompat(Section &target, Section const &other, size_t delta
 				    other.org
 				);
 			}
+		} else if (other.org < delta) {
+			// Check that `org` did not underflow. Note that it's safe for the above checks to
+			// use an underflowed value, since their reported errors will still be accurate.
+			fatalTwoAt(
+			    target,
+			    other,
+			    "Section \"%s\" contains %" PRIu16
+			    " bytes, but is also defined with a lower fixed address $%04" PRIx16,
+			    target.name.c_str(),
+			    target.size,
+			    other.org
+			);
 		}
 
 		target.isAddressFixed = true;

test/link/section-fragment/org-beyond-size/a.asm

@@ -0,0 +1,2 @@
+SECTION FRAGMENT "f", ROM0
+ds 32, 0

test/link/section-fragment/org-beyond-size/b.asm

@@ -0,0 +1,2 @@
+SECTION FRAGMENT "f", ROM0[$10]
+db 42

test/link/section-fragment/org-beyond-size/out.err

@@ -0,0 +1,5 @@
+FATAL: Section "f" contains 32 bytes, but is also defined with a lower fixed address $0010
+    at section-fragment/org-beyond-size/a.asm(1)
+    and also:
+    at section-fragment/org-beyond-size/b.asm(1)
+Linking aborted with 1 error

test/link/test.sh

@@ -417,6 +417,15 @@ rgblinkQuiet -o "$gbtemp" "$otemp" "$gbtemp2"
 tryCmpRom "$test"/ref.out.bin
 evaluateTest
 
+test="section-fragment/org-beyond-size"
+startTest
+"$RGBASM" -o "$otemp" "$test"/a.asm
+"$RGBASM" -o "$gbtemp" "$test"/b.asm
+continueTest
+rgblinkQuiet "$otemp" "$gbtemp" 2>"$outtemp"
+tryDiff "$test"/out.err "$outtemp"
+evaluateTest
+
 test="section-normal/same-name"
 startTest
 "$RGBASM" -o "$otemp" "$test"/a.asm