Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1436405 > unrolled thread
| Started by | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| First post | 2016-07-04 18:50 +0200 |
| Last post | 2016-07-05 15:40 +0200 |
| Articles | 4 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH] vfs: fix typo in link_path_walk() Luis de Bethencourt <luisbg@osg.samsung.com> - 2016-07-04 18:50 +0200
Re: [PATCH] vfs: fix typo in link_path_walk() Omar Sandoval <osandov@osandov.com> - 2016-07-04 22:20 +0200
Re: [PATCH] vfs: fix typo in link_path_walk() Al Viro <viro@ZenIV.linux.org.uk> - 2016-07-04 23:00 +0200
Re: [PATCH] vfs: fix typo in link_path_walk() Luis de Bethencourt <luisbg@osg.samsung.com> - 2016-07-05 15:40 +0200
| From | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| Date | 2016-07-04 18:50 +0200 |
| Subject | [PATCH] vfs: fix typo in link_path_walk() |
| Message-ID | <rRfei-7YV-15@gated-at.bofh.it> |
Comment documenting the path parsing in link_path_walk() has a typo, fixing
it.
Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
---
Hello,
Noticed this typo while studying the code paths when sys_open is called.
Thanks,
Luis
fs/namei.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/namei.c b/fs/namei.c
index 32f9fe9..ade7667 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -2064,7 +2064,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
if (!*name)
goto OK;
/*
- * If it wasn't NUL, we know it was '/'. Skip that
+ * If it wasn't NULL, we know it was '/'. Skip that
* slash, and continue until no more slashes.
*/
do {
--
2.6.4
[toc] | [next] | [standalone]
| From | Omar Sandoval <osandov@osandov.com> |
|---|---|
| Date | 2016-07-04 22:20 +0200 |
| Message-ID | <rRivv-1HR-1@gated-at.bofh.it> |
| In reply to | #1436405 |
On Mon, Jul 04, 2016 at 05:48:53PM +0100, Luis de Bethencourt wrote:
> Comment documenting the path parsing in link_path_walk() has a typo, fixing
> it.
>
> Signed-off-by: Luis de Bethencourt <luisbg@osg.samsung.com>
> ---
>
> Hello,
>
> Noticed this typo while studying the code paths when sys_open is called.
>
> Thanks,
> Luis
>
> fs/namei.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/namei.c b/fs/namei.c
> index 32f9fe9..ade7667 100644
> --- a/fs/namei.c
> +++ b/fs/namei.c
> @@ -2064,7 +2064,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
> if (!*name)
> goto OK;
> /*
> - * If it wasn't NUL, we know it was '/'. Skip that
> + * If it wasn't NULL, we know it was '/'. Skip that
> * slash, and continue until no more slashes.
> */
> do {
> --
> 2.6.4
NUL is the ASCII '\0' character. NULL is a pointer. So NUL is correct.
--
Omar
[toc] | [prev] | [next] | [standalone]
| From | Al Viro <viro@ZenIV.linux.org.uk> |
|---|---|
| Date | 2016-07-04 23:00 +0200 |
| Message-ID | <rRj8d-1UJ-1@gated-at.bofh.it> |
| In reply to | #1436405 |
On Mon, Jul 04, 2016 at 05:48:53PM +0100, Luis de Bethencourt wrote:
> Comment documenting the path parsing in link_path_walk() has a typo, fixing
> it.
> @@ -2064,7 +2064,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
> if (!*name)
> goto OK;
> /*
> - * If it wasn't NUL, we know it was '/'. Skip that
> + * If it wasn't NULL, we know it was '/'. Skip that
"If it wasn't <this>, we know it was <that>" really implies that <this> and
<that> are values possible for the same expression, doesn't it? How could
a pointer (NULL) and a character ('/') possibly be such?
Could you explain the meaning of thus "fixed" comment? I'm not even asking
to explain why it is correct that way, just what the hell is it supposed to
mean?
NAK, in case it's not obvious from the above...
[toc] | [prev] | [next] | [standalone]
| From | Luis de Bethencourt <luisbg@osg.samsung.com> |
|---|---|
| Date | 2016-07-05 15:40 +0200 |
| Message-ID | <rRyJY-3nN-9@gated-at.bofh.it> |
| In reply to | #1436592 |
On 04/07/16 21:50, Al Viro wrote:
> On Mon, Jul 04, 2016 at 05:48:53PM +0100, Luis de Bethencourt wrote:
>> Comment documenting the path parsing in link_path_walk() has a typo, fixing
>> it.
>
>> @@ -2064,7 +2064,7 @@ static int link_path_walk(const char *name, struct nameidata *nd)
>> if (!*name)
>> goto OK;
>> /*
>> - * If it wasn't NUL, we know it was '/'. Skip that
>> + * If it wasn't NULL, we know it was '/'. Skip that
>
> "If it wasn't <this>, we know it was <that>" really implies that <this> and
> <that> are values possible for the same expression, doesn't it? How could
> a pointer (NULL) and a character ('/') possibly be such?
>
> Could you explain the meaning of thus "fixed" comment? I'm not even asking
> to explain why it is correct that way, just what the hell is it supposed to
> mean?
>
> NAK, in case it's not obvious from the above...
>
Hi,
I was misinformed and thought NUL was only used as an abbreviation for the Null
character, and in a sentence the full name was meant to be used. As in,
null-terminated string. I didn't mean NULL pointer, since char *name is
dereferenced in the if check.
I see now that NULL can be more ambiguous than NUL or '\0'.
Sorry about that, I should had thought better before posting.
Apologies,
Luis
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web