Skip to content

Conversation

geyslan
Copy link
Member

@geyslan geyslan commented Sep 4, 2025

1. Explain what the PR does

ea089c8 feat(ebpf): add save_bytes_to_buf_max() function

save_bytes_to_buf() now wraps the new save_bytes_to_buf_max() function,
which accepts an additional parameter to specify the maximum number of
bytes to save.

2. Explain how to test it

3. Other comments

save_bytes_to_buf() now wraps the new save_bytes_to_buf_max() function,
which accepts an additional parameter to specify the maximum number of
bytes to save.
@geyslan geyslan requested a review from yanivagman September 4, 2025 19:07
@geyslan geyslan self-assigned this Sep 4, 2025
@Copilot Copilot AI review requested due to automatic review settings September 4, 2025 19:07
Copy link

@Copilot 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

This PR introduces a new parameterized function save_bytes_to_buf_max() that allows configurable maximum size limits when saving byte arrays to buffers in the eBPF code. The existing save_bytes_to_buf() function is refactored to wrap the new function with the default MAX_BYTES_ARR_SIZE limit.

Key changes:

  • Added save_bytes_to_buf_max() function with configurable maximum size parameter
  • Refactored existing save_bytes_to_buf() to use the new function as a wrapper
  • Enhanced documentation with detailed parameter descriptions and function behavior

Copy link
Collaborator

@yanivagman yanivagman left a comment

Choose a reason for hiding this comment

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

LGTM

@geyslan geyslan merged commit 4a028ec into aquasecurity:main Sep 5, 2025
107 of 111 checks passed
@geyslan geyslan deleted the geyslan/save-buf-max branch September 10, 2025 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants