X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=AcessNative%2Facesskernel_src%2Fvfs_handle.c;h=93b6a9d674d7354fbb65b8cce57566f1fa7eda6c;hb=c01cd6b0f42fa4cd400c5d9493c8b7ec92e72c5b;hp=1a5ebd52a379b9af70bca8d1d9cad02454e1d059;hpb=132e159bfadf0f7669e80ef34ef660b99aa8b83c;p=tpg%2Facess2.git

diff --git a/AcessNative/acesskernel_src/vfs_handle.c b/AcessNative/acesskernel_src/vfs_handle.c
index 1a5ebd52..93b6a9d6 100644
--- a/AcessNative/acesskernel_src/vfs_handle.c
+++ b/AcessNative/acesskernel_src/vfs_handle.c
@@ -40,13 +40,16 @@ tUserHandles *VFS_int_GetUserHandles(int PID, int bCreate)
 		if( ent->PID == PID ) {
 			//if( bCreate )
 			//	Log_Warning("VFS", "Process %i already has a handle list", PID);
+			LOG("Found list for %i", PID);
 			return ent;
 		}
 		if( ent->PID > PID )	break;
 	}
 	
-	if(!bCreate)
+	if(!bCreate) {
+		LOG("Not creating for %i", PID);
 		return NULL;
+	}
 	
 	ent = calloc( 1, sizeof(tUserHandles) );
 	ent->PID = PID;
@@ -58,7 +61,7 @@ tUserHandles *VFS_int_GetUserHandles(int PID, int bCreate)
 		ent->Next = gpUserHandles;
 		gpUserHandles = ent;
 	}
-	Log_Notice("VFS", "Created handle list for process %i", PID);
+	LOG("Created handle list for process %i", PID);
 	return ent;
 }
 
