Skip to content

Prep for libnat20 kernel module#81

Merged
werwurm merged 1 commit intomainfrom
werwurm/current_cdi
Feb 26, 2026
Merged

Prep for libnat20 kernel module#81
werwurm merged 1 commit intomainfrom
werwurm/current_cdi

Conversation

@werwurm
Copy link
Copy Markdown
Contributor

@werwurm werwurm commented Feb 26, 2026

Rename current to current_cdi, because it clashes with the Linux Kernel macro current.

Rename current to current_cdi, because it clashes with the Linux Kernel macro `current`.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Renames the n20_next_level_cdi_attest parameter from current to current_cdi to avoid a name clash with the Linux kernel current macro when preparing for a kernel-module build.

Changes:

  • Rename n20_next_level_cdi_attest’s CDI input parameter from current to current_cdi in the C implementation.
  • Update the public header prototype and its Doxygen parameter documentation to match.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/core/functionality.c Renames the function parameter and updates the internal call to n20_derive_key.
include/nat20/functionality.h Updates the exported API signature and Doxygen @param name to current_cdi.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

LCOV of commit b6a418f during lcov-test-coverage-report #53

Summary coverage rate:
  lines......: 96.1% (2420 of 2517 lines)
  functions..: 99.5% (194 of 195 functions)
  branches...: 87.3% (1373 of 1573 branches)

Files changed coverage rate: n/a

@werwurm werwurm merged commit 5132a32 into main Feb 26, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants