[AtModem] Fix NullReferenceException in AtChannel.ProcessMessage (#1221)

rikkreeftenberg · Ellerbach · web-flow · commit 45bdffe18d25 · 2025-01-08T09:22:38.000Z
Co-authored-by: Laurent Ellerbach &lt;laurelle@microsoft.com&gt;
diff --git a/devices/AtModem/AtParser/AtChannel.cs b/devices/AtModem/AtParser/AtChannel.cs
@@ -409,7 +409,11 @@ private void ReaderLoopAsync()
                 }
                 else
                 {
-                    ProcessMessage(line1);
+                    // lock access to _currentCommand & _currentResponse
+                    lock (_lock)
+                    {
+                        ProcessMessage(line1);
+                    }
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -409,7 +409,11 @@ private void ReaderLoopAsync()`
`409`	`409`	`}`
`410`	`410`	`else`
`411`	`411`	`{`
`412`		`- ProcessMessage(line1);`
	`412`	`+ // lock access to _currentCommand & _currentResponse`
	`413`	`+ lock (_lock)`
	`414`	`+ {`
	`415`	`+ ProcessMessage(line1);`
	`416`	`+ }`
`413`	`417`	`}`
`414`	`418`	`}`
`415`	`419`	`}`