@@ -68,20 +71,21 @@ tUserHandles *VFS_int_GetUserHandles(int PID, int bCreate)
 void VFS_CloneHandleList(int PID)
 {
 	tUserHandles	*ent;
-	tUserHandles	*cur;
-	 int	i, maxhandles;
+	const tUserHandles	*cur;
+	 int	maxhandles;
 	
 	cur = VFS_int_GetUserHandles(Threads_GetPID(), 0);
 	if(!cur)	return ;	// Don't need to do anything if the current list is empty
 	
 	ent = VFS_int_GetUserHandles(PID, 1);
 	
-	maxhandles = *Threads_GetMaxFD();
+	maxhandles = *Threads_GetMaxFD(NULL);
 	memcpy(ent->Handles, cur->Handles, maxhandles*sizeof(tVFS_Handle));
 	
-	for( i = 0; i < maxhandles; i ++ )
+	// Reference all
+	for( int i = 0; i < maxhandles; i ++ )
 	{
-		if(!cur->Handles[i].Node)	continue;
+		if(!ent->Handles[i].Node)	continue;
 		
 		if(ent->Handles[i].Node->Type->Reference)
 			ent->Handles[i].Node->Type->Reference(ent->Handles[i].Node);
@@ -91,31 +95,35 @@ void VFS_CloneHandleList(int PID)
 void VFS_CloneHandlesFromList(int PID, int nFD, int FDs[])
 {
 	tUserHandles	*ent;
-	tUserHandles	*cur;
-	 int	i, maxhandles;
+	const tUserHandles	*cur;
+	 int	maxhandles;
 	
 	cur = VFS_int_GetUserHandles(Threads_GetPID(), 0);
 	if(!cur)	return ;	// Don't need to do anything if the current list is empty
 	
 	ent = VFS_int_GetUserHandles(PID, 1);
-	
-	maxhandles = *Threads_GetMaxFD();
+
+	LOG("Copying %i FDs from %i", nFD, PID);
+
+	maxhandles = *Threads_GetMaxFD(NULL);
 	if( nFD > maxhandles )
 		nFD = maxhandles;
-	for( i = 0; i < nFD; i ++ )
+	for( int i = 0; i < nFD; i ++ )
 	{
-		if( FDs[i] >= maxhandles ) {
+		 int	fd = FDs[i];
+		if( fd >= maxhandles ) {
 			ent->Handles[i].Node = NULL;
 			continue ;
 		}
-		memcpy(&ent->Handles[i], &cur->Handles[ FDs[i] ], sizeof(tVFS_Handle));
+		memcpy(&ent->Handles[i], &cur->Handles[fd], sizeof(tVFS_Handle));
 	}
-	for( ; i < maxhandles; i ++ )
-		cur->Handles[i].Node = NULL;
+	for( int i = nFD; i < maxhandles; i ++ )
+		ent->Handles[i].Node = NULL;
 	
-	for( i = 0; i < maxhandles; i ++ )
+	// Reference
+	for( int i = 0; i < maxhandles; i ++ )
 	{
-		if(!cur->Handles[i].Node)	continue;
+		if(!ent->Handles[i].Node)	continue;
 		
 		if(ent->Handles[i].Node->Type->Reference)
 			ent->Handles[i].Node->Type->Reference(ent->Handles[i].Node);
@@ -147,11 +155,10 @@ tVFS_Handle *VFS_GetHandle(int FD)
 	}
 	else
 	{
-		tUserHandles	*ent;
 		 int	pid = Threads_GetPID();
-		 int	maxhandles = *Threads_GetMaxFD();
+		 int	maxhandles = *Threads_GetMaxFD(NULL);
 		
-		ent = VFS_int_GetUserHandles(pid, 0);
+		tUserHandles *ent = VFS_int_GetUserHandles(pid, 0);
 		if(!ent) {
 			Log_Error("VFS", "Client %i does not have a handle list (>)", pid);
 			return NULL;
@@ -186,17 +193,17 @@ int VFS_SetHandle(int FD, tVFS_Node *Node, int Mode)
 	else {
 		tUserHandles	*ent;
 		 int	pid = Threads_GetPID();
-		 int	maxhandles = *Threads_GetMaxFD();
+		 int	maxhandles = *Threads_GetMaxFD(NULL);
 		
 		ent = VFS_int_GetUserHandles(pid, 0);
 		if(!ent) {
 			Log_Error("VFS", "Client %i does not have a handle list (>)", pid);
-			return NULL;
+			return -1;
 		}
 		
 		if(FD >= maxhandles) {
 			LOG("FD (%i) > Limit (%i), RETURN NULL", FD, maxhandles);
-			return NULL;
+			return -1;
 		}
 		h = &ent->Handles[ FD ];
 	}
@@ -208,17 +215,15 @@ int VFS_SetHandle(int FD, tVFS_Node *Node, int Mode)
 
 int VFS_AllocHandle(int bIsUser, tVFS_Node *Node, int Mode)
 {
-	 int	i;
-	
 	// Check for a user open
 	if(bIsUser)
 	{
 		tUserHandles	*ent;
-		 int	maxhandles = *Threads_GetMaxFD();
+		 int	maxhandles = *Threads_GetMaxFD(NULL);
 		// Find the PID's handle list
 		ent = VFS_int_GetUserHandles(Threads_GetPID(), 1);
 		// Get a handle
-		for( i = 0; i < maxhandles; i ++ )
+		for( int i = 0; i < maxhandles; i ++ )
 		{
 			if(ent->Handles[i].Node)	continue;
 			ent->Handles[i].Node = Node;
@@ -230,7 +235,7 @@ int VFS_AllocHandle(int bIsUser, tVFS_Node *Node, int Mode)
 	else
 	{
 		// Get a handle
-		for(i=0;i<MAX_KERNEL_FILES;i++)
+		for(int i = 0; i < MAX_KERNEL_FILES; i ++)
 		{
 			if(gaKernelHandles[i].Node)	continue;
 			gaKernelHandles[i].Node = Node;
@@ -242,3 +247,18 @@ int VFS_AllocHandle(int bIsUser, tVFS_Node *Node, int Mode)
 	
 	return -1;
 }
+
+void VFS_ClearHandles(int PID)
+{
+	// Find the PID's handle list
+	tUserHandles *ent = VFS_int_GetUserHandles(PID, 0);
+	if( !ent )	return;
+	// Get a handle
+	 int	maxhandles = *Threads_GetMaxFD(NULL);
+	for( int i = 0; i < maxhandles; i ++ )
+	{
+		if(ent->Handles[i].Node)	continue;
+		_CloseNode(ent->Handles[i].Node);
+		ent->Handles[i].Node = NULL;
+	}
+}