source/drivers/MicroBitCompass.cpp@30:db87179335d5, 2016-07-13 (annotated)

Committer:

LancasterUniversity

Date:

Wed Jul 13 12:18:08 2016 +0100

Revision:

30:db87179335d5

Parent:

1:8aa5cdb4ab67

Child:

31:87789e55bac7

Synchronized with git rev 3b435c0d

Author: James Devine

microbit-dal: BUGFIX in MicroBitStorage



There was an off by one error when storing the key of the key value

pair, where the null terminator was dropped. This would mean that if

the returned key of the KeyValuePair were used, it would cause a number

of issues.



Another issue raised was the copying a random 48 bytes from memory

regardless of the position of memory in the stack. If the memory was

smaller than 48 bytes, and existed at the top of the stack, this could

have dire consequences. As a result, MicroBitStorage now accepts a size

parameter which informs the number of bytes to be copied into flash.



#130

Who changed what in which revision?