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

Side by Side Diff: chrome/browser/password_manager/native_backend_libsecret.cc

Issue 1929573002: Changed location of LibsecretLoader (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Moved omitted implementation and also moved LibsecretAttributesBuilder Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/password_manager/native_backend_libsecret.h" 5 #include "chrome/browser/password_manager/native_backend_libsecret.h"
6 6
7 #include <dlfcn.h>
8 #include <stddef.h> 7 #include <stddef.h>
9 #include <stdint.h> 8 #include <stdint.h>
10 9
11 #include <limits> 10 #include <limits>
12 #include <list> 11 #include <list>
13 #include <memory> 12 #include <memory>
14 #include <utility> 13 #include <utility>
15 #include <vector> 14 #include <vector>
16 15
16 #include <libsecret/secret.h>
17
17 #include "base/logging.h" 18 #include "base/logging.h"
18 #include "base/metrics/histogram.h" 19 #include "base/metrics/histogram.h"
19 #include "base/strings/string_number_conversions.h" 20 #include "base/strings/string_number_conversions.h"
20 #include "base/strings/stringprintf.h" 21 #include "base/strings/stringprintf.h"
21 #include "base/strings/utf_string_conversions.h" 22 #include "base/strings/utf_string_conversions.h"
22 #include "components/password_manager/core/browser/password_manager_metrics_util .h" 23 #include "components/password_manager/core/browser/password_manager_metrics_util .h"
23 #include "components/password_manager/core/browser/password_manager_util.h" 24 #include "components/password_manager/core/browser/password_manager_util.h"
24 #include "url/origin.h" 25 #include "url/origin.h"
25 26
26 using autofill::PasswordForm; 27 using autofill::PasswordForm;
27 using base::UTF8ToUTF16; 28 using base::UTF8ToUTF16;
28 using base::UTF16ToUTF8; 29 using base::UTF16ToUTF8;
29 30
30 namespace { 31 namespace {
31 const char kEmptyString[] = ""; 32 const char kEmptyString[] = "";
32 const int kMaxPossibleTimeTValue = std::numeric_limits<int>::max(); 33 const int kMaxPossibleTimeTValue = std::numeric_limits<int>::max();
33 } // namespace 34 } // namespace
34 35
35 typeof(&::secret_password_store_sync)
36 LibsecretLoader::secret_password_store_sync;
37 typeof(&::secret_service_search_sync)
38 LibsecretLoader::secret_service_search_sync;
39 typeof(&::secret_password_clear_sync)
40 LibsecretLoader::secret_password_clear_sync;
41 typeof(&::secret_item_get_secret) LibsecretLoader::secret_item_get_secret;
42 typeof(&::secret_value_get_text) LibsecretLoader::secret_value_get_text;
43 typeof(&::secret_item_get_attributes)
44 LibsecretLoader::secret_item_get_attributes;
45 typeof(&::secret_item_load_secret_sync)
46 LibsecretLoader::secret_item_load_secret_sync;
47 typeof(&::secret_value_unref) LibsecretLoader::secret_value_unref;
48
49 bool LibsecretLoader::libsecret_loaded = false;
50
51 const LibsecretLoader::FunctionInfo LibsecretLoader::functions[] = {
52 {"secret_password_store_sync",
53 reinterpret_cast<void**>(&secret_password_store_sync)},
54 {"secret_service_search_sync",
55 reinterpret_cast<void**>(&secret_service_search_sync)},
56 {"secret_password_clear_sync",
57 reinterpret_cast<void**>(&secret_password_clear_sync)},
58 {"secret_item_get_secret",
59 reinterpret_cast<void**>(&secret_item_get_secret)},
60 {"secret_value_get_text", reinterpret_cast<void**>(&secret_value_get_text)},
61 {"secret_item_get_attributes",
62 reinterpret_cast<void**>(&secret_item_get_attributes)},
63 {"secret_item_load_secret_sync",
64 reinterpret_cast<void**>(&secret_item_load_secret_sync)},
65 {"secret_value_unref", reinterpret_cast<void**>(&secret_value_unref)},
66 {nullptr, nullptr}};
67
68 bool LibsecretLoader::LoadLibsecret() {
69 if (libsecret_loaded)
70 return true;
71
72 void* handle = dlopen("libsecret-1.so.0", RTLD_NOW | RTLD_GLOBAL);
73 if (!handle) {
74 // We wanted to use libsecret, but we couldn't load it. Warn, because
75 // either the user asked for this, or we autodetected it incorrectly. (Or
76 // the system has broken libraries, which is also good to warn about.)
77 LOG(WARNING) << "Could not load libsecret-1.so.0: " << dlerror();
78 return false;
79 }
80
81 for (size_t i = 0; functions[i].name; ++i) {
82 dlerror();
83 *functions[i].pointer = dlsym(handle, functions[i].name);
84 const char* error = dlerror();
85 if (error) {
86 VLOG(1) << "Unable to load symbol " << functions[i].name << ": " << error;
87 dlclose(handle);
88 return false;
89 }
90 }
91
92 libsecret_loaded = true;
93 // We leak the library handle. That's OK: this function is called only once.
94 return true;
95 }
96
97 namespace { 36 namespace {
98 37
99 const char kLibsecretAppString[] = "chrome"; 38 const char kLibsecretAppString[] = "chrome";
100 39
101 // Schema is analagous to the fields in PasswordForm. 40 // Schema is analagous to the fields in PasswordForm.
102 const SecretSchema kLibsecretSchema = { 41 const SecretSchema kLibsecretSchema = {
103 "chrome_libsecret_password_schema", 42 "chrome_libsecret_password_schema",
104 // We have to use SECRET_SCHEMA_DONT_MATCH_NAME in order to get old 43 // We have to use SECRET_SCHEMA_DONT_MATCH_NAME in order to get old
105 // passwords stored with gnome_keyring. 44 // passwords stored with gnome_keyring.
106 SECRET_SCHEMA_DONT_MATCH_NAME, 45 SECRET_SCHEMA_DONT_MATCH_NAME,
(...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 bool success = DeserializeFormDataFromBase64String(encoded_form_data, 147 bool success = DeserializeFormDataFromBase64String(encoded_form_data,
209 &form->form_data); 148 &form->form_data);
210 password_manager::metrics_util::FormDeserializationStatus status = 149 password_manager::metrics_util::FormDeserializationStatus status =
211 success ? password_manager::metrics_util::GNOME_SUCCESS 150 success ? password_manager::metrics_util::GNOME_SUCCESS
212 : password_manager::metrics_util::GNOME_FAILURE; 151 : password_manager::metrics_util::GNOME_FAILURE;
213 LogFormDataDeserializationStatus(status); 152 LogFormDataDeserializationStatus(status);
214 } 153 }
215 return form; 154 return form;
216 } 155 }
217 156
218 class LibsecretAttributesBuilder {
219 public:
220 LibsecretAttributesBuilder();
221 ~LibsecretAttributesBuilder();
222 void Append(const std::string& name, const std::string& value);
223 void Append(const std::string& name, int64_t value);
224 // GHashTable, its keys and values returned from Get() are destroyed in
225 // |LibsecretAttributesBuilder| desctructor.
226 GHashTable* Get() { return attrs_; }
227
228 private:
229 // |name_values_| is a storage for strings referenced in |attrs_|.
230 std::list<std::string> name_values_;
231 GHashTable* attrs_;
232 };
233
234 LibsecretAttributesBuilder::LibsecretAttributesBuilder() {
235 attrs_ = g_hash_table_new_full(g_str_hash, g_str_equal,
236 nullptr, // no deleter for keys
237 nullptr); // no deleter for values
238 }
239
240 LibsecretAttributesBuilder::~LibsecretAttributesBuilder() {
241 g_hash_table_destroy(attrs_);
242 }
243
244 void LibsecretAttributesBuilder::Append(const std::string& name,
245 const std::string& value) {
246 name_values_.push_back(name);
247 gpointer name_str =
248 static_cast<gpointer>(const_cast<char*>(name_values_.back().c_str()));
249 name_values_.push_back(value);
250 gpointer value_str =
251 static_cast<gpointer>(const_cast<char*>(name_values_.back().c_str()));
252 g_hash_table_insert(attrs_, name_str, value_str);
253 }
254
255 void LibsecretAttributesBuilder::Append(const std::string& name,
256 int64_t value) {
257 Append(name, base::Int64ToString(value));
258 }
259
260 // Generates a profile-specific app string based on profile_id_. 157 // Generates a profile-specific app string based on profile_id_.
261 std::string GetProfileSpecificAppString(LocalProfileId id) { 158 std::string GetProfileSpecificAppString(LocalProfileId id) {
262 // Originally, the application string was always just "chrome" and used only 159 // Originally, the application string was always just "chrome" and used only
263 // so that we had *something* to search for since GNOME Keyring won't search 160 // so that we had *something* to search for since GNOME Keyring won't search
264 // for nothing. Now we use it to distinguish passwords for different profiles. 161 // for nothing. Now we use it to distinguish passwords for different profiles.
265 return base::StringPrintf("%s-%d", kLibsecretAppString, id); 162 return base::StringPrintf("%s-%d", kLibsecretAppString, id);
266 } 163 }
267 164
268 } // namespace 165 } // namespace
269 166
270 bool LibsecretLoader::LibsecretIsAvailable() {
271 if (!libsecret_loaded)
272 return false;
273 // A dummy query is made to check for availability, because libsecret doesn't
274 // have a dedicated availability function. For performance reasons, the query
275 // is meant to return an empty result.
276 LibsecretAttributesBuilder attrs;
277 attrs.Append("application", "chrome-string_to_get_empty_result");
278
279 GError* error = nullptr;
280 GList* found = secret_service_search_sync(nullptr, // default secret service
281 &kLibsecretSchema, attrs.Get(),
282 SECRET_SEARCH_ALL,
283 nullptr, // no cancellable ojbect
284 &error);
285 bool success = (error == nullptr);
286 if (error)
287 g_error_free(error);
288 if (found)
289 g_list_free(found);
290
291 return success;
292 }
293
294 NativeBackendLibsecret::NativeBackendLibsecret(LocalProfileId id) 167 NativeBackendLibsecret::NativeBackendLibsecret(LocalProfileId id)
295 : app_string_(GetProfileSpecificAppString(id)) { 168 : app_string_(GetProfileSpecificAppString(id)) {
296 } 169 }
297 170
298 NativeBackendLibsecret::~NativeBackendLibsecret() { 171 NativeBackendLibsecret::~NativeBackendLibsecret() {
299 } 172 }
300 173
301 bool NativeBackendLibsecret::Init() { 174 bool NativeBackendLibsecret::Init() {
302 return LoadLibsecret() && LibsecretIsAvailable(); 175 return LoadLibsecret() && LibsecretIsAvailable();
303 } 176 }
(...skipping 379 matching lines...) Expand 10 before | Expand all | Expand 10 after
683 if (lookup_form) { 556 if (lookup_form) {
684 UMA_HISTOGRAM_ENUMERATION("PasswordManager.PslDomainMatchTriggering", 557 UMA_HISTOGRAM_ENUMERATION("PasswordManager.PslDomainMatchTriggering",
685 allow_psl_match 558 allow_psl_match
686 ? psl_domain_match_metric 559 ? psl_domain_match_metric
687 : password_manager::PSL_DOMAIN_MATCH_NOT_USED, 560 : password_manager::PSL_DOMAIN_MATCH_NOT_USED,
688 password_manager::PSL_DOMAIN_MATCH_COUNT); 561 password_manager::PSL_DOMAIN_MATCH_COUNT);
689 } 562 }
690 g_list_free(found); 563 g_list_free(found);
691 return forms; 564 return forms;
692 } 565 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698