On 15/08/05, Paul Moore wrote: > On Wednesday, August 05, 2015 04:19:09 PM Richard Guy Briggs wrote: > > Clean up a number of places were casted magic numbers are used to represent > > unset inode and device numbers in preparation for the audit by executable > > path patch set. > > > > Signed-off-by: Richard Guy Briggs <r...@redhat.com> > > --- > > v5: Move macros from include/uapi/linux/audit.h to include/linux/audit.h > > Use "unsigned int" rather than bare "unsigned". > > > > include/linux/audit.h | 3 +++ > > kernel/audit.c | 2 +- > > kernel/audit_watch.c | 8 ++++---- > > kernel/auditsc.c | 6 +++--- > > 4 files changed, 11 insertions(+), 8 deletions(-) > > > > diff --git a/include/linux/audit.h b/include/linux/audit.h > > index c2e7e3a..48ae90c 100644 > > --- a/include/linux/audit.h > > +++ b/include/linux/audit.h > > @@ -27,6 +27,9 @@ > > #include <linux/ptrace.h> > > #include <uapi/linux/audit.h> > > > > +#define AUDIT_INO_UNSET (unsigned long)-1 > > +#define AUDIT_DEV_UNSET (unsigned int)-1 > > I suspect it was lost in the noise when I mentioned it on v4, but how about > changing AUDIT_DEV_UNSET to "(dev_t)-1"?
I saw your comment only after resubmitting. I'm fine either way. If it is needed for uapi later it can be changed then. Is it easy to change in your workflow, or should I resubmit? I know you routinely change the patch description, but could not remember if you have actually changed the patch itself... > paul moore - RGB -- Richard Guy Briggs <rbri...@redhat.com> Senior Software Engineer, Kernel Security, AMER ENG Base Operating Systems, Red Hat Remote, Ottawa, Canada Voice: +1.647.777.2635, Internal: (81) 32635, Alt: +1.613.693.0684x3545 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
