Browse Source

Merge remote-tracking branch 'origin/master'

release/3.x.x
Christian Kratky 4 years ago
parent
commit
63c9329190
2 changed files with 17 additions and 2 deletions
  1. +1
    -0
      Build/MQTTnet.nuspec
  2. +16
    -2
      Source/MQTTnet/Implementations/MqttTcpChannel.cs

+ 1
- 0
Build/MQTTnet.nuspec View File

@@ -17,6 +17,7 @@
* [Server] Adjusted some namespaces (BREAKING CHANGE!)
* [Server] Added state checks (throw if not started etc.) for most server APIs.
* [Server] Exposed real X509Certificate2 (instead byte array) to TLS options (thanks to @borigas).
* [Core] Fixed a null reference exception in the MqttTcpChannel with WriteAsync and ReadAsync.
</releaseNotes>
<copyright>Copyright Christian Kratky 2016-2020</copyright>
<tags>MQTT Message Queue Telemetry Transport MQTTClient MQTTServer Server MQTTBroker Broker NETStandard IoT InternetOfThings Messaging Hardware Arduino Sensor Actuator M2M ESP Smart Home Cities Automation Xamarin Blazor</tags>


+ 16
- 2
Source/MQTTnet/Implementations/MqttTcpChannel.cs View File

@@ -127,7 +127,14 @@ namespace MQTTnet.Implementations
// Workaround for: https://github.com/dotnet/corefx/issues/24430
using (cancellationToken.Register(Dispose))
{
return await _stream.ReadAsync(buffer, offset, count, cancellationToken).ConfigureAwait(false);
var stream = _stream;

if (stream == null)
{
throw new ObjectDisposedException(nameof(stream));
}

return await stream.ReadAsync(buffer, offset, count, cancellationToken).ConfigureAwait(false);
}
}
catch (ObjectDisposedException)
@@ -157,7 +164,14 @@ namespace MQTTnet.Implementations
// Workaround for: https://github.com/dotnet/corefx/issues/24430
using (cancellationToken.Register(Dispose))
{
await _stream.WriteAsync(buffer, offset, count, cancellationToken).ConfigureAwait(false);
var stream = _stream;

if (stream == null)
{
throw new ObjectDisposedException(nameof(stream));
}

await stream.WriteAsync(buffer, offset, count, cancellationToken).ConfigureAwait(false);
}
}
catch (ObjectDisposedException)


Loading…
Cancel
Save