From 831ef743f2e9ef0ea4618a81e1a270588657dd63 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=B6nke=20Ludwig?= <sludwig@rejectedsoftware.com>
Date: Wed, 2 Nov 2016 20:54:43 +0100
Subject: [PATCH] Safe-ify the file module.

---
 source/vibe/core/file.d | 31 ++++++++++++++++++-------------
 1 file changed, 18 insertions(+), 13 deletions(-)

diff --git a/source/vibe/core/file.d b/source/vibe/core/file.d
index ca97bfa..c8431e7 100644
--- a/source/vibe/core/file.d
+++ b/source/vibe/core/file.d
@@ -25,11 +25,12 @@ import std.file;
 import std.path;
 import std.string;
 
-
 version(Posix){
 	private extern(C) int mkstemps(char* templ, int suffixlen);
 }
 
+@safe:
+
 
 /**
 	Opens a file stream with the specified mode.
@@ -154,7 +155,7 @@ FileStream createTempFile(string suffix = null)
 		templ[pattern.length .. $-1] = (suffix)[];
 		templ[$-1] = '\0';
 		assert(suffix.length <= int.max);
-		auto fd = mkstemps(templ.ptr, cast(int)suffix.length);
+		auto fd = () @trusted { return mkstemps(templ.ptr, cast(int)suffix.length); } ();
 		enforce(fd >= 0, "Failed to create temporary file.");
 		assert(false);
 		//return eventDriver.adoptFile(fd, Path(templ[0 .. $-1].idup), FileMode.createTrunc);
@@ -257,7 +258,7 @@ bool existsFile(string path) nothrow
 	Throws: A `FileException` is thrown if the file does not exist.
 */
 FileInfo getFileInfo(Path path)
-{
+@trusted {
 	auto ent = DirEntry(path.toNativeString());
 	return makeFileInfo(ent);
 }
@@ -272,7 +273,7 @@ FileInfo getFileInfo(string path)
 */
 void createDirectory(Path path)
 {
-	mkdir(path.toNativeString());
+	() @trusted { mkdir(path.toNativeString()); } ();
 }
 /// ditto
 void createDirectory(string path)
@@ -284,7 +285,7 @@ void createDirectory(string path)
 	Enumerates all files in the specified directory.
 */
 void listDirectory(Path path, scope bool delegate(FileInfo info) del)
-{
+@trusted {
 	foreach( DirEntry ent; dirEntries(path.toNativeString(), SpanMode.shallow) )
 		if( !del(makeFileInfo(ent)) )
 			break;
@@ -331,7 +332,7 @@ DirectoryWatcher watchDirectory(string path, bool recursive = true)
 */
 Path getWorkingDirectory()
 {
-	return Path(std.file.getcwd());
+	return Path(() @trusted { return std.file.getcwd(); } ());
 }
 
 
@@ -375,6 +376,8 @@ enum FileMode {
 	Accesses the contents of a file as a stream.
 */
 struct FileStream {
+	@safe:
+
 	private {
 		FileFD m_fd;
 		Path m_path;
@@ -393,13 +396,13 @@ struct FileStream {
 
 	this(this)
 	{
-		if (m_fd != FileFD.init)
+		if (m_fd != FileFD.invalid)
 			eventDriver.files.addRef(m_fd);
 	}
 	
 	~this()
 	{
-		if (m_fd != FileFD.init)
+		if (m_fd != FileFD.invalid)
 			eventDriver.files.releaseRef(m_fd);
 	}
 
@@ -409,10 +412,12 @@ struct FileStream {
 	@property Path path() const { return m_path; }
 
 	/// Determines if the file stream is still open
-	@property bool isOpen() const { return m_fd != FileFD.init; }
+	@property bool isOpen() const { return m_fd != FileFD.invalid; }
 	@property ulong size() const nothrow { return m_size; }
-	@property bool readable() const { return m_mode != FileMode.append; }
-	@property bool writable() const { return m_mode != FileMode.read; }
+	@property bool readable() const nothrow { return m_mode != FileMode.append; }
+	@property bool writable() const nothrow { return m_mode != FileMode.read; }
+
+	bool opCast(T)() if (is (T == bool)) { return m_fd != FileFD.invalid; }
 
 	void takeOwnershipOfFD()
 	{
@@ -424,7 +429,7 @@ struct FileStream {
 		m_ptr = offset;
 	}
 
-	ulong tell() { return m_ptr; }
+	ulong tell() nothrow { return m_ptr; }
 
 	/// Closes the file handle.
 	void close()
@@ -624,7 +629,7 @@ struct DirectoryChange {
 
 
 private FileInfo makeFileInfo(DirEntry ent)
-{
+@trusted {
 	FileInfo ret;
 	ret.name = baseName(ent.name);
 	if( ret.name.length == 0 ) ret.name = ent.name;