[hpoussin] 21708: Don't dereference device objects if they were not referenced Remove one hack from kbdclass

Author: hpoussin
Date: Sat Apr 22 18:08:43 2006
New Revision: 21708

URL: http://svn.reactos.ru/svn/reactos?rev=21708&view=rev
Log:
Don't dereference device objects if they were not referenced
Remove one hack from kbdclass

Modified:
    trunk/reactos/base/setup/usetup/console.c
    trunk/reactos/drivers/input/kbdclass/kbdclass.c
    trunk/reactos/drivers/input/mouclass/mouclass.c

Modified: trunk/reactos/base/setup/usetup/console.c
URL: http://svn.reactos.ru/svn/reactos/trunk/reactos/base/setup/usetup/console.c?...
==============================================================================
--- trunk/reactos/base/setup/usetup/console.c (original)
+++ trunk/reactos/base/setup/usetup/console.c Sat Apr 22 18:08:43 2006
@@ -52,17 +52,16 @@
 NTSTATUS
 ConAllocConsole(VOID)
 {
+  UNICODE_STRING ScreenName = RTL_CONSTANT_STRING(L"\??\BlueScreen");
+  UNICODE_STRING KeyboardName = RTL_CONSTANT_STRING(L"\Device\KeyboardClass0");
   OBJECT_ATTRIBUTES ObjectAttributes;
   IO_STATUS_BLOCK IoStatusBlock;
-  UNICODE_STRING Name;
   NTSTATUS Status;
   CONSOLE_SCREEN_BUFFER_INFO csbi;

/* Open the screen */
-  RtlInitUnicodeString(&Name,
-		       L"\??\BlueScreen");
   InitializeObjectAttributes(&ObjectAttributes,
-			     &Name,
+			     &ScreenName,
    		     0,
    		     NULL,
    		     NULL);
@@ -76,10 +75,8 @@
     return(Status);

/* Open the keyboard */
-  RtlInitUnicodeString(&Name,
-		       L"\??\Keyboard");
   InitializeObjectAttributes(&ObjectAttributes,
-			     &Name,
+			     &KeyboardName,
    		     0,
    		     NULL,
    		     NULL);

Modified: trunk/reactos/drivers/input/kbdclass/kbdclass.c
URL: http://svn.reactos.ru/svn/reactos/trunk/reactos/drivers/input/kbdclass/kbdcl...
==============================================================================
--- trunk/reactos/drivers/input/kbdclass/kbdclass.c (original)
+++ trunk/reactos/drivers/input/kbdclass/kbdclass.c Sat Apr 22 18:08:43 2006
@@ -284,7 +284,6 @@
    OUT PDEVICE_OBJECT *ClassDO OPTIONAL)
 {
    PCLASS_DRIVER_EXTENSION DriverExtension;
-	UNICODE_STRING SymbolicLinkName = RTL_CONSTANT_STRING(L"\??\Keyboard");
    ULONG DeviceId = 0;
    ULONG PrefixLength;
    UNICODE_STRING DeviceNameU;
@@ -374,10 +373,6 @@
    	DriverExtension->RegistryPath.Buffer,
    	DriverExtension->RegistryPath.MaximumLength);

-	/* HACK: 1st stage setup needs a keyboard to open it in user-mode
-	 * Create a link to user space... */
-	IoCreateSymbolicLink(&SymbolicLinkName, &DeviceNameU);
-
    ExFreePool(DeviceNameU.Buffer);

if (ClassDO)
@@ -597,7 +592,6 @@
    if (!NT_SUCCESS(Status))
    {
    	DPRINT("ConnectPortDriver() failed with status 0x%08lx\n", Status);
-		ObDereferenceObject(Fdo);
    	goto cleanup;
    }
    Fdo->Flags &= ~DO_DEVICE_INITIALIZING;

Modified: trunk/reactos/drivers/input/mouclass/mouclass.c
URL: http://svn.reactos.ru/svn/reactos/trunk/reactos/drivers/input/mouclass/moucl...
==============================================================================
--- trunk/reactos/drivers/input/mouclass/mouclass.c (original)
+++ trunk/reactos/drivers/input/mouclass/mouclass.c Sat Apr 22 18:08:43 2006
@@ -569,7 +569,6 @@
    if (!NT_SUCCESS(Status))
    {
    	DPRINT("ConnectPortDriver() failed with status 0x%08lx\n", Status);
-		ObDereferenceObject(Fdo);
    	goto cleanup;
    }
    Fdo->Flags &= ~DO_DEVICE_INITIALIZING;