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

BUG: import the audit-validation code into a new repo on github.com/linux-audit #11

Open
pcmoore opened this issue Feb 24, 2016 · 2 comments
Assignees

Comments

@pcmoore
Copy link
Contributor

pcmoore commented Feb 24, 2016

Source: https://people.redhat.com/sgrubb/audit/audit-validation-0.1.tar.gz

@pcmoore pcmoore added the bug label Feb 24, 2016
@pcmoore pcmoore added this to the migration milestone Feb 24, 2016
@pcmoore pcmoore changed the title BUG: import the audit-validation code into the github.com/linux-audit BUG: import the audit-validation code into a new repo on github.com/linux-audit Feb 24, 2016
@pcmoore pcmoore modified the milestones: migration, repo migration May 4, 2016
@WOnder93
Copy link

Preliminary verbatim import of version 0.1 for review:

https://github.com/WOnder93/audit-validation

@WOnder93
Copy link

WOnder93 commented Jun 1, 2018

I had a quick look at the code and I have the impression that this tool is still a bit crude and unfinished. Basically, it currently does two things:

  • Checks whether the log contains system boot/shutdown/runlevel records in logical order.
  • Checks whether the log contains service and daemon start/end/abort records in logical order. The code for this feature seems outdated since it reports a lot of false positives for me (it looks at the comm field, which is always systemd on current Fedora...).

Based on the comments in the code and documentation it seems it should eventually also perform a similar validation of user-session-related records, but this is not yet implemented (there is just a stub function for that).

Given that the tool does only basic sanity checking that is prone to false positives and is unfinished, I don't recommend it for importing into linux-audit at this point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants