On 10/03, Sukadev Bhattiprolu wrote: > > static void reparent_thread(struct task_struct *father, struct task_struct *p, > struct list_head *dead) > { > - if (p->pdeath_signal) > - group_send_sig_info(p->pdeath_signal, SEND_SIG_NOINFO, p); > + if (p->pdeath_signal) { > + struct siginfo info; > + > + info.si_code = SI_USER; > + info.si_signo = p->pdeath_signal; > + info.si_errno = 0; > + > + rcu_read_lock(); > + info.si_pid = task_tgid_nr_ns(father, task_active_pid_ns(p)); > + info.si_uid = __task_cred(father)->uid; > + rcu_read_unlock(); > + > + group_send_sig_info(p->pdeath_signal, &info, p); > + } I think the patch is correct. But afaics we should clarify the "from user" semantics and fix send_signal() instead. What do you think about this simple series? (the last 2 patches are pure cosmetic and off-topic). Oleg. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers