[PATCH 3/7] ntdll: fix endianness of three fields in DVD_LAYER_DESCRIPTOR (resend)

Dan Kegel dank at kegel.com
Fri Feb 10 17:24:44 CST 2012


On Fri, Feb 10, 2012 at 2:00 PM, Charles Davis <cdavis at mymail.mines.edu> wrote:
>> -            p->StartingDataSector = l->start_sector;
>> -            p->EndDataSector = l->end_sector;
>> -            p->EndLayerZeroSector = l->end_sector_l0;
>> +            p->StartingDataSector = GET_BE_DWORD(l->start_sector);
>> +            p->EndDataSector = GET_BE_DWORD(l->end_sector);
>> +            p->EndLayerZeroSector = GET_BE_DWORD(l->end_sector_l0);
> I don't know about this. To make your original patch work right on Mac, one of the things I had to do was get rid of the OSReadBigInt32() calls that swapped the endianness of the sector fields from big to host.

This was needed for Dragon Age Origins able to detect its disc, see
http://bugs.winehq.org/show_bug.cgi?id=29667
That's our source of truth, I guess.  Do you have a copy of Dragon Age
handy to test with?
- Dan



More information about the wine-devel mailing list