diff options
| author | georgey@chromium.org <georgey@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-08 19:52:20 +0000 |
|---|---|---|
| committer | georgey@chromium.org <georgey@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-09-08 19:52:20 +0000 |
| commit | e14de0e1876642944ee924e25438a7b24dbb3857 (patch) | |
| tree | 88c7e848a62e679b9c59b8667ab759eb3d6acfec /chrome/browser/autofill | |
| parent | ae75a18bf4bc3f566e7315327375794f1a6c220c (diff) | |
| download | chromium_src-e14de0e1876642944ee924e25438a7b24dbb3857.zip chromium_src-e14de0e1876642944ee924e25438a7b24dbb3857.tar.gz chromium_src-e14de0e1876642944ee924e25438a7b24dbb3857.tar.bz2 | |
Fix for 54439 Browser crash @ WideToUTF8(std::basic_string<wchar_t,std::char_traits<wchar_t>,std::allocator<wchar_t> > const &)
Upload data was called after the InfoBar changed the current page thus resulting in the NULL pointer dereference.
BUG=54439
TEST=Should not crash
Review URL: http://codereview.chromium.org/3291019
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@58864 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/autofill')
| -rw-r--r-- | chrome/browser/autofill/autofill_manager.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/autofill/autofill_manager.cc b/chrome/browser/autofill/autofill_manager.cc index 10d2385..e66e9ef 100644 --- a/chrome/browser/autofill/autofill_manager.cc +++ b/chrome/browser/autofill/autofill_manager.cc @@ -446,7 +446,7 @@ void AutoFillManager::HandleSubmit() { } void AutoFillManager::UploadFormData() { - if (!disable_download_manager_requests_) { + if (!disable_download_manager_requests_ && upload_form_structure_.get()) { bool was_autofilled = false; // Check if the form among last 3 forms that were auto-filled. // Clear older signatures. |