2005-07-21 Atsushi Enomoto <atsushi@ximian.com>
[mono.git] / mono / metadata / file-io.c
index cad73f10311f00a550060d64b803a54b892b13c5..8fdc927669fa7eb305a4d8ac3007002691b67f7e 100644 (file)
@@ -9,12 +9,17 @@
 
 #include <config.h>
 #include <glib.h>
+#include <string.h>
+#include <errno.h>
+#include <signal.h>
+#include <unistd.h>
 
 #include <mono/metadata/object.h>
 #include <mono/io-layer/io-layer.h>
 #include <mono/metadata/file-io.h>
 #include <mono/metadata/exception.h>
 #include <mono/metadata/appdomain.h>
+#include <mono/metadata/marshal.h>
 
 #undef DEBUG
 
@@ -154,10 +159,10 @@ static guint32 convert_seekorigin(MonoSeekOrigin origin)
 
 static gint64 convert_filetime (const FILETIME *filetime)
 {
-       gint64 *ticks;
-
-       ticks = (gint64 *)filetime;
-       return *ticks;
+       guint64 ticks = filetime->dwHighDateTime;
+       ticks <<= 32;
+       ticks += filetime->dwLowDateTime;
+       return (gint64)ticks;
 }
 
 static void convert_win32_file_attribute_data (const WIN32_FILE_ATTRIBUTE_DATA *data, const gunichar2 *name, MonoIOStat *stat)
@@ -177,321 +182,543 @@ static void convert_win32_file_attribute_data (const WIN32_FILE_ATTRIBUTE_DATA *
        stat->name = mono_string_new_utf16 (mono_domain_get (), name, len);
 }
 
