]>
Commit | Line | Data |
---|---|---|
9e8e5432 GKH |
1 | From 12d63702c53bc2230dfc997e91ca891f39cb6446 Mon Sep 17 00:00:00 2001 |
2 | From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
3 | Date: Tue, 18 Sep 2012 13:37:18 +0400 | |
4 | Subject: Revert: lockd: use rpc client's cl_nodename for id encoding | |
5 | ||
6 | From: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
7 | ||
8 | This reverts 12d63702c53bc2230dfc997e91ca891f39cb6446 which was commit | |
9 | 303a7ce92064c285a04c870f2dc0192fdb2968cb upstream. | |
10 | ||
11 | Taking hostname from uts namespace if not safe, because this cuold be | |
12 | performind during umount operation on child reaper death. And in this case | |
13 | current->nsproxy is NULL already. | |
14 | ||
49c00592 | 15 | Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
9e8e5432 GKH |
16 | Cc: Stanislav Kinsbursky <skinsbursky@parallels.com> |
17 | Cc: Trond Myklebust <Trond.Myklebust@netapp.com> | |
9e8e5432 GKH |
18 | |
19 | --- | |
20 | fs/lockd/mon.c | 4 +--- | |
21 | 1 file changed, 1 insertion(+), 3 deletions(-) | |
22 | ||
23 | --- a/fs/lockd/mon.c | |
24 | +++ b/fs/lockd/mon.c | |
25 | @@ -40,7 +40,6 @@ struct nsm_args { | |
26 | u32 proc; | |
27 | ||
28 | char *mon_name; | |
29 | - char *nodename; | |
30 | }; | |
31 | ||
32 | struct nsm_res { | |
33 | @@ -94,7 +93,6 @@ static int nsm_mon_unmon(struct nsm_hand | |
34 | .vers = 3, | |
35 | .proc = NLMPROC_NSM_NOTIFY, | |
36 | .mon_name = nsm->sm_mon_name, | |
37 | - .nodename = utsname()->nodename, | |
38 | }; | |
39 | struct rpc_message msg = { | |
40 | .rpc_argp = &args, | |
41 | @@ -431,7 +429,7 @@ static void encode_my_id(struct xdr_stre | |
42 | { | |
43 | __be32 *p; | |
44 | ||
45 | - encode_nsm_string(xdr, argp->nodename); | |
46 | + encode_nsm_string(xdr, utsname()->nodename); | |
47 | p = xdr_reserve_space(xdr, 4 + 4 + 4); | |
48 | *p++ = cpu_to_be32(argp->prog); | |
49 | *p++ = cpu_to_be32(argp->vers); |