From: Rafael Espindola <rafael.espindola@gmail.com>
Date: Fri, 5 Jul 2013 15:05:40 +0000 (+0000)
Subject: Fix PR16547.
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=38d7c34f75eed2089802e209fb29bc2dfbf1b7a7;p=clang

Fix PR16547.

We should not be asking unique_file to prepend the system temporary directory
when creating the html report. Unfortunately I don't think we can test this
with the current infrastructure since unique_file ignores MakeAbsolute if the
directory is already absolute and the paths provided by lit are.

I will take a quick look at making this api a bit less error prone.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@185707 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp b/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
index 120a63e7a2..e32b85b9b3 100644
--- a/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
+++ b/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
@@ -247,8 +247,9 @@ void HTMLDiagnostics::ReportDiag(const PathDiagnostic& D,
   SmallString<128> Model, ResultPath;
   llvm::sys::path::append(Model, Directory, "report-%%%%%%.html");
 
-  if (llvm::error_code EC =
-          llvm::sys::fs::unique_file(Model.str(), FD, ResultPath)) {
+  if (llvm::error_code EC = llvm::sys::fs::unique_file(
+          Model.str(), FD, ResultPath, false,
+          llvm::sys::fs::all_read | llvm::sys::fs::all_write)) {
     llvm::errs() << "warning: could not create file in '" << Directory
                  << "': " << EC.message() << '\n';
     return;