X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=Kernel%2Fvfs%2Fopen.c;h=600cedc60e2364df622e8c334802e0cb1a6b283a;hb=54bf151b1a05b74debdb5f3baec02c18406b74d1;hp=552276d9f37caaf546856d742ae547cce604da89;hpb=7c2a4b6359a81dac11d89eda748d1cc0640f6d33;p=tpg%2Facess2.git diff --git a/Kernel/vfs/open.c b/Kernel/vfs/open.c index 552276d9..600cedc6 100644 --- a/Kernel/vfs/open.c +++ b/Kernel/vfs/open.c @@ -1,33 +1,28 @@ /* - * AcessMicro VFS + * Acess2 VFS * - Open, Close and ChDir */ #define DEBUG 0 #include -#include #include "vfs.h" #include "vfs_int.h" #include "vfs_ext.h" // === CONSTANTS === #define OPEN_MOUNT_ROOT 1 -#define MAX_KERNEL_FILES 128 #define MAX_PATH_SLASHES 256 // === IMPORTS === extern tVFS_Node gVFS_MemRoot; extern tVFS_Mount *gVFS_RootMount; - -// === GLOBALS === -tVFS_Handle *gaUserHandles = (void*)MM_PPD_VFS; -tVFS_Handle *gaKernelHandles = (void*)MM_KERNEL_VFS; +extern int VFS_AllocHandle(int bIsUser, tVFS_Node *Node, int Mode); // === CODE === /** - * \fn char *VFS_GetAbsPath(char *Path) + * \fn char *VFS_GetAbsPath(const char *Path) * \brief Create an absolute path from a relative one */ -char *VFS_GetAbsPath(char *Path) +char *VFS_GetAbsPath(const char *Path) { char *ret; int pathLen = strlen(Path); @@ -35,9 +30,9 @@ char *VFS_GetAbsPath(char *Path) char *tmpStr; int iPos = 0; int iPos2 = 0; - char *chroot = CFGPTR(CFG_VFS_CHROOT); + const char *chroot = CFGPTR(CFG_VFS_CHROOT); int chrootLen; - char *cwd = CFGPTR(CFG_VFS_CWD); + const char *cwd = CFGPTR(CFG_VFS_CWD); int cwdLen; ENTER("sPath", Path); @@ -46,7 +41,7 @@ char *VFS_GetAbsPath(char *Path) if(Path[0] == '$') { ret = malloc(strlen(Path)+1); if(!ret) { - Warning("VFS_GetAbsPath - malloc() returned NULL"); + Log_Warning("VFS", "VFS_GetAbsPath: malloc() returned NULL"); return NULL; } strcpy(ret, Path); @@ -66,7 +61,7 @@ char *VFS_GetAbsPath(char *Path) if(Path[0] == '/') { ret = malloc(pathLen + 1); if(!ret) { - Warning("VFS_GetAbsPath - malloc() returned NULL"); + Log_Warning("VFS", "VFS_GetAbsPath: malloc() returned NULL"); return NULL; } strcpy(ret, Path); @@ -83,7 +78,7 @@ char *VFS_GetAbsPath(char *Path) strcpy(ret, cwd); ret[cwdLen] = '/'; strcpy(&ret[cwdLen+1], Path); - //Log("ret = '%s'\n", ret); + //Log("ret = '%s'", ret); } // Parse Path @@ -167,10 +162,10 @@ char *VFS_GetAbsPath(char *Path) } /** - * \fn char *VFS_ParsePath(char *Path, char **TruePath) + * \fn char *VFS_ParsePath(const char *Path, char **TruePath) * \brief Parses a path, resolving sysmlinks and applying permissions */ -tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) +tVFS_Node *VFS_ParsePath(const char *Path, char **TruePath) { tVFS_Mount *mnt; tVFS_Mount *longestMount = gVFS_RootMount; // Root is first @@ -191,8 +186,8 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) LEAVE('p', curNode); return curNode; } - // For root we always fast return + // For root we always fast return if(Path[0] == '/' && Path[1] == '\0') { if(TruePath) { *TruePath = malloc( gVFS_RootMount->MountPointLen+1 ); @@ -202,7 +197,7 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) return gVFS_RootMount->RootNode; } - // Check if there is anything mounted + // Check if there is an`ything mounted if(!gVFS_Mounts) { Warning("WTF! There's nothing mounted?"); return NULL; @@ -237,12 +232,6 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) longestMount = mnt; } - // Sanity Check - /*if(!longestMount) { - Log("VFS_ParsePath - ERROR: No Root Node\n"); - return NULL; - }*/ - // Save to shorter variable mnt = longestMount; @@ -260,13 +249,15 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) curNode->ReferenceCount ++; // Parse Path ofs = mnt->MountPointLen+1; - for(; (nextSlash = strpos(&Path[ofs], '/')) != -1; Path[nextSlash]='/',ofs = nextSlash + 1) + for(; (nextSlash = strpos(&Path[ofs], '/')) != -1; ofs += nextSlash + 1) { - nextSlash += ofs; - Path[nextSlash] = '\0'; - - // Check for empty string - if( Path[ofs] == '\0' ) continue; + char pathEle[nextSlash+1]; + + // Empty String + if(nextSlash == 0) continue; + + memcpy(pathEle, &Path[ofs], nextSlash); + pathEle[nextSlash] = 0; // Check permissions on root of filesystem if( !VFS_CheckACL(curNode, VFS_PERM_EXECUTE) ) { @@ -288,27 +279,28 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) free(*TruePath); *TruePath = NULL; } - Path[nextSlash] = '/'; //Log("FindDir fail on '%s'", Path); LEAVE('n'); return NULL; } - LOG("FindDir(%p, '%s')", curNode, &Path[ofs]); + LOG("FindDir{=%p}(%p, '%s')", curNode->FindDir, curNode, pathEle); // Get Child Node - tmpNode = curNode->FindDir(curNode, &Path[ofs]); + tmpNode = curNode->FindDir(curNode, pathEle); LOG("tmpNode = %p", tmpNode); - if(curNode->Close) curNode->Close(curNode); + if(curNode->Close) { + //LOG2("curNode->Close = %p", curNode->Close); + curNode->Close(curNode); + } curNode = tmpNode; // Error Check if(!curNode) { - LOG("Node '%s' not found in dir '%s'", &Path[ofs], Path); + LOG("Node '%s' not found in dir '%s'", pathEle, Path); if(TruePath) { free(*TruePath); *TruePath = NULL; } - //Log("Child fail on '%s' ('%s)", Path, &Path[ofs]); - Path[nextSlash] = '/'; + //Log("Child fail on '%s' ('%s)", Path, pathEle); LEAVE('n'); return NULL; } @@ -319,11 +311,19 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) free(*TruePath); *TruePath = NULL; } - tmp = malloc( curNode->Size + 1 ); if(!curNode->Read) { Warning("VFS_ParsePath - Read of node %p is NULL (%s)", curNode, Path); if(curNode->Close) curNode->Close(curNode); + // No need to free *TruePath, see above + LEAVE('n'); + return NULL; + } + + tmp = malloc( curNode->Size + 1 ); + if(!tmp) { + Log_Warning("VFS", "VFS_ParsePath - Malloc failure"); + // No need to free *TruePath, see above LEAVE('n'); return NULL; } @@ -332,21 +332,24 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) // Parse Symlink Path curNode = VFS_ParsePath(tmp, TruePath); + if(TruePath) + LOG("VFS", "*TruePath='%s'", *TruePath); // Error Check if(!curNode) { - Log("Symlink fail '%s'", tmp); + Log_Debug("VFS", "Symlink fail '%s'", tmp); free(tmp); // Free temp string + if(TruePath) free(TruePath); LEAVE('n'); return NULL; } + // Free temp link + free(tmp); + // Set Path Variable if(TruePath) { - *TruePath = tmp; - retLength = strlen(tmp); - } else { - free(tmp); // Free temp string + retLength = strlen(*TruePath); } continue; @@ -365,11 +368,12 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) if(!TruePath) continue; // Increase buffer space - tmp = realloc( *TruePath, retLength + strlen(&Path[ofs]) + 1 + 1 ); + tmp = realloc( *TruePath, retLength + strlen(pathEle) + 1 + 1 ); // Check if allocation succeeded if(!tmp) { Warning("VFS_ParsePath - Unable to reallocate true path buffer"); free(*TruePath); + *TruePath = NULL; if(curNode->Close) curNode->Close(curNode); LEAVE('n'); return NULL; @@ -377,9 +381,23 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) *TruePath = tmp; // Append to path (*TruePath)[retLength] = '/'; - strcpy(*TruePath+retLength+1, &Path[ofs]); + strcpy(*TruePath+retLength+1, pathEle); + + LOG("*TruePath = '%s'", *TruePath); + // - Extend Path - retLength += strlen(&Path[ofs])+1; + retLength += nextSlash + 1; + } + + if( !curNode->FindDir ) { + if(curNode->Close) curNode->Close(curNode); + if(TruePath) { + free(*TruePath); + *TruePath = NULL; + } + Log("FindDir fail on '%s'", Path); + LEAVE('n'); + return NULL; } // Get last node @@ -422,10 +440,10 @@ tVFS_Node *VFS_ParsePath(char *Path, char **TruePath) } /** - * \fn int VFS_Open(char *Path, Uint Mode) + * \fn int VFS_Open(const char *Path, Uint Mode) * \brief Open a file */ -int VFS_Open(char *Path, Uint Mode) +int VFS_Open(const char *Path, Uint Mode) { tVFS_Node *node; char *absPath; @@ -435,6 +453,10 @@ int VFS_Open(char *Path, Uint Mode) // Get absolute path absPath = VFS_GetAbsPath(Path); + if(absPath == NULL) { + Log_Warning("VFS", "VFS_Open: Path expansion failed '%s'", Path); + return -1; + } LOG("absPath = \"%s\"", absPath); // Parse path and get mount point node = VFS_ParsePath(absPath, NULL); @@ -485,50 +507,10 @@ int VFS_Open(char *Path, Uint Mode) return -1; } - // Check for a user open - if(Mode & VFS_OPENFLAG_USER) - { - // Allocate Buffer - if( MM_GetPhysAddr( (Uint)gaUserHandles ) == 0 ) - { - Uint addr, size; - size = CFGINT(CFG_VFS_MAXFILES) * sizeof(tVFS_Handle); - for(addr = 0; addr < size; addr += 0x1000) - MM_Allocate( (Uint)gaUserHandles + addr ); - memset( gaUserHandles, 0, size ); - } - // Get a handle - for(i=0;i= 0 ) { + LEAVE('x', i); + return i; } Log("VFS_Open: Out of handles"); @@ -540,7 +522,7 @@ int VFS_Open(char *Path, Uint Mode) /** * \brief Open a file from an open directory */ -int VFS_OpenChild(Uint *Errno, int FD, char *Name, Uint Mode) +int VFS_OpenChild(Uint *Errno, int FD, const char *Name, Uint Mode) { tVFS_Handle *h; tVFS_Node *node; @@ -585,50 +567,10 @@ int VFS_OpenChild(Uint *Errno, int FD, char *Name, Uint Mode) return -1; } - // Check for a user open - if(Mode & VFS_OPENFLAG_USER) - { - // Allocate Buffer - if( MM_GetPhysAddr( (Uint)gaUserHandles ) == 0 ) - { - Uint addr, size; - size = CFGINT(CFG_VFS_MAXFILES) * sizeof(tVFS_Handle); - for(addr = 0; addr < size; addr += 0x1000) - MM_Allocate( (Uint)gaUserHandles + addr ); - memset( gaUserHandles, 0, size ); - } - // Get a handle - for(i=0;i= 0 ) { + LEAVE('x', i); + return i; } Log_Error("VFS", "VFS_OpenChild - Out of handles"); @@ -648,7 +590,7 @@ void VFS_Close(int FD) // Get handle h = VFS_GetHandle(FD); if(h == NULL) { - Log_Warning("VFS", "Invalid file handle passed to VFS_Close, 0x%x\n", FD); + Log_Warning("VFS", "Invalid file handle passed to VFS_Close, 0x%x", FD); return; } @@ -760,32 +702,6 @@ int VFS_ChRoot(char *New) return 1; } -/** - * \fn tVFS_Handle *VFS_GetHandle(int FD) - * \brief Gets a pointer to the handle information structure - */ -tVFS_Handle *VFS_GetHandle(int FD) -{ - tVFS_Handle *h; - - //Log_Debug("VFS", "VFS_GetHandle: (FD=0x%x)", FD); - - if(FD < 0) return NULL; - - if(FD & VFS_KERNEL_FLAG) { - FD &= (VFS_KERNEL_FLAG - 1); - if(FD >= MAX_KERNEL_FILES) return NULL; - h = &gaKernelHandles[ FD ]; - } else { - if(FD >= CFGINT(CFG_VFS_MAXFILES)) return NULL; - h = &gaUserHandles[ FD ]; - } - - if(h->Node == NULL) return NULL; - //Log_Debug("VFS", "VFS_GetHandle: RETURN %p", h); - return h; -} - // === EXPORTS === EXPORT(VFS_Open); EXPORT(VFS_Close);