diff mbox

namei: fix hardlink speeling error

Message ID 68d4b8c4ad6b85d4336f2a2893e3ea04ffdc7794.1518661649.git.rgb@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Richard Guy Briggs Feb. 15, 2018, 2:28 a.m. UTC
Fix handlink to hardlink.

Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
---
 fs/namei.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Matthew Wilcox Feb. 15, 2018, 3:44 a.m. UTC | #1
On Wed, Feb 14, 2018 at 09:28:54PM -0500, Richard Guy Briggs wrote:
>  	 * This ensures that not everyone will be able to create
> -	 * handlink using the passed filedescriptor.
> +	 * hardlink using the passed filedescriptor.
>  	 */

You're changing this line and you didn't feel the need to fix the
missing space or the missing article?
Philipp Hahn Feb. 15, 2018, 9:57 a.m. UTC | #2
Hello,

Am 15.02.2018 um 03:28 schrieb Richard Guy Briggs:
> Fix handlink to hardlink.

and introduce a new sp*el*ling error in the subject line ;-)

Philipp
Richard Guy Briggs Feb. 15, 2018, 11:58 a.m. UTC | #3
On 2018-02-15 10:57, Philipp Hahn wrote:
> Hello,
> 
> Am 15.02.2018 um 03:28 schrieb Richard Guy Briggs:
> > Fix handlink to hardlink.
> 
> and introduce a new sp*el*ling error in the subject line ;-)

That one was quite intentional for ironic effect.  I could have added an
extra "e" for exaggerated effect if you like.  :-)

As for filedescriptor, that one I chose to leave alone due to 7 other
instances (out of ~700), while handlink stood alone (out of 126 hardlink
without and 77 with space) and was clearly misspelled in the context of
the source file.

I can file a followup patch to fix all "filedescriptor" in the same
patch if desired, but I'm not so sure that one is warranted.

> Philipp

- RGB

--
Richard Guy Briggs <rgb@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
diff mbox

Patch

diff --git a/fs/namei.c b/fs/namei.c
index bf1c046b..bbfb21d3 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -4258,7 +4258,7 @@  int vfs_link(struct dentry *old_dentry, struct inode *dir, struct dentry *new_de
 	/*
 	 * To use null names we require CAP_DAC_READ_SEARCH
 	 * This ensures that not everyone will be able to create
-	 * handlink using the passed filedescriptor.
+	 * hardlink using the passed filedescriptor.
 	 */
 	if (flags & AT_EMPTY_PATH) {
 		if (!capable(CAP_DAC_READ_SEARCH))