[libvirt] Re: [Libvir] [PATCH] lxc: handle SIGCHLD from exiting container

Dave Leskovec <dlesko(a)linux.vnet.ibm.com&gt; wrote: > Jim Meyering wrote: >> Dave Leskovec <dlesko(a)linux.vnet.ibm.com&gt; wrote:

> Index: b/qemud/qemud.c > =================================================================== ... > +static void sig_handler(int sig, siginfo_t * siginfo, > + void* context ATTRIBUTE_UNUSED) { ... > - unsigned char sigc; > + siginfo_t siginfo; > int ret; > > - if (read(server->sigread, &sigc, 1) != 1) { > + if (read(server->sigread, &siginfo, sizeof(siginfo)) != sizeof(siginfo)) { Looks good, but that should be saferead instead of "read". Now that it's reading more than one byte, EINTR can make a difference.