From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Subject: autofs: small cleanup in autofs_getpath() We don't set "*name" so it's slightly nicer to just pass "name" instead of "&name". Link: http://lkml.kernel.org/r/20180531064736.lnisb55eajwjynvk@kili.mountain Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: "Eric W. Biederman" <ebiederm@xxxxxxxxxxxx> Acked-by: Ian Kent <raven@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/autofs/waitq.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff -puN fs/autofs/waitq.c~autofs-small-cleanup-in-autofs_getpath fs/autofs/waitq.c --- a/fs/autofs/waitq.c~autofs-small-cleanup-in-autofs_getpath +++ a/fs/autofs/waitq.c @@ -179,7 +179,7 @@ static void autofs_notify_daemon(struct } static int autofs_getpath(struct autofs_sb_info *sbi, - struct dentry *dentry, char **name) + struct dentry *dentry, char *name) { struct dentry *root = sbi->sb->s_root; struct dentry *tmp; @@ -189,7 +189,7 @@ static int autofs_getpath(struct autofs_ unsigned seq; rename_retry: - buf = *name; + buf = name; len = 0; seq = read_seqbegin(&rename_lock); @@ -395,7 +395,7 @@ int autofs_wait(struct autofs_sb_info *s if (IS_ROOT(dentry) && autofs_type_trigger(sbi->type)) qstr.len = sprintf(name, "%p", dentry); else { - qstr.len = autofs_getpath(sbi, dentry, &name); + qstr.len = autofs_getpath(sbi, dentry, name); if (!qstr.len) { kfree(name); return -ENOENT; _ -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html