X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=Kernel%2Fvfs%2Fopen.c;h=9069a73d2cd6d3089e1479124aa99edd691a2009;hb=12e1d80b700a6a47e8506dc23bb6e1d64a172bd4;hp=eebaa82d000d314f6461acf768de3b89ceeca085;hpb=9b4ce124f950e1bebbf4e011804e0b8ed8472f6a;p=tpg%2Facess2.git

diff --git a/Kernel/vfs/open.c b/Kernel/vfs/open.c
index eebaa82d..9069a73d 100644
--- a/Kernel/vfs/open.c
+++ b/Kernel/vfs/open.c
@@ -2,26 +2,16 @@
  * AcessMicro VFS
  * - Open, Close and ChDir
  */
+#define DEBUG	0
 #include <common.h>
 #include "vfs.h"
 #include "vfs_int.h"
 #include "vfs_ext.h"
 
-#define DEBUG	0
-
-#if DEBUG
-#else
-# undef ENTER
-# undef LOG
-# undef LEAVE
-# define ENTER(...)
-# define LOG(...)
-# define LEAVE(...)
-#endif
-
 // === CONSTANTS ===
 #define	OPEN_MOUNT_ROOT	1
 #define MAX_KERNEL_FILES	128
+#define MAX_PATH_SLASHES	256
 
 // === IMPORTS ===
 extern tVFS_Node	gVFS_MemRoot;
@@ -42,7 +32,7 @@ char *VFS_GetAbsPath(char *Path)
 	 int	pathLen = strlen(Path);
 	 int	read, write;
 	 int	pos, slashNum=0, baseLen;
-	Uint	slashOffsets[256];
+	Uint	slashOffsets[MAX_PATH_SLASHES];
 	char	*cwd = CFGPTR(CFG_VFS_CWD);
 	 int	cwdLen;
 	
@@ -72,20 +62,14 @@ char *VFS_GetAbsPath(char *Path)
 	} else {
 		cwdLen = strlen(cwd);
 		// Prepend the current directory
-		ret = malloc(cwdLen+pathLen+1);
+		ret = malloc(cwdLen+pathLen+2);
 		strcpy(ret, cwd);
-		strcpy(&ret[cwdLen], Path);
-	
-		// Pre-fill the slash positions
-		pos = 0;
-		while( (pos = strpos( &ret[pos+1], '/' )) != -1 )
-			slashOffsets[slashNum++] = pos;
-			
-		baseLen = cwdLen;
+		ret[cwdLen] = '/';
+		strcpy(&ret[cwdLen+1], Path);
 	}
 	
 	// Remove . and ..
-	read = write = baseLen;	// Cwd has already been parsed
+	read = write = 1;	// Cwd has already been parsed
 	for(; read < baseLen+pathLen; read = pos+1)
 	{
 		pos = strpos( &ret[read], '/' );
@@ -93,18 +77,21 @@ char *VFS_GetAbsPath(char *Path)
 		if(pos == -1)	pos = baseLen+pathLen;
 		else	pos += read;	// Else, Adjust to absolute
 		
+		//Log("pos-read = %i", pos-read);
+		
 		// Check Length
 		if(pos - read <= 2)
 		{
+			//Log("&ret[read] = '%s'", &ret[read]);
 			// Current Dir "."
 			if(strncmp(&ret[read], ".", pos-read) == 0)	continue;
 			// Parent ".."
 			if(strncmp(&ret[read], "..", pos-read) == 0)
 			{
 				// If there is no higher, silently ignore
-				if(!slashNum)	continue;
+				if(slashNum < 2)	continue;
 				// Reverse write pointer
-				write = slashOffsets[ slashNum-- ];
+				write = slashOffsets[ --slashNum - 1 ];
 				continue;
 			}
 		}
@@ -112,10 +99,11 @@ char *VFS_GetAbsPath(char *Path)
 		
 		// Only copy if the positions differ
 		if(read != write) {
+			Log("write = %i, read = %i", write, read);
 			memcpy( &ret[write], &ret[read], pos-read+1 );
 		}
 		write = pos+1;
-		if(slashNum < 256)
+		if(slashNum < MAX_PATH_SLASHES)
 			slashOffsets[ slashNum++ ] = pos;
 		else {
 			LOG("Path '%s' has too many elements", Path);
@@ -127,6 +115,7 @@ char *VFS_GetAbsPath(char *Path)
 	
 	// `ret` should now be the absolute path
 	LEAVE('s', ret);
+	Log("VFS_GetAbsPath: RETURN '%s'", ret);
 	return ret;
 }
 
@@ -538,8 +527,9 @@ int VFS_ChDir(char *New)
 	// Close file
 	VFS_Close(fd);
 	
-	// Free working directory and set new one
-	free( CFGPTR(CFG_VFS_CWD) );
+	// Free old working directory
+	if( CFGPTR(CFG_VFS_CWD) )	free( CFGPTR(CFG_VFS_CWD) );
+	// Set new
 	CFGPTR(CFG_VFS_CWD) = buf;
 	
 	Log("Updated CWD to '%s'", buf);