Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-09-03 09:09:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

- Fix cleanup code

Browse source 
- We could possibly end up there with a NULL FileHandleObject and return a garbage FileHandle or crash
 - Now we always return a NULL FileHandle when we fail and free FileHandleObject if it has been allocated

svn path=/branches/aicom-network-fixes/; revision=35946
This commit is contained in:
Cameron Gutman 2008-09-04 19:08:08 +00:00
parent 4ff5fc5caa
commit 16427a1482
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
drivers/network/ndis/ndis/stubs.c
View file

@ -331,10 +331,12 @@ cleanup:
ExFreePool ( FullFileName.Buffer );
FullFileName.Buffer = NULL;
}
if ( !NT_SUCCESS(*Status) && FileHandleObject != NULL )
if ( !NT_SUCCESS(*Status) )
{
ExFreePool ( FileHandleObject );
FileHandleObject = NULL;
if( FileHandleObject ) {
ExFreePool ( FileHandleObject );
FileHandleObject = NULL;
}
*FileHandle = NULL;
}
else