Fix DumpWords: use uint.TryParse to handle word values >= 0x80000000

Author: divadiow

BK7231Flasher/Flashers/RTLZ2Flasher.cs

@@ -38,7 +38,7 @@ public class RTLZ2Flasher : BaseFlasher
 		const int HashRetryLimit = 3;
 		const int CommandRetryLimit = 3;
 		const int FallbackBaudRate = 115200;
-		const string InternalBuildId = "rtlz2-resiliency-r25";
+		const string InternalBuildId = "rtlz2-resiliency-r26";
 
 		public RTLZ2Flasher(CancellationToken ct) : base(ct)
 		{
@@ -478,11 +478,11 @@ bool DumpBytes(uint start, int count, out byte[] bytes)
 			return true;
 		}
 
-		bool DumpWords(uint start, int count, out int[] ints)
+		bool DumpWords(uint start, int count, out uint[] words)
 		{
 			int bytesRead = 0;
 			int expectedBytes = count * 4;
-			var data = new List<int>(Math.Max(1, count));
+			var data = new List<uint>(Math.Max(1, count));
 			Command($"DW {start:X} {count}");
 			// DW outputs ~11.75 ASCII chars per 4-byte word (address + 4 hex words per line + separators).
 			// Use a baud-aware deadline matching DumpBytes to avoid false timeouts at any baud rate.
@@ -526,7 +526,10 @@ bool DumpWords(uint start, int count, out int[] ints)
 
 					for(int i = 1; i < 5 && bytesRead < expectedBytes; i++)
 					{
-						if(!int.TryParse(parts[i].Trim('\r'), System.Globalization.NumberStyles.HexNumber, null, out var value))
+						// uint.TryParse handles the full 32-bit range (0x00000000–0xFFFFFFFF).
+						// int.TryParse would silently fail for values >= 0x80000000, corrupting
+						// flash data with high bits set.
+						if(!uint.TryParse(parts[i].Trim('\r'), System.Globalization.NumberStyles.HexNumber, null, out var value))
 						{
 							throw new Exception("Invalid word dump data");
 						}
@@ -542,18 +545,18 @@ bool DumpWords(uint start, int count, out int[] ints)
 
 			if(bytesRead != expectedBytes || data.Count != count)
 			{
-				ints = null;
+				words = null;
 				return false;
 			}
 
-			ints = data.ToArray();
+			words = data.ToArray();
 			return true;
 		}
 
 		// Flash-read variant of DumpWords: takes a byte count (must be a multiple of 4),
 		// calls DumpWords with the mapped flash address, and converts the returned 32-bit words
-		// to a byte array. ARM is little-endian, so BitConverter.GetBytes preserves byte order
-		// correctly on any LE host (x86/x64 Windows or Linux).
+		// to a byte array. ARM is little-endian, so BitConverter.GetBytes(uint) preserves byte
+		// order correctly on any LE host (x86/x64 Windows or Linux).
 		bool DumpFlashWords(uint start, int byteCount, out byte[] bytes)
 		{
 			int wordCount = byteCount / 4;
@@ -580,7 +583,7 @@ int RegisterRead(uint addr)
 			{
 				throw new Exception($"Register read index failed at 0x{addr:X}");
 			}
-			return words[index];
+			return (int)words[index];
 		}
 
 		void RegisterWrite(uint addr, uint value)