The problem was identified in r273443 but the fix was applied to the wrong place. The sizes are in native order after swapping for the file to memory case, and before swapping in the memory to file case.
Details
Details
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
Comment Actions
The first part of the diff is wrong because ROUNDUP2 is called *after* the bytes are swapped. That's exactly the thing you can't do that was fixed by my commit. The second part is correct in that it fixes exactly this problem.
Comment Actions
ROUNDUP2 has to be called with the values in native endianness, which is after swapping in the file-to-memory case.