MslControl missing negation operator on new channel input stream

Netflix / msl

Message Security Layer

Apache License 2.0

716 stars 79 forks source link

MslControl missing negation operator on new channel input stream #280

Closed wmiaw closed 6 years ago

wmiaw commented 6 years ago

MslControl.cpp:1946 is missing a negation operator inside the conditional when checking for no new response (input stream). The correct logic, which matches the other languages, is:

if (maxMessagesHit || (newChannel && !newChannel->input()))