There are potential bugs in the memory manager module that arise from not checking if the requested address range in read/write operations is valid. For example, a user might request to read 10 bytes from address 0xFFFA. This should result in an error since there aren't 10 bytes left in that address. But right now there is now validation in the read/write functions, so this should be fixed.
This also means that the read/write functions would return a Result now instead of just the data, so we need to make sure the interface is fixed for every usage of these functions.
There are potential bugs in the memory manager module that arise from not checking if the requested address range in read/write operations is valid. For example, a user might request to read 10 bytes from address 0xFFFA. This should result in an error since there aren't 10 bytes left in that address. But right now there is now validation in the read/write functions, so this should be fixed.
This also means that the read/write functions would return a Result now instead of just the data, so we need to make sure the interface is fixed for every usage of these functions.