google / google/mundane

Test BoringSSL refcounting

Open
#14 2 comments 0 reactions 0 assignees View on GitHub
experience-medium
Dominant language
Rust
Stars
1.1k
Forks
42
PR merge metrics
No merged PRs in 30d

Description

Currently, we have a few small smoke tests to make sure that we don't decrement BoringSSL refcounts too far. However, we have no tests to ensure that we decrement refcounts far enough (in other words, that we don't leak resources by leaving unused objects allocated and constructed).

We should also test the latter. The way to do this is probably to figure out how to inspect the refcount on a BoringSSL object and use that mechanism to ensure that the refcount on these objects is 1 when we only hold a single reference. That said, perhaps there's a better way to do this.

EDIT: Let's first try to see if we can enable ASan (#15), which should address this.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the existing BoringSSL refcount smoke tests, then investigate issue #15 for the proposed ASan approach. The work is done when tests verify that unused BoringSSL objects are released appropriately, either through refcount inspection or the agreed sanitizer-based mechanism.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
testing-qa
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.