Le mercredi 24 avril 2013 à 08:55 -0500, Serge Hallyn a écrit :
> Quoting Frederic Crozat (fcro...@suse.com):
> > Le mardi 23 avril 2013 à 10:09 -0500, Serge Hallyn a écrit :
> > > Quoting Frederic Crozat (fcro...@suse.com):
> > 
> > > Thanks - clearly my patch needed some love :)  Just a few comments below.
> > 
> > hehe :)
> > > 
> > > > +
> > > > +       /* check if directory exists, and create it otherwise */
> > > > +       if (access(path, F_OK)) {
> > > > +               ret = mkdir(path, 0755);
> > > > +               if (ret == -1 && errno == EEXIST) {
> > > 
> > > Did you mean errno != EEXIST?
> > 
> > I blindly copied code from lxc_one_cgroup_create (now
> > create_lxcgroups :)
> 
> That has:
> 
>                         if (ret == -1 && errno != EEXIST) {

My copy didn't, because I didn't had
d7436fc1dfe0323fb5e6905457f342da5ed8ca97 backported..

I'll fix that..
-- 
Frederic Crozat <fcro...@suse.com>
SUSE


------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service 
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_apr
_______________________________________________
Lxc-users mailing list
Lxc-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-users

Reply via email to