diff options
| author | Hui Su <sh_def@163.com> | 2020-11-18 00:17:50 +0800 |
|---|---|---|
| committer | Christian Brauner <christian.brauner@ubuntu.com> | 2020-11-18 11:06:47 +0100 |
| commit | 5c62634fc65101d350cbd47722fb76f02693059d (patch) | |
| tree | 64293c407c33853d3164e8d14a13a8792213b82d /kernel/time | |
| parent | 716e343f014e2b25320f332677363e884684b742 (diff) | |
namespace: make timens_on_fork() return nothing
timens_on_fork() always return 0, and maybe not
need to judge the return value in copy_namespaces().
So make timens_on_fork() return nothing and do not
judge its return val in copy_namespaces().
Signed-off-by: Hui Su <sh_def@163.com>
Link: https://lore.kernel.org/r/20201117161750.GA45121@rlk
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
Diffstat (limited to 'kernel/time')
| -rw-r--r-- | kernel/time/namespace.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/kernel/time/namespace.c b/kernel/time/namespace.c index afc65e6be33e..e0f9509b17c3 100644 --- a/kernel/time/namespace.c +++ b/kernel/time/namespace.c @@ -308,22 +308,20 @@ static int timens_install(struct nsset *nsset, struct ns_common *new) return 0; } -int timens_on_fork(struct nsproxy *nsproxy, struct task_struct *tsk) +void timens_on_fork(struct nsproxy *nsproxy, struct task_struct *tsk) { struct ns_common *nsc = &nsproxy->time_ns_for_children->ns; struct time_namespace *ns = to_time_ns(nsc); /* create_new_namespaces() already incremented the ref counter */ if (nsproxy->time_ns == nsproxy->time_ns_for_children) - return 0; + return; get_time_ns(ns); put_time_ns(nsproxy->time_ns); nsproxy->time_ns = ns; timens_commit(tsk, ns); - - return 0; } static struct user_namespace *timens_owner(struct ns_common *ns) |