[lxc-devel] [lxc/lxc] 75d219: c/r: use /proc/self/tid/children instead of pidfil...

GitHub noreply at github.com
Fri Aug 26 16:48:39 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/lxc/lxc
  Commit: 75d219f0cc6be29d192677c5a8bd1bfd9528437b
      https://github.com/lxc/lxc/commit/75d219f0cc6be29d192677c5a8bd1bfd9528437b
  Author: Tycho Andersen <tycho.andersen at canonical.com>
  Date:   2016-08-26 (Fri, 26 Aug 2016)

  Changed paths:
    M src/lxc/criu.c

  Log Message:
  -----------
  c/r: use /proc/self/tid/children instead of pidfile

All we really needed a unique temp file for was passing the pid. Since CRIU
opened this with O_EXCL | O_CREAT, this was "safe" (users could still
overwrite it afterwards, but the monitor would immediately die since the
only valid number in there was the init process).

In any case, we can just read /proc/self/tid/children, which lists the
child process.

Closes #1150

Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>


  Commit: 69dffc98e32ce2f523073e108229d7d00e6493d9
      https://github.com/lxc/lxc/commit/69dffc98e32ce2f523073e108229d7d00e6493d9
  Author: Christian Brauner <cbrauner at suse.de>
  Date:   2016-08-26 (Fri, 26 Aug 2016)

  Changed paths:
    M src/lxc/criu.c

  Log Message:
  -----------
  Merge pull request #1152 from tych0/fix-migration-2.0.4

c/r: use /proc/self/tid/children instead of pidfile


Compare: https://github.com/lxc/lxc/compare/e854c5e31579...69dffc98e32c


More information about the lxc-devel mailing list