| Message ID | 1467650933-20089-1-git-send-email-luisbg@osg.samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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.
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... -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
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 {
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(-)