Re: [lxc-devel] [PATCH] namespace: check stack allocation for clone succeeds

2011-04-11 Thread Michael Santos
On Mon, Apr 11, 2011 at 12:42:49PM -0500, Nathan Lynch wrote: > On Mon, 2011-04-11 at 12:42 -0400, Michael Santos wrote: > > --- > > src/lxc/namespace.c |7 ++- > > 1 files changed, 6 insertions(+), 1 deletions(-) > > > > diff --git a/src/lxc/namespace.

[lxc-devel] [PATCH] namespace: check stack allocation for clone succeeds

2011-04-11 Thread Michael Santos
--- src/lxc/namespace.c |7 ++- 1 files changed, 6 insertions(+), 1 deletions(-) diff --git a/src/lxc/namespace.c b/src/lxc/namespace.c index 8263fe7..5928eaa 100644 --- a/src/lxc/namespace.c +++ b/src/lxc/namespace.c @@ -67,9 +67,14 @@ pid_t lxc_clone(int (*fn)(void *), void *arg, int fl

[lxc-devel] [PATCH 5/5] confile: check allocation succeeds

2011-03-19 Thread Michael Santos
Signed-off-by: Michael Santos --- src/lxc/confile.c | 37 + 1 files changed, 29 insertions(+), 8 deletions(-) diff --git a/src/lxc/confile.c b/src/lxc/confile.c index 1d30cba..791f04f 100644 --- a/src/lxc/confile.c +++ b/src/lxc/confile.c @@ -195,7 +195,7

[lxc-devel] [PATCH 4/5] conf: increase buffer size to include spaces

2011-03-19 Thread Michael Santos
Signed-off-by: Michael Santos --- src/lxc/conf.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/lxc/conf.c b/src/lxc/conf.c index f73eaac..ae5b259 100644 --- a/src/lxc/conf.c +++ b/src/lxc/conf.c @@ -205,14 +205,18 @@ static int run_script(const char *name

[lxc-devel] [PATCH 3/5] arguments: check allocation succeeds

2011-03-19 Thread Michael Santos
Signed-off-by: Michael Santos --- src/lxc/arguments.c | 16 +--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/src/lxc/arguments.c b/src/lxc/arguments.c index 3c11661..51ebadb 100644 --- a/src/lxc/arguments.c +++ b/src/lxc/arguments.c @@ -96,6 +96,9 @@ static void

[lxc-devel] [PATCH 2/5] network: ensure interface name is NULL terminated

2011-03-19 Thread Michael Santos
IFNAMSIZ includes the trailing NULL. Signed-off-by: Michael Santos --- src/lxc/network.c | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/lxc/network.c b/src/lxc/network.c index 2ec8b71..420117a 100644 --- a/src/lxc/network.c +++ b/src/lxc

[lxc-devel] [PATCH 1/5] lxc_unshare: edit help output

2011-03-19 Thread Michael Santos
Signed-off-by: Michael Santos --- src/lxc/lxc_unshare.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lxc/lxc_unshare.c b/src/lxc/lxc_unshare.c index a82490e..eb93c89 100644 --- a/src/lxc/lxc_unshare.c +++ b/src/lxc/lxc_unshare.c @@ -46,10 +46,10 @@ void usage

[lxc-devel] [PATCH 2/2] Ensure monitored container name is null terminated

2011-03-16 Thread Michael Santos
regexec() expects a null terminated name. Signed-off-by: Michael Santos --- src/lxc/monitor.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/lxc/monitor.c b/src/lxc/monitor.c index 7ec1701..e51b59a 100644 --- a/src/lxc/monitor.c +++ b/src/lxc/monitor.c @@ -70,6

[lxc-devel] [PATCH 1/2] Exit if allocation fails

2011-03-16 Thread Michael Santos
Signed-off-by: Michael Santos --- src/lxc/lxc_monitor.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/src/lxc/lxc_monitor.c b/src/lxc/lxc_monitor.c index 314d4a5..1da0906 100644 --- a/src/lxc/lxc_monitor.c +++ b/src/lxc/lxc_monitor.c @@ -69,6 +69,10 @@ int main(int