Safer MachineInfo Key Reads
ClosedPublic

Authored by rizzitello on Aug 4 2019, 4:14 PM.

Details

Diff Detail

Repository
R232 AtCore
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
rizzitello requested review of this revision.Aug 4 2019, 4:14 PM
rizzitello created this revision.
src/core/machineinfo.cpp
132

Are you sure that is <= with .side() ?

rizzitello updated this revision to Diff 63112.Aug 5 2019, 11:51 AM

update all calls use correct range

rizzitello marked an inline comment as done.Aug 5 2019, 11:51 AM
patrickelectric accepted this revision.Aug 5 2019, 11:52 AM
This revision is now accepted and ready to land.Aug 5 2019, 11:52 AM
laysrodrigues accepted this revision.Aug 18 2019, 8:03 PM
This revision was automatically updated to reflect the committed changes.