[lxc-devel] [lxc/lxc] cd43d2: Fix bug in preserve_ns

GitHub noreply at github.com
Sat Jan 11 04:25:33 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/lxc/lxc
  Commit: cd43d2d1b989f8ef292151d795c51e96df1cf903
      https://github.com/lxc/lxc/commit/cd43d2d1b989f8ef292151d795c51e96df1cf903
  Author: Serge Hallyn <serge.hallyn at ubuntu.com>
  Date:   2014-01-10 (Fri, 10 Jan 2014)

  Changed paths:
    M src/lxc/start.c

  Log Message:
  -----------
  Fix bug in preserve_ns

If /proc/self/ns does not exist, then preserve_ns was failing to
initialize the saved_ns[i] to -1.  This caused attach_ns() to try
and attach, and of course fail.

Initialize the saved ns values before returning an error.

The return values of preserve_ns and attach_ns were also being
ignored.  Honor them.

Signed-off-by: Serge Hallyn <serge.hallyn at ubuntu.com>
Acked-by: Stéphane Graber <stgraber at ubuntu.com>


  Commit: 2e59ba029ebd5ffbcf0442cc16d9d4375a060e6f
      https://github.com/lxc/lxc/commit/2e59ba029ebd5ffbcf0442cc16d9d4375a060e6f
  Author: Stéphane Graber <stgraber at ubuntu.com>
  Date:   2014-01-10 (Fri, 10 Jan 2014)

  Changed paths:
    M src/lxc/bdev.c
    M src/lxc/lxc.functions.in
    M src/lxc/lxc_config.c
    M src/lxc/utils.c

  Log Message:
  -----------
  One more rename of the system options

This should be the last change of name for those options. This change
basically move them all to lxc.bdev.<backend>.<option>.

Signed-off-by: Stéphane Graber <stgraber at ubuntu.com>
Acked-by: Serge Hallyn <serge.hallyn at ubuntu.com>


Compare: https://github.com/lxc/lxc/compare/dad87e3bd14e...2e59ba029ebd


More information about the lxc-devel mailing list