Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check err param in filepath.WalkFunc #6007

Merged
merged 1 commit into from
Jan 7, 2018

Conversation

andrewkroh
Copy link
Member

There was a missing error check in the file_integrity module's scanner that could result in a panic.

Fixes #6005

There was a missing error check in the file_integrity module's scanner that could result in a panic.

Fixes elastic#6005
@andrewkroh andrewkroh added Auditbeat bug needs_backport PR is waiting to be backported to other branches. v6.2.0 labels Jan 7, 2018
Copy link
Contributor

@tsg tsg left a comment

Choose a reason for hiding this comment

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

LGTM.

@tsg tsg merged commit e8db561 into elastic:master Jan 7, 2018
andrewkroh added a commit to andrewkroh/beats that referenced this pull request Jan 7, 2018
There was a missing error check in the file_integrity module's scanner that could result in a panic.

Fixes elastic#6005

(cherry picked from commit e8db561)
@andrewkroh andrewkroh removed the needs_backport PR is waiting to be backported to other branches. label Jan 7, 2018
exekias pushed a commit that referenced this pull request Jan 8, 2018
There was a missing error check in the file_integrity module's scanner that could result in a panic.

Fixes #6005

(cherry picked from commit e8db561)
@andrewkroh andrewkroh deleted the bugfix/ab/fim-scan-panic branch January 17, 2018 16:43
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
There was a missing error check in the file_integrity module's scanner that could result in a panic.

Fixes elastic#6005

(cherry picked from commit eca195a)
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