Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(323)

Issue 2465083002: Add a dummy entry with libsecret when initializing OSCrypt. (Closed)

Created:
4 years, 1 month ago by Tom (Use chromium acct)
Modified:
4 years, 1 month ago
Reviewers:
Tom Anderson
CC:
chromium-reviews, gcasto+watchlist_chromium.org, vabr+watchlistpasswordmanager_chromium.org, cfroussios
Target Ref:
refs/pending/branch-heads/2883
Project:
chromium
Visibility:
Public.

Description

Add a dummy entry with libsecret when initializing OSCrypt. Under certain conditions, gnome libsecret methods that perform a lookup in keyring may ignore a locked keyring and return empty results, instead of unlocking said keyring. Store operations are unaffected by this quirk (or bug, presumably). With this change, we store a dummy entry during the initialisation of OSCrypt. This ensures that the default keyring gets unlocked and is available for future reads. This solution is also applied to Password Manager, before performing calls to secret_service_search_sync. This solution should be temporary, until the underlying issue gets fixed or there is an official workaround. BUG=660005, 631171 Review-Url: https://codereview.chromium.org/2461453002 Cr-Commit-Position: refs/heads/master@{#428378} NOTRY=true NOPRESUBMIT=true

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+68 lines, -3 lines) Patch
M chrome/browser/password_manager/native_backend_libsecret.h View 1 chunk +4 lines, -0 lines 0 comments Download
M chrome/browser/password_manager/native_backend_libsecret.cc View 3 chunks +12 lines, -2 lines 0 comments Download
M chrome/browser/password_manager/native_backend_libsecret_unittest.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M components/os_crypt/key_storage_libsecret.cc View 1 chunk +4 lines, -1 line 0 comments Download
M components/os_crypt/key_storage_libsecret_unittest.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M components/os_crypt/libsecret_util_linux.h View 1 chunk +4 lines, -0 lines 0 comments Download
M components/os_crypt/libsecret_util_linux.cc View 2 chunks +36 lines, -0 lines 0 comments Download

Depends on Patchset:

Messages

Total messages: 10 (6 generated)
Tom Anderson
lgtm from my @chromium account
4 years, 1 month ago (2016-11-01 00:02:39 UTC) #3
commit-bot: I haz the power
This CL has an open dependency (Issue 2463263002 Patch 1). Please resolve the dependency and ...
4 years, 1 month ago (2016-11-01 00:03:38 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2465083002/1
4 years, 1 month ago (2016-11-01 00:09:58 UTC) #8
commit-bot: I haz the power
4 years, 1 month ago (2016-11-01 00:14:02 UTC) #10
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698