[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [ruby-libvirt] Don't free more entries than we retrieved



On Tue, Jan 7, 2014 at 4:04 PM, Guido Günther <agx sigxcpu org> wrote:
> The check for < 0 might be superfluos but freeing num entries instead of
> only r ones looks wrong to me since r is the number of allocated names,
> not num. I'm happy to send a patch dropping the < 0 part.

Oh, bah humbug.  Yeah, you are definitely right about that.

Please do send a patch with just the num -> r bit.  That is definitely a bug.

Thanks,
Chris


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]