security: Update file_set_fowner documentation

l0kod · pcmoore · commit 19c9d55d72a9 · 2024-09-09T12:30:51.000-04:00
Highlight that the file_set_fowner hook is now called with a lock held.

Cc: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
Cc: Casey Schaufler &lt;casey@schaufler-ca.com&gt;
Cc: Christian Brauner &lt;brauner@kernel.org&gt;
Cc: James Morris &lt;jmorris@namei.org&gt;
Cc: Jann Horn &lt;jannh@google.com&gt;
Cc: Ondrej Mosnacek &lt;omosnace@redhat.com&gt;
Cc: Paul Moore &lt;paul@paul-moore.com&gt;
Cc: Serge E. Hallyn &lt;serge@hallyn.com&gt;
Cc: Stephen Smalley &lt;stephen.smalley.work@gmail.com&gt;
Signed-off-by: Mickaël Salaün &lt;mic@digikod.net&gt;
Signed-off-by: Paul Moore &lt;paul@paul-moore.com&gt;
diff --git a/security/security.c b/security/security.c
@@ -3048,6 +3048,8 @@ int security_file_fcntl(struct file *file, unsigned int cmd, unsigned long arg)
  * Save owner security information (typically from current->security) in
  * file->f_security for later use by the send_sigiotask hook.
  *
+ * This hook is called with file->f_owner.lock held.
+ *
  * Return: Returns 0 on success.
  */
 void security_file_set_fowner(struct file *file)

Original file line number	Diff line number	Diff line change
`@@ -3048,6 +3048,8 @@ int security_file_fcntl(struct file *file, unsigned int cmd, unsigned long arg)`
`3048`	`3048`	`* Save owner security information (typically from current->security) in`
`3049`	`3049`	`* file->f_security for later use by the send_sigiotask hook.`
`3050`	`3050`	`*`
	`3051`	`+ * This hook is called with file->f_owner.lock held.`
	`3052`	`+ *`
`3051`	`3053`	`* Return: Returns 0 on success.`
`3052`	`3054`	`*/`
`3053`	`3055`	`void security_file_set_fowner(struct file *file)`