[AtModem] Fix NullReferenceException in AtChannel.ReaderLoopAsync (#1216)

rikkreeftenberg · Ellerbach · web-flow · commit fea4ee8f09ae · 2025-01-04T08:41:58.000+01:00
Co-authored-by: Laurent Ellerbach &lt;laurelle@microsoft.com&gt;
diff --git a/devices/AtModem/AtParser/AtChannel.cs b/devices/AtModem/AtParser/AtChannel.cs
@@ -335,8 +335,11 @@ public AtResponse SendFullCommand(AtCommand command, CancellationToken cancellat
             }
             finally
             {
-                _currentCommand = default;
-                _currentResponse = default;
+                lock (_lock)
+                {
+                    _currentCommand = default;
+                    _currentResponse = default;
+                }
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -335,8 +335,11 @@ public AtResponse SendFullCommand(AtCommand command, CancellationToken cancellat`
`335`	`335`	`}`
`336`	`336`	`finally`
`337`	`337`	`{`
`338`		`- _currentCommand = default;`
`339`		`- _currentResponse = default;`
	`338`	`+ lock (_lock)`
	`339`	`+ {`
	`340`	`+ _currentCommand = default;`
	`341`	`+ _currentResponse = default;`
	`342`	`+ }`
`340`	`343`	`}`
`341`	`344`	`}`
`342`	`345`