ConflictResolver::ProcessSimpleConflict() does not handle the case of corrupted data |
|
Issue descriptionIn [1], [2], and [3], it is assumed that cryptographer->DecryptToString() won't fail. We should investigate if this hide some issue or not. However, this is a directory code that should be obsolete after full migration to USS. [1] https://cs.chromium.org/chromium/src/components/sync/engine_impl/conflict_resolver.cc?l=117&rcl=20b501c316f87203b461a6769f3d113711ecaa1e [2] https://cs.chromium.org/chromium/src/components/sync/engine_impl/conflict_resolver.cc?l=126&rcl=20b501c316f87203b461a6769f3d113711ecaa1e [3] https://cs.chromium.org/chromium/src/components/sync/engine_impl/conflict_resolver.cc?l=143&rcl=20b501c316f87203b461a6769f3d113711ecaa1e |
|
►
Sign in to add a comment |
|
Comment 1 by mamir@chromium.org
, Nov 26