diff options
| author | J.C. Jones <jjones@mozilla.com> | 2018-03-22 07:13:05 -0700 |
|---|---|---|
| committer | J.C. Jones <jjones@mozilla.com> | 2018-03-22 07:13:05 -0700 |
| commit | 39596a619367115c6341b6baab39d0b5cf03a170 (patch) | |
| tree | e7fd3b3b854970832e116138cd29cc5c44b92a4b | |
| parent | 04607a28949e0a46b7f387ae7d630a45d2f92088 (diff) | |
| download | nss-hg-39596a619367115c6341b6baab39d0b5cf03a170.tar.gz | |
Bug 1447816 - Fix assignment where a comparison intended r=mt
Summary:
Coverity found an unintended assignment in dtls_GatherData in a PORT_Assert,
which is only evaluated in debug builds.
Reviewers: mt
Reviewed By: mt
Bug #: 1447816
Differential Revision: https://phabricator.services.mozilla.com/D787
| -rw-r--r-- | lib/ssl/ssl3gthr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ssl/ssl3gthr.c b/lib/ssl/ssl3gthr.c index 0f01c0107..5ea7cc249 100644 --- a/lib/ssl/ssl3gthr.c +++ b/lib/ssl/ssl3gthr.c @@ -350,7 +350,7 @@ dtls_GatherData(sslSocket *ss, sslGather *gs, int flags) } else if (headerLen == 7) { gs->remainder = (gs->hdr[5] << 8) | gs->hdr[6]; } else { - PORT_Assert(headerLen = 2); + PORT_Assert(headerLen == 2); gs->remainder = gs->dtlsPacket.len - gs->dtlsPacketOffset; } |