-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Error in ossec-reportd #1233
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
Comments
Seems to only occur when there is at least one matching alert:
|
I built v2.9.1 from source with
|
I tried again with different count of alerts, but it's the same. |
#1088 wasn't merged until 12 hours after 2.9.2 was finalized, so the patch to fix reportd isn't included in 2.9.2. ossec-hids/src/shared/report_op.c Line 525 in afd09b3
versus ossec-hids/src/shared/report_op.c Line 525 in cb2ac11
|
Tagged this for 2.9.3, we should have this out in the next few days. 3.0 snapshots are in the pipeline right now ahead of it |
Hi @atomicturtle is there anything we can do to help the 2.9.3 release get out? |
Absolutely, I could use a hand backporting any rule/decoder updates from master and fixes (the double-free for example). At the moment Im wrapping up on the 3.0 snapshots, but should be back into this one soon. |
Did this land in 2.9.3?
I also noticed it is still reporting its version as 2.9.2.
|
Looks like it: https://github.com/ossec/ossec-hids/blob/master/src/shared/report_op.c#L525 Can you provide a test case? It's not crashing for me in my simple tests. |
Here's an example given this alert.log:
|
It looks like the potential fix is not in the |
Ahhh, I was looking at master, not the branch. Sorry about that. |
Any chance of a |
Can anyone comment on whether this has been fixed? I am still seeing the error, and no word on the release of v2.9.3. Luckily I had backed up my previous version and simply replaced "ossec-reportd", which fixed the problem. Reading other comment threads, it looks like this has been reported since December 2015. |
@the-getdown It's fixed in master. |
Pretty sure I just pushed it to the 2.9.4 branch as well. So that could be tested. |
Thank you. Out of interest, when will v2.9.4 be available for download from ossec.github.io? Apologies, but I'm not a Linux/Github guru. I assume if I pull it from the code repository I will need to compile it? |
Yes you would. I think next week or the week after. I'd have to check with @atomicturtle |
Would there be any chance of backporting the fix to v2.9.2? I have installed OSSEC from the atomicorp debian stretch repository and am running 2.9.2 Thanks |
There's a chance. Backporting takes me away from doing the fun stuff, but I'll do it. It doesn't solve the problem of the lack of testing of a possible 2.9.4. I don't have any 2.9 hosts to test on |
Ok, the fix I had previously applied is in the v2.9.4 branch. |
I'm trying create a report.
cat /var/ossec/logs/alerts/alerts.log | /var/ossec/bin/ossec-reportd
2017/08/28 16:18:47 ossec-reportd: INFO: Started (pid: 4079).
*** Error in `/var/ossec/bin/ossec-reportd': free(): invalid pointer: 0x0000000000bdd388 ***
Aborted
cat /var/ossec/logs/alerts/alerts.log | /var/ossec/bin/ossec-reportd -f level 10
2017/08/28 16:18:57 ossec-reportd: INFO: Started (pid: 4083).
*** Error in `/var/ossec/bin/ossec-reportd': double free or corruption (out): 0x00000000021ba670 ***
Aborted
OSSEC version is v.9.2
The text was updated successfully, but these errors were encountered: