#37 closed defect (fixed)
Unaligned access in saveload.cpp
| Reported by: | SF/vasyl | Owned by: | SF/strigeus |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCUMM |
| Keywords: | Cc: | ||
| Game: |
Description
Function saveLoadArrayOf contains four instances of
unaligned access (byte pointer "at" is forced to
uint16 and uint32 types). Two of these instances can
be replaced by READ_LE_UINT16/UINT32. Two others
either require more access functions to be added to
scummvm.h or just using byte access instead.
This bug was reported to me by Oleg Zakharov.
This is not the same as the other bug reported
earlier -- that one referred to unaligned access
functions not being implemented in header, now fixed.
Ticket imported from: #480571. Ticket imported from: bugs/37.
Change History (6)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
| Owner: | set to |
|---|
comment:3 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:5 by , 13 months ago
| Component: | → Engine: SCUMM |
|---|
Note:
See TracTickets
for help on using tickets.
saveLoadArrayOf should only read from data structures that
are already aligned properly, so there's no need to change
anything there.
The source of the problem might have been that some
structures were declared in the packed section of scumm.h
even though they didn't need to be packed.
I will commit the update in a little while.