On Mon, Sep 08, 2008 at 11:51:28AM +0200, Jim Meyering wrote:
> "Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
> > On Fri, Sep 05, 2008 at 05:40:16PM +0200, Jim Meyering wrote:
> ...
> >> All looks correct.
> >> However, I'm a little nervous about hard-coding those '[3]'s.
> >> What if someone ins
On Mon, Sep 08, 2008 at 04:06:26PM +0400, Evgeniy Sokolov wrote:
> >On Fri, Sep 05, 2008 at 05:40:16PM +0200, Jim Meyering wrote:
> >>"Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
> >>>Jim pointed out some places where the openvz driver could deference some
> >>>NULs, so this patch fixes them
> >
On Fri, Sep 05, 2008 at 05:40:16PM +0200, Jim Meyering wrote:
"Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
Jim pointed out some places where the openvz driver could deference some
NULs, so this patch fixes them
...
Index: src/openvz_driver.c
=
"Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
> On Fri, Sep 05, 2008 at 05:40:16PM +0200, Jim Meyering wrote:
...
>> All looks correct.
>> However, I'm a little nervous about hard-coding those '[3]'s.
>> What if someone inserts a new --foo option somewhere before
>> the NULL place-holder, or othe
On Fri, Sep 05, 2008 at 05:40:16PM +0200, Jim Meyering wrote:
> "Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
> > Jim pointed out some places where the openvz driver could deference some
> > NULs, so this patch fixes them
> ...
> > Index: src/openvz_driver.c
> > ==
"Daniel P. Berrange" <[EMAIL PROTECTED]> wrote:
> Jim pointed out some places where the openvz driver could deference some
> NULs, so this patch fixes them
...
> Index: src/openvz_driver.c
> ===
> RCS file: /data/cvs/libvirt/src/openvz
Jim pointed out some places where the openvz driver could deference some
NULs, so this patch fixes them
Daniel
Index: src/openvz_driver.c
===
RCS file: /data/cvs/libvirt/src/openvz_driver.c,v
retrieving revision 1.46
diff -u -p -r1.