diff options
| author | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-06 03:36:34 +0000 |
|---|---|---|
| committer | aa@chromium.org <aa@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-02-06 03:36:34 +0000 |
| commit | 935bc2f807ac41e56e5e980387d202d504782bab (patch) | |
| tree | ecbe81de5ac4907ef125a18aa0b17481a5bac866 /base | |
| parent | c44783fbed512aaff7f73f1fdef563a3e6a6ec2d (diff) | |
| download | chromium_src-935bc2f807ac41e56e5e980387d202d504782bab.zip chromium_src-935bc2f807ac41e56e5e980387d202d504782bab.tar.gz chromium_src-935bc2f807ac41e56e5e980387d202d504782bab.tar.bz2 | |
Attempt to fix crash in SHFileOperation.
Latest logging shows we are definition crashing in this function. Some forums online have posts where people fixed it by explicitly setting this property to NULL, and the documentation agrees that it must be set this way.
BUG=108724
TBR=yoz@chromium.org
Review URL: https://chromiumcodereview.appspot.com/9323067
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@120534 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base')
| -rw-r--r-- | base/file_util_win.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/base/file_util_win.cc b/base/file_util_win.cc index 4723b96..173cc39 100644 --- a/base/file_util_win.cc +++ b/base/file_util_win.cc @@ -108,6 +108,7 @@ bool Delete(const FilePath& path, bool recursive) { SHFILEOPSTRUCT file_operation = {0}; file_operation.wFunc = FO_DELETE; file_operation.pFrom = double_terminated_path; + file_operation.pTo = NULL; // Must be NULL according to documentation. file_operation.fFlags = FOF_NOERRORUI | FOF_SILENT | FOF_NOCONFIRMATION; if (!recursive) file_operation.fFlags |= FOF_NORECURSION | FOF_FILESONLY; |