[libvirt] [PATCH v2 10/18] LXC from native: convert lxc.id_map into <idmap>
Daniel P. Berrange
berrange at redhat.com
Tue Feb 4 12:35:22 UTC 2014
On Thu, Jan 30, 2014 at 10:55:58AM +0100, Cédric Bosdonnat wrote:
> ---
> src/lxc/lxc_native.c | 43 ++++++++++++++++++++++++++
> tests/lxcconf2xmldata/lxcconf2xml-idmap.config | 5 +++
> tests/lxcconf2xmldata/lxcconf2xml-idmap.xml | 28 +++++++++++++++++
> tests/lxcconf2xmltest.c | 1 +
> 4 files changed, 77 insertions(+)
> create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-idmap.config
> create mode 100644 tests/lxcconf2xmldata/lxcconf2xml-idmap.xml
> +
> + if (type == 'u') {
> + if (VIR_EXPAND_N(def->idmap.uidmap, def->idmap.nuidmap, 1) < 0)
> + return -1;
> + idmap = &def->idmap.uidmap[def->idmap.nuidmap - 1];
> + } else if (type == 'g') {
> + if (VIR_EXPAND_N(def->idmap.gidmap, def->idmap.ngidmap, 1) < 0)
> + return -1;
> + idmap = &def->idmap.gidmap[def->idmap.ngidmap - 1];
> + } else
> + return -1;
Style nit-pick - when one part of an if/else has {}, then all parts
should have {}.
ACK with style nitpick.
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list