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

Why do we need #ifndef HAVE_AUDIT_SYSCALLS in bin/auditdistd/parse.y? #18

Open
0mp opened this issue Oct 31, 2017 · 1 comment
Open

Comments

@0mp
Copy link
Contributor

0mp commented Oct 31, 2017

What is the reason behind this piece of code in bin/auditdistd/parse.y:

#ifndef HAVE_AUDIT_SYSCALLS
pjdlog_error("Sender functionality is not available.");
return (1);
#endif

I'm trying to port auditdistd over to Linux, which obviously doesn't support HAVE_AUDIT_SYSCALLS (as fair as I know). I wasn't able to find out, however, what syscalls are needed by auditdistd to run the presence of which is tested with HAVE_AUDIT_SYSCALLS.

Thanks!

References

@0mp 0mp changed the title Why do we need #ifndef HAVE_AUDIT_SYSCALLS in bin/auditdistd/parse.y Why do we need #ifndef HAVE_AUDIT_SYSCALLS in bin/auditdistd/parse.y? Oct 31, 2017
0mp added a commit to 0mp/openbsm that referenced this issue Feb 1, 2018
This ifdef prevents auditdistd from running on Linux. I tried to understand
what is the reason behind even putting those lines into the code but to no
avail.  There is an issue open regarding the importance of those lines in the
OpenBSM repository on GitHub: openbsm#18
0mp added a commit to 0mp/openbsm that referenced this issue Mar 6, 2018
This ifdef prevents auditdistd from running on Linux. I tried to understand
what is the reason behind even putting those lines into the code but to no
avail.  There is an issue open regarding the importance of those lines in the
OpenBSM repository on GitHub: openbsm#18
0mp added a commit to 0mp/openbsm that referenced this issue Mar 16, 2018
This ifdef prevents auditdistd from running on Linux. I tried to understand
what is the reason behind even putting those lines into the code but to no
avail.  There is an issue open regarding the importance of those lines in the
OpenBSM repository on GitHub: openbsm#18
0mp added a commit to 0mp/openbsm that referenced this issue Mar 16, 2018
This ifdef prevents auditdistd from running on Linux. I tried to understand
what is the reason behind even putting those lines into the code but to no
avail.  There is an issue open regarding the importance of those lines in the
OpenBSM repository on GitHub: openbsm#18
0mp added a commit to 0mp/openbsm that referenced this issue Mar 16, 2018
This ifdef prevents auditdistd from running on Linux. I tried to understand
what is the reason behind even putting those lines into the code but to no
avail.  There is an issue open regarding the importance of those lines in the
OpenBSM repository on GitHub: openbsm#18
@0mp
Copy link
Contributor Author

0mp commented Apr 25, 2018

As per this ancient commit message:

Use HAVE_AUDIT_SYSCALLS to detect if we are being compiled on a
system that will also use our auditd. If HAVE_AUDIT_SYSCALLS is
not define, return an error when 'sender' configuration exists
in the configuration file

It looks like we could remove it in the future when auditdistd is finally ported to run on Linux.

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

No branches or pull requests

1 participant