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

json: somewhere along the way IP/port pairs had gotten swapped in #915

Merged
merged 1 commit into from
Apr 2, 2014
Merged

json: somewhere along the way IP/port pairs had gotten swapped in #915

merged 1 commit into from
Apr 2, 2014

Conversation

decanio
Copy link
Contributor

@decanio decanio commented Apr 2, 2014

Fix swapped IP/port pairs for http and ssh eve json loggers.

@inliniac
Copy link
Contributor

inliniac commented Apr 2, 2014

Thanks Tom. In the future, please add the prscript output to the PR.

I've run it succesfully through now:

@inliniac inliniac merged commit 4085f08 into OISF:master Apr 2, 2014
alessandro-guido pushed a commit to alessandro-guido/suricata that referenced this pull request Jun 20, 2014
@poona @victorjulien
fix for OISF#915. Fix segv when we send NULL to snprintf.
d292f1a
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
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@decanio @inliniac