On Wed, Feb 27, 2008 at 05:27:33PM +0100, Jim Meyering wrote:
> Whoops. You'll want to close that file descriptor.
> E.g., put this right before "argv = NULL;":
>
> if (close (ret))
> return -1;
>
> It'd be nice to declare a new variable, say fd, for that bit.
Oh yes, that would be
"Richard W.M. Jones" <[EMAIL PROTECTED]> wrote:
> Here's the final version of the "3 arrays" version of the patch, with
> the memory corruptor fixed.
...
> Index: src/qemu_driver.c
> ===
> RCS file: /data/cvs/libvirt/src/qemu_driver.c,
Here's the final version of the "3 arrays" version of the patch, with
the memory corruptor fixed.
Rich.
--
Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones
virt-p2v converts physical machines to virtual machines. Boot with a
live CD or over the network (PXE) and turn
On Mon, Feb 25, 2008 at 05:28:05PM +, Richard W.M. Jones wrote:
> On Mon, Feb 25, 2008 at 04:49:00PM +, Richard W.M. Jones wrote:
> [...]
>
> Thanks to Jim for pointing me in the right direction here. If you try
> this patch you'll need to insert memset() calls at the two places
> indicat
On Tue, Feb 26, 2008 at 12:01:04PM +, Richard W.M. Jones wrote:
> On Tue, Feb 26, 2008 at 11:43:59AM +0100, Jim Meyering wrote:
> > "Richard W.M. Jones" <[EMAIL PROTECTED]> wrote:
> > > This patch is an evolution of the previous patch for implementing DHCP
> > ...
> > > +if (maxmappings ==
On Tue, Feb 26, 2008 at 11:43:59AM +0100, Jim Meyering wrote:
> "Richard W.M. Jones" <[EMAIL PROTECTED]> wrote:
> > This patch is an evolution of the previous patch for implementing DHCP
> ...
> > +if (maxmappings == 0) return 0;
> ...
> > +if (col == 0) hwaddr = token;
> ...
>
"Richard W.M. Jones" <[EMAIL PROTECTED]> wrote:
> This patch is an evolution of the previous patch for implementing DHCP
...
> +if (maxmappings == 0) return 0;
...
> +if (col == 0) hwaddr = token;
...
> +if (!(*hwaddrs)[lines]) goto mem_error;
Hi Rich,
Is there
On Mon, Feb 25, 2008 at 04:49:00PM +, Richard W.M. Jones wrote:
[...]
Thanks to Jim for pointing me in the right direction here. If you try
this patch you'll need to insert memset() calls at the two places
indicated below. That fixes all the problems I can see and it appears
to all work fine