Skip to content

Add array checks to packetbeat procfs parser#48428

Merged
fearful-symmetry merged 8 commits intoelastic:mainfrom
fearful-symmetry:proc-parser-packetbeat-fix
Jan 20, 2026
Merged

Add array checks to packetbeat procfs parser#48428
fearful-symmetry merged 8 commits intoelastic:mainfrom
fearful-symmetry:proc-parser-packetbeat-fix

Conversation

@fearful-symmetry
Copy link
Contributor

Proposed commit message

This cleans up the procfs parser code in packet beat to make is significantly more readable, and also adds an array length check.

In draft while I write a quick test.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works. Where relevant, I have used the stresstest.sh script to run them under stress conditions and race detector to verify their stability.
  • I have added an entry in ./changelog/fragments using the changelog tool.
@fearful-symmetry fearful-symmetry self-assigned this Jan 14, 2026
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 14, 2026
@github-actions
Copy link
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
@mergify
Copy link
Contributor

mergify bot commented Jan 14, 2026

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @fearful-symmetry? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.
@fearful-symmetry fearful-symmetry marked this pull request as ready for review January 16, 2026 18:46
@fearful-symmetry fearful-symmetry requested a review from a team as a code owner January 16, 2026 18:46
@fearful-symmetry fearful-symmetry added backport-active-all Automated backport with mergify to all the active branches and removed needs_team Indicates that the issue/PR needs a Team:* label labels Jan 20, 2026
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 20, 2026
@fearful-symmetry fearful-symmetry added the Team:Security-Linux Platform Linux Platform Team in Security Solution label Jan 20, 2026
@elasticmachine
Copy link
Contributor

Pinging @elastic/sec-linux-platform (Team:Security-Linux Platform)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 20, 2026
@fearful-symmetry fearful-symmetry merged commit 9410984 into elastic:main Jan 20, 2026
45 of 46 checks passed
@github-actions
Copy link
Contributor

@Mergifyio backport 8.19 9.1 9.2 9.3

@mergify
Copy link
Contributor

mergify bot commented Jan 20, 2026

backport 8.19 9.1 9.2 9.3

✅ Backports have been created

Details
mergify bot pushed a commit that referenced this pull request Jan 20, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)
mergify bot pushed a commit that referenced this pull request Jan 20, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)
mergify bot pushed a commit that referenced this pull request Jan 20, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)
mergify bot pushed a commit that referenced this pull request Jan 20, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)
pierrehilbert pushed a commit that referenced this pull request Jan 22, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)

Co-authored-by: Alex K. <8418476+fearful-symmetry@users.noreply.github.com>
pierrehilbert pushed a commit that referenced this pull request Jan 23, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)

Co-authored-by: Alex K. <8418476+fearful-symmetry@users.noreply.github.com>
pierrehilbert pushed a commit that referenced this pull request Jan 23, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)

Co-authored-by: Alex K. <8418476+fearful-symmetry@users.noreply.github.com>
fearful-symmetry added a commit that referenced this pull request Jan 23, 2026
* init commit

* fix int parsing

* linter, uint16 cast

* changelog

* linter cleanup

* add test

* remove old IO lib

(cherry picked from commit 9410984)

Co-authored-by: Alex K. <8418476+fearful-symmetry@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-active-all Automated backport with mergify to all the active branches bugfix Team:Security-Linux Platform Linux Platform Team in Security Solution

3 participants