2018-02-15 02:34:14

by Richard Guy Briggs

[permalink] [raw]
Subject: [PATCH] namei: fix hardlink speeling error

Fix handlink to hardlink.

Signed-off-by: Richard Guy Briggs <[email protected]>
---
fs/namei.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

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))
--
1.8.3.1



2018-02-15 03:45:06

by Matthew Wilcox

[permalink] [raw]
Subject: Re: [PATCH] namei: fix hardlink speeling error

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?

2018-02-15 10:08:57

by Philipp Hahn

[permalink] [raw]
Subject: Re: [PATCH] namei: fix hardlink speeling error

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

2018-02-15 12:03:21

by Richard Guy Briggs

[permalink] [raw]
Subject: Re: [PATCH] namei: fix hardlink speeling error

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 <[email protected]>
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