msi: Fix some copy/paste bugs in the implementation of condition operators.

EA Durbin ead1234 at hotmail.com
Mon Jun 5 10:36:04 CDT 2006


My analysis has not changed it still remains the same to this day.

Case, america's army installer is working backward, not going through the 
media in the correct order.

Upon troubleshooting this I found that this is due to the query returning 
the wrong results for LastSequence. I don't have an analysis for what is 
causing this yet. But this is what is causing the bug and I've never swayed 
on that.

I did in fact write a test, which does prove this. Hans has a copy of it, 
and i think i sent it to wine-devel too.

As for files.c, the query in ready_media_for_file() should be passing an 
unsinged integer as LastSequence must be greater than zero.

http://search.msdn.microsoft.com/search/Redirect.aspx?title=Media+Table+%5bWindows+Installer%5d&url=http://msdn.microsoft.com/library/en-us/msi/setup/media_table.asp

 >less than or equal to the value in the LastSequence column, and greater 
than the LastSequence value of the previous disk (or greater than 0, for the 
first entry in the Media table).

>From: Mike McCormack <mike at codeweavers.com>
>To: EA Durbin <ead1234 at hotmail.com>
>CC: dank at kegel.com,  wine-devel at winehq.org,  hans at it.vu.nl
>Subject: Re: msi: Fix some copy/paste bugs in the implementation of 
>condition operators.
>Date: Tue, 06 Jun 2006 00:24:00 +0900
>
>
>EA Durbin wrote:
>
>>obviously my patch wasn't "wrong" as you so put it, as this is the same 
>>thing as the one I submitted. I just phrased the definition of it wrong 
>>assuming the conditional statements were used by SQL.
>
>Your patch caused the regression tests to fail, and Hans' did not.
>
>err:msi:msi_get_dialog_record query failed for dialog L"boo"
>fixme:msi:ACTION_PerformUIAction unhandled msi action L"boo"
>package.c:618: Test failed: wrong return val
>make[1]: *** [package.ok] Error 1
>
>You seem to have a different theory of what's causing the failure every 
>day.
>
>I'm somewhat disappointed that you refuse to write a regression test to 
>show what you've found.  It would give me confidence in what your analysis 
>of the problem, assist me in fixing the problem, and prevent it from 
>happening again.
>
>Mike
>





More information about the wine-devel mailing list