-/* System.IO.MonoIO internal calls */
-
-gint32 
-ves_icall_System_IO_MonoIO_GetLastError ()
+/* Managed file attributes have nearly but not quite the same values
+ * as the w32 equivalents.
+ */
+static guint32 convert_attrs(MonoFileAttributes attrs)
 {
-       return GetLastError ();
+       if(attrs & FileAttributes_Encrypted) {
+               attrs |= FILE_ATTRIBUTE_ENCRYPTED;
+       }
+       
+       return(attrs);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_CreateDirectory (MonoString *path)
-{
-       gunichar2 *utf16_path;
-       gboolean result;
+/* System.IO.MonoIO internal calls */
 
-       utf16_path = mono_string_to_utf16 (path);
-       result = CreateDirectory (utf16_path, NULL);
-       g_free (utf16_path);
+MonoBoolean
+ves_icall_System_IO_MonoIO_CreateDirectory (MonoString *path, gint32 *error)
+{
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=CreateDirectory (mono_string_chars (path), NULL);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_RemoveDirectory (MonoString *path)
+MonoBoolean
+ves_icall_System_IO_MonoIO_RemoveDirectory (MonoString *path, gint32 *error)
 {
-       gunichar2 *utf16_path;
-       gboolean result;
-
-       utf16_path = mono_string_to_utf16 (path);
-       result = RemoveDirectory (utf16_path);
-       g_free (utf16_path);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=RemoveDirectory (mono_string_chars (path));
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 HANDLE 
-ves_icall_System_IO_MonoIO_FindFirstFile (MonoString *path, MonoIOStat *stat)
+ves_icall_System_IO_MonoIO_FindFirstFile (MonoString *path, MonoIOStat *stat,
+                                         gint32 *error)
 {
-       gunichar2 *utf16_path;
        WIN32_FIND_DATA data;
        HANDLE result;
+       gboolean r = TRUE;
 
-       utf16_path = mono_string_to_utf16 (path);
-       result = FindFirstFile (utf16_path, &data);
-       g_free (utf16_path);
+       MONO_ARCH_SAVE_REGS;
 
-       /* note: WIN32_FIND_DATA is an extension of WIN32_FILE_ATTRIBUTE_DATA */
+       *error=ERROR_SUCCESS;
 
-       if (result != INVALID_HANDLE_VALUE)
-               convert_win32_file_attribute_data ((const WIN32_FILE_ATTRIBUTE_DATA *)&data,
-                                                  &data.cFileName [0], stat);
+       result = FindFirstFile (mono_string_chars (path), &data);
+
+       /* note: WIN32_FIND_DATA is an extension of WIN32_FILE_ATTRIBUTE_DATA */
+       while (result != INVALID_HANDLE_VALUE && r) {
+               if ((data.cFileName [0] == '.' && data.cFileName [1] == 0) ||
+                   (data.cFileName [0] == '.' && data.cFileName [1] == '.' && data.cFileName [2] == 0)) {
+                       r = FindNextFile (result, &data);
+               } else {
+                       convert_win32_file_attribute_data ((const WIN32_FILE_ATTRIBUTE_DATA *)&data,
+                                                          &data.cFileName [0], stat);
+                       break;
+               } 
+       }
+       
+       if (result == INVALID_HANDLE_VALUE) {
+               *error=GetLastError ();
+       }
+       
+       if (r == FALSE) {
+               /* No more files were found, after we discarded . and .. */
+               FindClose(result);
+               result = INVALID_HANDLE_VALUE;
+               *error = ERROR_NO_MORE_FILES;
+       }
 
        return result;
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_FindNextFile (HANDLE find, MonoIOStat *stat)
+MonoBoolean
+ves_icall_System_IO_MonoIO_FindNextFile (HANDLE find, MonoIOStat *stat,
+                                        gint32 *error)
 {
        WIN32_FIND_DATA data;
        gboolean result;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        result = FindNextFile (find, &data);
-       if (result)
-               convert_win32_file_attribute_data ((const WIN32_FILE_ATTRIBUTE_DATA *)&data,
-                                                  &data.cFileName [0], stat);
+       while (result != FALSE) {
+               if ((data.cFileName [0] == '.' && data.cFileName [1] == 0) ||
+                   (data.cFileName [0] == '.' && data.cFileName [1] == '.' && data.cFileName [2] == 0)) {
+                       result = FindNextFile (find, &data);
+               } else {
+                       convert_win32_file_attribute_data ((const WIN32_FILE_ATTRIBUTE_DATA *)&data,
+                                                          &data.cFileName [0], stat);
+                       break;
+               } 
+       }
+       
+       if (result == FALSE) {
+               *error=GetLastError ();
+       }
 
        return result;
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_FindClose (HANDLE find)
+MonoBoolean
+ves_icall_System_IO_MonoIO_FindClose (HANDLE find, gint32 *error)
 {
-       return FindClose (find);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
+       ret=FindClose (find);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 MonoString *
-ves_icall_System_IO_MonoIO_GetCurrentDirectory ()
+ves_icall_System_IO_MonoIO_GetCurrentDirectory (gint32 *error)
 {
        MonoString *result;
        gunichar2 *buf;
        int len;
 
+       MONO_ARCH_SAVE_REGS;
+
        len = MAX_PATH + 1;
        buf = g_new (gunichar2, len);
        
+       *error=ERROR_SUCCESS;
        result = NULL;
+
        if (GetCurrentDirectory (len, buf) > 0) {
                len = 0;
                while (buf [len])
                        ++ len;
 
                result = mono_string_new_utf16 (mono_domain_get (), buf, len);
+       } else {
+               *error=GetLastError ();
        }
 
        g_free (buf);
        return result;
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_SetCurrentDirectory (MonoString *path)
+MonoBoolean
+ves_icall_System_IO_MonoIO_SetCurrentDirectory (MonoString *path,
+                                               gint32 *error)
 {
-       gunichar2 *utf16_path;
-       gboolean result;
+       gboolean ret;
        
-       utf16_path = mono_string_to_utf16 (path);
-       result = SetCurrentDirectory (utf16_path);
-       g_free (utf16_path);
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=SetCurrentDirectory (mono_string_chars (path));
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_MoveFile (MonoString *path, MonoString *dest)
+MonoBoolean
+ves_icall_System_IO_MonoIO_MoveFile (MonoString *path, MonoString *dest,
+                                    gint32 *error)
 {
-       gunichar2 *utf16_path, *utf16_dest;
-       gboolean result;
-
-       utf16_path = mono_string_to_utf16 (path);
-       utf16_dest = mono_string_to_utf16 (dest);
-       result = MoveFile (utf16_path, utf16_dest);
-       g_free (utf16_path);
-       g_free (utf16_dest);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=MoveFile (mono_string_chars (path), mono_string_chars (dest));
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_CopyFile (MonoString *path, MonoString *dest, gboolean overwrite)
+MonoBoolean
+ves_icall_System_IO_MonoIO_CopyFile (MonoString *path, MonoString *dest,
+                                    MonoBoolean overwrite, gint32 *error)
 {
-       gunichar2 *utf16_path, *utf16_dest;
-       gboolean result;
-
-       utf16_path = mono_string_to_utf16 (path);
-       utf16_dest = mono_string_to_utf16 (dest);
-       result = CopyFile (utf16_path, utf16_dest, !overwrite);
-       g_free (utf16_path);
-       g_free (utf16_dest);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=CopyFile (mono_string_chars (path), mono_string_chars (dest), !overwrite);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_DeleteFile (MonoString *path)
+MonoBoolean
+ves_icall_System_IO_MonoIO_DeleteFile (MonoString *path, gint32 *error)
 {
-       gunichar2 *utf16_path;
-       gboolean result;
-
-       utf16_path = mono_string_to_utf16 (path);
-       result = DeleteFile (utf16_path);
-       g_free (utf16_path);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=DeleteFile (mono_string_chars (path));
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 gint32 
-ves_icall_System_IO_MonoIO_GetFileAttributes (MonoString *path)
+ves_icall_System_IO_MonoIO_GetFileAttributes (MonoString *path, gint32 *error)
 {
-       gunichar2 *utf16_path;
-       gint32 result;
-
-       utf16_path = mono_string_to_utf16 (path);
-       result = GetFileAttributes (utf16_path);
-       g_free (utf16_path);
+       gint32 ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=GetFileAttributes (mono_string_chars (path));
+
+       /* 
+        * The definition of INVALID_FILE_ATTRIBUTES in the cygwin win32
+        * headers is wrong, hence this temporary workaround.
+        * See
+        * http://cygwin.com/ml/cygwin/2003-09/msg01771.html
+        */
+       if (ret==-1) {
+         /* if(ret==INVALID_FILE_ATTRIBUTES) { */
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean
-ves_icall_System_IO_MonoIO_SetFileAttributes (MonoString *path, gint32 attrs)
+MonoBoolean
+ves_icall_System_IO_MonoIO_SetFileAttributes (MonoString *path, gint32 attrs,
+                                             gint32 *error)
 {
-       gunichar2 *utf16_path;
-       gboolean result;
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
+       ret=SetFileAttributes (mono_string_chars (path),
+                              convert_attrs (attrs));
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
+}
 
-       utf16_path = mono_string_to_utf16 (path);
-       result = SetFileAttributes (utf16_path, attrs);
-       g_free (utf16_path);
+gint32
+ves_icall_System_IO_MonoIO_GetFileType (HANDLE handle, gint32 *error)
+{
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=GetFileType (handle);
+       if(ret==FILE_TYPE_UNKNOWN) {
+               /* Not necessarily an error, but the caller will have
+                * to decide based on the error value.
+                */
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_GetFileStat (MonoString *path, MonoIOStat *stat)
+MonoBoolean
+ves_icall_System_IO_MonoIO_GetFileStat (MonoString *path, MonoIOStat *stat,
+                                       gint32 *error)
 {
-       gunichar2 *utf16_path;
        gboolean result;
        WIN32_FILE_ATTRIBUTE_DATA data;
 
-       utf16_path = mono_string_to_utf16 (path);
-       result = GetFileAttributesEx (utf16_path, GetFileExInfoStandard, &data);
-       g_free (utf16_path);
+       MONO_ARCH_SAVE_REGS;
 
-       if (result)
-               convert_win32_file_attribute_data (&data, utf16_path, stat);
+       *error=ERROR_SUCCESS;
+       
+       result = GetFileAttributesEx (mono_string_chars (path), GetFileExInfoStandard, &data);
+
+       if (result) {
+               convert_win32_file_attribute_data (&data,
+                                                  mono_string_chars (path),
+                                                  stat);
+       } else {
+               *error=GetLastError ();
+       }
 
        return result;
 }
 
 HANDLE 
-ves_icall_System_IO_MonoIO_Open (MonoString *filename, gint32 mode, gint32 access_mode, gint32 share)
+ves_icall_System_IO_MonoIO_Open (MonoString *filename, gint32 mode,
+                                gint32 access_mode, gint32 share,
+                                MonoBoolean async, gint32 *error)
 {
-       gunichar2 *utf16_filename;
-       HANDLE result;
-
-       utf16_filename = mono_string_to_utf16 (filename);
-       result = CreateFile (utf16_filename, convert_access (access_mode), convert_share (share),
-                            NULL, convert_mode (mode), FILE_ATTRIBUTE_NORMAL, NULL);
-       g_free (utf16_filename);
+       HANDLE ret;
+       
+       MONO_ARCH_SAVE_REGS;
 
-       return result;
+       *error=ERROR_SUCCESS;
+       
+       ret=CreateFile (mono_string_chars (filename),
+                       convert_access (access_mode), convert_share (share),
+                       NULL, convert_mode (mode),
+                       FILE_ATTRIBUTE_NORMAL | ((async) ? FILE_FLAG_OVERLAPPED : 0),
+                       NULL);
+       if(ret==INVALID_HANDLE_VALUE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_Close (HANDLE handle)
+MonoBoolean
+ves_icall_System_IO_MonoIO_Close (HANDLE handle, gint32 *error)
 {
-       return CloseHandle (handle);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
+       ret=CloseHandle (handle);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 gint32 
-ves_icall_System_IO_MonoIO_Read (HANDLE handle, MonoArray *dest, gint32 dest_offset, gint32 count)
+ves_icall_System_IO_MonoIO_Read (HANDLE handle, MonoArray *dest,
+                                gint32 dest_offset, gint32 count,
+                                gint32 *error)
 {
        guchar *buffer;
        gboolean result;
        guint32 n;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        if (dest_offset + count > mono_array_length (dest))
                return 0;
 
        buffer = mono_array_addr (dest, guchar, dest_offset);
        result = ReadFile (handle, buffer, count, &n, NULL);
 
-       if (!result)
+       if (!result) {
+               *error=GetLastError ();
                return -1;
+       }
 
        return (gint32)n;
 }
 
 gint32 
-ves_icall_System_IO_MonoIO_Write (HANDLE handle, MonoArray *src, gint32 src_offset, gint32 count)
+ves_icall_System_IO_MonoIO_Write (HANDLE handle, MonoArray *src,
+                                 gint32 src_offset, gint32 count,
+                                 gint32 *error)
 {
        guchar *buffer;
        gboolean result;
        guint32 n;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        if (src_offset + count > mono_array_length (src))
                return 0;
        
        buffer = mono_array_addr (src, guchar, src_offset);
        result = WriteFile (handle, buffer, count, &n, NULL);
 
-       if (!result)
+       if (!result) {
+               *error=GetLastError ();
                return -1;
+       }
 
        return (gint32)n;
 }
 
 gint64 
-ves_icall_System_IO_MonoIO_Seek (HANDLE handle, gint64 offset, gint32 origin)
+ves_icall_System_IO_MonoIO_Seek (HANDLE handle, gint64 offset, gint32 origin,
+                                gint32 *error)
 {
        guint32 offset_hi;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        offset_hi = offset >> 32;
        offset = SetFilePointer (handle, offset & 0xFFFFFFFF, &offset_hi,
                                 convert_seekorigin (origin));
 
+       if(offset==INVALID_SET_FILE_POINTER) {
+               *error=GetLastError ();
+       }
+       
        return offset | ((gint64)offset_hi << 32);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_Flush (HANDLE handle)
+MonoBoolean
+ves_icall_System_IO_MonoIO_Flush (HANDLE handle, gint32 *error)
 {
-       return FlushFileBuffers (handle);
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
+       ret=FlushFileBuffers (handle);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 gint64 
-ves_icall_System_IO_MonoIO_GetLength (HANDLE handle)
+ves_icall_System_IO_MonoIO_GetLength (HANDLE handle, gint32 *error)
 {
        gint64 length;
        guint32 length_hi;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        length = GetFileSize (handle, &length_hi);
+       if(length==INVALID_FILE_SIZE) {
+               *error=GetLastError ();
+       }
+       
        return length | ((gint64)length_hi << 32);
 }
 
-gboolean 
-ves_icall_System_IO_MonoIO_SetLength (HANDLE handle, gint64 length)
+MonoBoolean
+ves_icall_System_IO_MonoIO_SetLength (HANDLE handle, gint64 length,
+                                     gint32 *error)
 {
-       gint64 offset;
+       gint64 offset, offset_set;
        gint32 offset_hi;
        gint32 length_hi;
        gboolean result;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        /* save file pointer */
 
        offset_hi = 0;
        offset = SetFilePointer (handle, 0, &offset_hi, FILE_CURRENT);
+       if(offset==INVALID_SET_FILE_POINTER) {
+               *error=GetLastError ();
+               return(FALSE);
+       }
 
        /* extend or truncate */
 
        length_hi = length >> 32;
-       SetFilePointer (handle, length & 0xFFFFFFFF, &length_hi, FILE_BEGIN);
+       offset_set=SetFilePointer (handle, length & 0xFFFFFFFF, &length_hi,
+                                  FILE_BEGIN);
+       if(offset_set==INVALID_SET_FILE_POINTER) {
+               *error=GetLastError ();
+               return(FALSE);
+       }
+
        result = SetEndOfFile (handle);
+       if(result==FALSE) {
+               *error=GetLastError ();
+               return(FALSE);
+       }
 
        /* restore file pointer */
 
-       SetFilePointer (handle, offset & 0xFFFFFFFF, &offset_hi, FILE_BEGIN);
+       offset_set=SetFilePointer (handle, offset & 0xFFFFFFFF, &offset_hi,
+                                  FILE_BEGIN);
+       if(offset_set==INVALID_SET_FILE_POINTER) {
+               *error=GetLastError ();
+               return(FALSE);
+       }
 
        return result;
 }
 
-gboolean
-ves_icall_System_IO_MonoIO_SetFileTime (HANDLE handle, gint64 creation_time, gint64 last_access_time, gint64 last_write_time)
+MonoBoolean
+ves_icall_System_IO_MonoIO_SetFileTime (HANDLE handle, gint64 creation_time,
+                                       gint64 last_access_time,
+                                       gint64 last_write_time, gint32 *error)
 {
+       gboolean ret;
        const FILETIME *creation_filetime;
        const FILETIME *last_access_filetime;
        const FILETIME *last_write_filetime;
 
+       MONO_ARCH_SAVE_REGS;
+
+       *error=ERROR_SUCCESS;
+       
        if (creation_time < 0)
                creation_filetime = NULL;
        else
@@ -507,34 +734,67 @@ ves_icall_System_IO_MonoIO_SetFileTime (HANDLE handle, gint64 creation_time, gin
        else
                last_write_filetime = (FILETIME *)&last_write_time;
 
-       return SetFileTime (handle, creation_filetime, last_access_filetime, last_write_filetime);
+       ret=SetFileTime (handle, creation_filetime, last_access_filetime, last_write_filetime);
+       if(ret==FALSE) {
+               *error=GetLastError ();
+       }
+       
+       return(ret);
 }
 
 HANDLE 
 ves_icall_System_IO_MonoIO_get_ConsoleOutput ()
 {
+       MONO_ARCH_SAVE_REGS;
+
        return GetStdHandle (STD_OUTPUT_HANDLE);
 }
 
 HANDLE 
 ves_icall_System_IO_MonoIO_get_ConsoleInput ()
 {
+       MONO_ARCH_SAVE_REGS;
+
        return GetStdHandle (STD_INPUT_HANDLE);
 }
 
 HANDLE 
 ves_icall_System_IO_MonoIO_get_ConsoleError ()
 {
+       MONO_ARCH_SAVE_REGS;
+
        return GetStdHandle (STD_ERROR_HANDLE);
 }
 
+MonoBoolean
+ves_icall_System_IO_MonoIO_CreatePipe (HANDLE *read_handle,
+                                      HANDLE *write_handle)
+{
+       SECURITY_ATTRIBUTES attr;
+       gboolean ret;
+       
+       MONO_ARCH_SAVE_REGS;
+
+       attr.nLength=sizeof(SECURITY_ATTRIBUTES);
+       attr.bInheritHandle=TRUE;
+       attr.lpSecurityDescriptor=NULL;
+       
+       ret=CreatePipe (read_handle, write_handle, &attr, 0);
+       if(ret==FALSE) {
+               /* FIXME: throw an exception? */
+               return(FALSE);
+       }
+       
+       return(TRUE);
+}
+
 gunichar2 
 ves_icall_System_IO_MonoIO_get_VolumeSeparatorChar ()
 {
 #if defined (PLATFORM_WIN32)
-       return (gunichar2) 0x003a;      /* colon */
+       return (gunichar2) ':'; /* colon */
 #else
-       return (gunichar2) 0x002f;      /* forward slash */
+       return (gunichar2) '/'; /* forward slash */
 #endif
 }
 
@@ -542,9 +802,9 @@ gunichar2
 ves_icall_System_IO_MonoIO_get_DirectorySeparatorChar ()
 {
 #if defined (PLATFORM_WIN32)
-       return (gunichar2) 0x005c;      /* backslash */
+       return (gunichar2) '\\';        /* backslash */
 #else
-       return (gunichar2) 0x002f;      /* forward slash */
+       return (gunichar2) '/'; /* forward slash */
 #endif
 }
 
@@ -552,9 +812,9 @@ gunichar2
 ves_icall_System_IO_MonoIO_get_AltDirectorySeparatorChar ()
 {
 #if defined (PLATFORM_WIN32)
-       return (gunichar2) 0x002f;      /* forward slash */
+       return (gunichar2) '/'; /* forward slash */
 #else
-       return (gunichar2) 0x005c;      /* backslash */
+       return (gunichar2) '/'; /* slash, same as DirectorySeparatorChar */
 #endif
 }
 
@@ -562,18 +822,29 @@ gunichar2
 ves_icall_System_IO_MonoIO_get_PathSeparator ()
 {
 #if defined (PLATFORM_WIN32)
-       return (gunichar2) 0x003b;      /* semicolon */
+       return (gunichar2) ';'; /* semicolon */
 #else
-       return (gunichar2) 0x003a;      /* colon */
+       return (gunichar2) ':'; /* colon */
 #endif
 }
 
-static gunichar2 invalid_path_chars [] = {
+static const gunichar2
+invalid_path_chars [] = {
 #if defined (PLATFORM_WIN32)
-       0x0022,                         /* double quote */
+       0x0022,                         /* double quote, which seems allowed in MS.NET but should be rejected */
        0x003c,                         /* less than */
        0x003e,                         /* greater than */
        0x007c,                         /* pipe */
+       0x0008,
+       0x0010,
+       0x0011,
+       0x0012,
+       0x0014,
+       0x0015,
+       0x0016,
+       0x0017,
+       0x0018,
+       0x0019,
 #endif
        0x0000                          /* null */
 };
@@ -585,13 +856,74 @@ ves_icall_System_IO_MonoIO_get_InvalidPathChars ()
        MonoDomain *domain;
        int i, n;
 
-       domain = mono_domain_get ();
-       chars = mono_array_new (domain, mono_defaults.char_class, 5);
+       MONO_ARCH_SAVE_REGS;
 
+       domain = mono_domain_get ();
        n = sizeof (invalid_path_chars) / sizeof (gunichar2);
+       chars = mono_array_new (domain, mono_defaults.char_class, n);
 
        for (i = 0; i < n; ++ i)
                mono_array_set (chars, gunichar2, i, invalid_path_chars [i]);
        
        return chars;
 }
+
+gint32
+ves_icall_System_IO_MonoIO_GetTempPath (MonoString **mono_name)
+{
+       gunichar2 *name;
+       int ret;
+
+       name=g_new0 (gunichar2, 256);
+       
+       ret=GetTempPath (256, name);
+       if(ret>255) {
+               /* Buffer was too short. Try again... */
+               g_free (name);
+               name=g_new0 (gunichar2, ret+2); /* include the terminator */
+               ret=GetTempPath (ret, name);
+       }
+       
+       if(ret>0) {
+#ifdef DEBUG
+               g_message (G_GNUC_PRETTY_FUNCTION
+                          ": Temp path is [%s] (len %d)", name, ret);
+#endif
+
+               *mono_name=mono_string_new_utf16 (mono_domain_get (), name,
+                                                 ret);
+       }
+
+       g_free (name);
+       
+       return(ret);
+}
+
+void ves_icall_System_IO_MonoIO_Lock (HANDLE handle, gint64 position,
+                                     gint64 length, gint32 *error)
+{
+       gboolean ret;
+       
+       *error=ERROR_SUCCESS;
+       
+       ret=LockFile (handle, position & 0xFFFFFFFF, position >> 32,
+                     length & 0xFFFFFFFF, length >> 32);
+       if (ret == FALSE) {
+               *error = GetLastError ();
+       }
+}
+
+void ves_icall_System_IO_MonoIO_Unlock (HANDLE handle, gint64 position,
+                                       gint64 length, gint32 *error)
+{
+       gboolean ret;
+       
+       *error=ERROR_SUCCESS;
+       
+       ret=UnlockFile (handle, position & 0xFFFFFFFF, position >> 32,
+                       length & 0xFFFFFFFF, length >> 32);
+       if (ret == FALSE) {
+               *error = GetLastError ();
+       }
+}
+