Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm...

[platform/kernel/linux-starfive.git] / kernel / pid.c

diff --git a/kernel/pid.c b/kernel/pid.c
index efd3487..bc21c0f 100644 (file)
--- a/kernel/pid.c
+++ b/kernel/pid.c
@@ -244,6 +244,16 @@ struct pid *alloc_pid(struct pid_namespace *ns, pid_t *set_tid,
                tmp = tmp->parent;
        }
 
+       /*
+        * ENOMEM is not the most obvious choice especially for the case
+        * where the child subreaper has already exited and the pid
+        * namespace denies the creation of any new processes. But ENOMEM
+        * is what we have exposed to userspace for a long time and it is
+        * documented behavior for pid namespaces. So we can't easily
+        * change it even if there were an error code better suited.
+        */
+       retval = -ENOMEM;
+
        get_pid_ns(ns);
        refcount_set(&pid->count, 1);
        for (type = 0; type < PIDTYPE_MAX; ++type)