On Wednesday, November 12, 2014 04:25:15 PM Richard Guy Briggs wrote: > On 14/11/12, Paul Moore wrote: > > My apologies, I should have noticed this sooner, but printk_once() is > > probably not a good choice here as only the first invalid netlink message > > will be displayed. This is fine if all the invalid netlink messages > > happen the same, but that isn't likely to be the case. > > This was the same situation with WARN_ONCE(), hence my comment about > difficulty in debugging... Unfortunately I didn't realize your point until now. > > Richard, any objections if I convert the printk_once() to a printk(WARN) > > and update the patch description accordingly? > > Use pr_warn() instead... Normally, yes, but the rest of the SELinux code uses printk(WARN) and I'm a stickler for consistency. -- paul moore security and virtualization @ redhat _______________________________________________ Selinux mailing list Selinux@xxxxxxxxxxxxx To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx. To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.
