sqlite_utils: don't dereference an empty vector.
user_settings: Add unit test intended to exercise the above path. In practice though these tests don't hit the bug, since EncryptData("") results in non-empty ciphertext. bonus: don't leak a sqlite handle in one of the other sync unit tests. TEST=none BUG=40217 Review URL: http://codereview.chromium.org/2801017 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50700 0039d316-1c4b-4281-b951-d872f2087c98
Showing
- chrome/browser/sync/util/data_encryption.cc 2 additions, 1 deletionchrome/browser/sync/util/data_encryption.cc
- chrome/browser/sync/util/user_settings_unittest.cc 33 additions, 4 deletionschrome/browser/sync/util/user_settings_unittest.cc
- chrome/common/sqlite_utils.cc 1 addition, 1 deletionchrome/common/sqlite_utils.cc
Loading
Please register or sign in to comment