On Mon, Jan 2, 2023 at 3:45 PM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: > > Use correct kernel-doc syntax in the function description to > prevent a kernel-doc warning: > > security/integrity/ima/ima_policy.c:1964: warning: expecting prototype for ima_delete_rules() called to cleanup invalid in(). Prototype was for ima_delete_rules() instead > > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > Cc: Mimi Zohar <zohar@xxxxxxxxxxxxx> > Cc: Dmitry Kasatkin <dmitry.kasatkin@xxxxxxxxx> > Cc: linux-integrity@xxxxxxxxxxxxxxx > Cc: Paul Moore <paul@xxxxxxxxxxxxxx> > Cc: James Morris <jmorris@xxxxxxxxx> > Cc: "Serge E. Hallyn" <serge@xxxxxxxxxx> > --- > security/integrity/ima/ima_policy.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Reviewed-by: Paul Moore <paul@xxxxxxxxxxxxxx> > diff -- a/security/integrity/ima/ima_policy.c b/security/integrity/ima/ima_policy.c > --- a/security/integrity/ima/ima_policy.c > +++ b/security/integrity/ima/ima_policy.c > @@ -1955,7 +1955,8 @@ ssize_t ima_parse_add_rule(char *rule) > } > > /** > - * ima_delete_rules() called to cleanup invalid in-flight policy. > + * ima_delete_rules() - called to cleanup invalid in-flight policy. > + * > * We don't need locking as we operate on the temp list, which is > * different from the active one. There is also only one user of > * ima_delete_rules() at a time. -- paul-moore.com