#37 closed defect (fixed)
Unaligned access in saveload.cpp
| Reported by: | SF/vasyl | Owned by: | SF/strigeus |
|---|---|---|---|
| Priority: | normal | Component: | Engine: SCUMM |
| Version: | 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 , 23 years ago
comment:2 by , 23 years ago
| Owner: | set to |
|---|
comment:3 by , 23 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:5 by , 6 years 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.