Election-Tech-Initiative / Election-Tech-Initiative/electionguard-cpp

🐞 Check classes for leaks

オープン
#329 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
bug
主要言語
C++
スター
26
フォーク
25
PR マージ指標
30日以内にマージされた PR はありません

説明

### Is there an existing issue for this?

- [X] I have searched the existing issues

### Current Behavior

The .NET implementation of the manifest, context, encryption device, could be leaking memory like the ballots did. This was due to the Handle to the C++ structures were getting closed before they were calling the code for Freeing the memory.

The rest of the classes should be looked at to make sure if they are leaking or not.

### Expected Behavior

_No response_

### Steps To Reproduce

_No response_

### Environment

```markdown
- OS:
```

### Anything else?

_No response_

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

.NET のマニフェスト、コンテキスト、暗号化デバイスの実装を確認し、その後、他のクラスについても、issue で説明されているものと同じハンドルおよびネイティブメモリのライフタイムに関する動作を調査します。まず、C++ 構造体のハンドルがメモリ解放呼び出しに対していつクローズされるのかを追跡します。クラスの確認が完了し、同様のリークがあれば対処され、リポジトリが対応している箇所ではテストまたは検証が追加されていれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
cpp, csharp
領域
cryptography
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。