diff options
| author | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2022-08-01 18:09:35 +0200 |
|---|---|---|
| committer | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2022-09-04 18:42:11 +0200 |
| commit | 575534e7cca310c6d6195ab16d482bf9dfba755e (patch) | |
| tree | fe105929f6a248c7df979d686b6902c8bacf84b7 /lib/csapi/device_management.cpp | |
| parent | 8e58d28ca0517aeeb43c99bd97ec9ba5ada11c95 (diff) | |
| download | libquotient-575534e7cca310c6d6195ab16d482bf9dfba755e.tar.gz libquotient-575534e7cca310c6d6195ab16d482bf9dfba755e.zip | |
Disallow direct events construction from JSON
Direct construction (using makeEvent() or explicitly constructing
an event) from JSON may create an event that has a type conflicting
with that stored in JSON. There's no such problem with loadEvent(),
even though it's considerably slower. Driven by the fact that almost
nowhere in the code direct construction is used on checked JSON
(one test is the only valid case), this commit moves all JSON-loading
constructors to the protected section, thereby disabling usage of
makeEvent() in JSON-loading capacity, and switches such cases across
the library to loadEvent().
Diffstat (limited to 'lib/csapi/device_management.cpp')
0 files changed, 0 insertions, 0 deletions