On 2017-05-04 17:05, Richard Guy Briggs wrote:
On 2017-05-04 16:49, Steve Grubb wrote:
> On Thursday, May 4, 2017 4:29:45 PM EDT Richard Guy Briggs wrote:
> > On 2017-05-04 16:11, Steve Grubb wrote:
> > > Was there a patch in this series that converted errormsg.h to use the
> > > macros?
> > I don't quite follow. Can you give a fictional example off the top of
> > your head of what you are hoping for?
>
> This table:
>
> static const struct msg_tab err_msgtab[] = {
> { -1, 2, "-F missing operation for" },
> { -2, 2, "-F unknown field:" },
> { -3, 1, "must be before -S" },
> { -4, 1, "machine type not found" },
> ...
>
> converted to using the defines. The libaudit return codes were fixed to
> defines. But the table the return codes are looked up in is still using
> numbers.
Ah, got it, yes, completely agree.
Ok, here is a pull request to make that change:
https://github.com/linux-audit/audit-userspace/pull/22
I've tagged it related to:
https://github.com/linux-audit/audit-userspace/issues/11
> -Steve
- RGB
- RGB
--
Richard Guy Briggs <rgb(a)redhat.com>
Sr. S/W Engineer, Kernel Security, Base Operating Systems
Remote, Ottawa, Red Hat Canada
IRC: rgb, SunRaycer
Voice: +1.647.777.2635, Internal: (81) 32635