aws / aws/aws-encryption-sdk-c

Fix missing coverage for aws_cryptosdk_priv_fail_session

Open
#246 0 comments 0 reactions 0 assignees View on GitHub
maintenance
Dominant language
C
Stars
63
Forks
59
PR merge metrics
No merged PRs in 30d

Description

Nothing currently ends up calling this in the tests. Probably we can consolidate some of the session_process codepaths to call it instead of open-coding the same sort of logic.

Contributor guide

Open the contributing guide

Research direction

Locate aws_cryptosdk_priv_fail_session and the session_process codepaths, then inspect the existing tests that exercise those paths. Consolidate the relevant path so the function is called rather than duplicating its logic, and verify that tests execute the function and improve coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
cryptography, security, testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.