Closed Bug 1760653 Opened 3 years ago Closed 3 years ago

Use of uninitialized memory in ssl gtests when checking connection status

Categories

(NSS :: Test, defect, P3)

Product:
NSS
Component:
Test
Version:
3.67
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: djackson, Assigned: djackson)

Details

Attachments

(1 file)

Bug 1760653 - Ensure we don't read uninitialized memory in ssl gtests. r=mt,#nss-reviewers
48 bytes, text/x-phabricator-request
Details | Review

Currently TlsConnectTestBase::CheckConnected EXPECT_TRUEs fetching the ciphersuite. If the ciphersuite isn't set, this will error but not abort the test, leading to an uninitialized read caught by valgrind. This doesn't pose any real issue but could mask other problems.

https://hg.mozilla.org/projects/nss/rev/9a7b3c7f4e705455b21679ffc12d914d8db1f5a8

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: