diff options
author | Jeff Sharkey <jsharkey@android.com> | 2014-01-17 16:52:37 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2014-01-17 16:52:37 +0000 |
commit | c5d43a1fc595ce650a9a20d2811ad47429ebd7da (patch) | |
tree | f3131043f03af7ca6f1e54950f6f9f4e234ebc52 | |
parent | 1e950b1a9345889a1197aeb4cb19e45f8b19a136 (diff) | |
parent | 12f656962b644268a02cecd370b1ea35fa22cf4e (diff) | |
download | android_packages_providers_DownloadProvider-c5d43a1fc595ce650a9a20d2811ad47429ebd7da.tar.gz android_packages_providers_DownloadProvider-c5d43a1fc595ce650a9a20d2811ad47429ebd7da.tar.bz2 android_packages_providers_DownloadProvider-c5d43a1fc595ce650a9a20d2811ad47429ebd7da.zip |
Merge "Creates parent directories before creates file"
-rw-r--r-- | src/com/android/providers/downloads/Helpers.java | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/src/com/android/providers/downloads/Helpers.java b/src/com/android/providers/downloads/Helpers.java index 013faf27..3562dac7 100644 --- a/src/com/android/providers/downloads/Helpers.java +++ b/src/com/android/providers/downloads/Helpers.java @@ -141,7 +141,15 @@ public class Helpers { // Claim this filename inside lock to prevent other threads from // clobbering us. We're not paranoid enough to use O_EXCL. try { - new File(path).createNewFile(); + File file = new File(path); + File parent = file.getParentFile(); + + // Make sure the parent directories exists before generates new file + if (parent != null && !parent.exists()) { + parent.mkdirs(); + } + + file.createNewFile(); } catch (IOException e) { throw new StopRequestException(Downloads.Impl.STATUS_FILE_ERROR, "Failed to create target file " + path, e); |