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.

Sign up for GitHub

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

Jump to bottom

fix redis parser panic #438

Merged
merged 1 commit into from
Dec 4, 2015
Merged

fix redis parser panic #438

merged 1 commit into from
Dec 4, 2015

Conversation

urso
Copy link

@urso urso commented Dec 3, 2015

Fix regression in redis command parsing accessing the array out of bounds if
command has no arguments.

fix regression introduced #384

fix redis parser panic
d15d1aa 
Fix regression in redis command parsing accessing the array out of bounds if
command has no arguments.
@ruflin
Copy link
Contributor

ruflin commented Dec 3, 2015

LGTM

andrewkroh added a commit that referenced this pull request Dec 4, 2015
@andrewkroh
Merge pull request #438 from urso/fix/regression-redis-command-no-args
55b5202 
fix redis parser panic
@andrewkroh andrewkroh merged commit 55b5202 into elastic:master Dec 4, 2015
@urso urso deleted the fix/regression-redis-command-no-args branch December 9, 2015 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Packetbeat review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@urso @ruflin @andrewkroh