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

[Dan Kegel]

On Fri, Feb 10, 2012 at 2:00 PM, Charles Davis  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