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

Re: [Libguestfs] [hivex][PATCH 2/8] generator: Add new return type to ABI: RLenValue



On Dec 5, 2011, at 00:45 , Richard W.M. Jones wrote:

> On Sun, Dec 04, 2011 at 04:45:27PM -0800, Alex Nelson wrote:
>> 
>> On Oct 20, 2011, at 04:46 , Richard W.M. Jones wrote:
>> 
>>> On Thu, Oct 20, 2011 at 12:37:54PM +0100, Richard W.M. Jones wrote:
>>>> of the tests could have picked up this type of bug.  I think the tests
>>>> (all of them) need to be changed to be more thorough.
>>> 
>>> And while you're doing that, is there a way to avoid the
>>> hard-coded '297680'?
>>> 
>>> I suggest two possibilities: create another test hive (derived from
>>> 'large') which will be specific to this test and we will never change
>>> in future.  Since this won't change, we can go ahead and use
>>> hard-coded offsets.
>> 
>> I see two options for the test hive.
>> 
>> The first, which I would prefer, is to use an openly available hive
>> from a research disk image.  For example, digitalcorpora.org has
>> several Windows computer disk images immediately available for
>> analysis.  Any hive from there could be included as a sample in the
>> images/ directory.  Unfortunately, I don't know how well this would
>> work with the GPL.
> 
> I checked this out, and it's not possible.  Our test hive 'minimal'
> was specially constructed so it does not contain anything
> copyrightable.  'large' was constructed from 'minimal' by a mechanical
> process.
Thought so.
> 
> Nevertheless Digital Corpora looks interesting and useful.  I need to
> see if we can add tests (including for libguestfs) that would use that
> for people who had downloaded the corpus themselves.
> 
>> Alternatively, there is the attached patch, which adds a
>> not-too-creatively named hive generated from ''minimal'.  (I didn't
>> see a benefit to deriving it from 'large,' but it's easy enough to
>> modify that.)
> 
> OK.  The patch is the sort of thing I was suggesting.  This is going
> to be combined with the revised patch series?
Yes.  Thanks!

--Alex


> 
> [...]
> 
> Rich.
> 
> -- 
> Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
> virt-p2v converts physical machines to virtual machines.  Boot with a
> live CD or over the network (PXE) and turn machines into Xen guests.
> http://et.redhat.com/~rjones/virt-p2v



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