[tfaber] 62584: [NTOS:IO] - Fix some incredibly useless debug prints

Author: tfaber
Date: Sun Mar 30 09:35:50 2014
New Revision: 62584

URL: http://svn.reactos.org/svn/reactos?rev=62584&view=rev
Log:
[NTOS:IO]
- Fix some incredibly useless debug prints

Modified:
    trunk/reactos/ntoskrnl/io/iomgr/iorsrce.c

Modified: trunk/reactos/ntoskrnl/io/iomgr/iorsrce.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/iorsrce.c...
==============================================================================
--- trunk/reactos/ntoskrnl/io/iomgr/iorsrce.c	[iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/io/iomgr/iorsrce.c	[iso-8859-1] Sun Mar 30 09:35:50 2014
@@ -233,8 +233,8 @@
             /* How much buffer space */
             Status = ZwQueryValueKey(ControllerKeyHandle, &ControllerString, KeyValueFullInformation, NULL, 0, &LenKeyFullInformation);

-	    if(!NT_SUCCESS(Status) && Status != STATUS_BUFFER_TOO_SMALL && Status != STATUS_BUFFER_OVERFLOW)
-	      continue;
+            if (!NT_SUCCESS(Status) && Status != STATUS_BUFFER_TOO_SMALL && Status != STATUS_BUFFER_OVERFLOW)
+               continue;

/* Allocate it */
             ControllerInformation[ControllerLoop] = ExAllocatePoolWithTag(PagedPool, LenKeyFullInformation, TAG_IO_RESOURCE);
@@ -337,7 +337,7 @@

/* Create String */
          Status |= RtlAppendUnicodeToString(&ControllerRootRegName, L"\");
-	 Status |= RtlAppendUnicodeStringToString(&ControllerRootRegName, &TempString);
+         Status |= RtlAppendUnicodeStringToString(&ControllerRootRegName, &TempString);

/* Something messed up */
          if (!NT_SUCCESS(Status)) break;
@@ -362,11 +362,11 @@
                /* How much buffer space */
                Status = ZwQueryValueKey(PeripheralKeyHandle, &PeripheralString, KeyValueFullInformation, NULL, 0, &LenKeyFullInformation);

-	       if(!NT_SUCCESS(Status) && Status != STATUS_BUFFER_TOO_SMALL && Status != STATUS_BUFFER_OVERFLOW)
-	       {
-	         PeripheralInformation[PeripheralLoop] = NULL;
-		 continue;
-	       }
+               if (!NT_SUCCESS(Status) && Status != STATUS_BUFFER_TOO_SMALL && Status != STATUS_BUFFER_OVERFLOW)
+               {
+                 PeripheralInformation[PeripheralLoop] = NULL;
+                 continue;
+               }

/* Allocate it */
                PeripheralInformation[PeripheralLoop] = ExAllocatePoolWithTag(PagedPool, LenKeyFullInformation, TAG_IO_RESOURCE);
@@ -482,7 +482,7 @@
    /* Allocate it */
    FullInformation = ExAllocatePoolWithTag(PagedPool, LenFullInformation, TAG_IO_RESOURCE);

-   if(!FullInformation)
+   if (!FullInformation)
      return STATUS_NO_MEMORY;

/* Get the Information */
@@ -722,7 +722,7 @@
                                       &ObjectAttributes);
     if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed opening given symbolic link!\n");
+        DPRINT("Failed to open symlink %wZ, Status=%lx\n", NtSystemPartitionDeviceName, Status);
         return;
     }

@@ -742,7 +742,7 @@

if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed querying given symbolic link!\n");
+        DPRINT("Failed querying symlink %wZ, Status=%lx\n", NtSystemPartitionDeviceName, Status);
         return;
     }

@@ -756,7 +756,7 @@
                                   KEY_ALL_ACCESS);
     if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed opening registry!\n");
+        DPRINT("Failed to open HKLM\SYSTEM, Status=%lx\n", Status);
         return;
     }

@@ -775,7 +775,7 @@

if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed opening/creating Setup key!\n");
+        DPRINT("Failed opening/creating Setup key, Status=%lx\n", Status);
         return;
     }

@@ -791,7 +791,7 @@
                            LinkTarget.Length + sizeof(WCHAR));
     if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed writing SystemPartition value!\n");
+        DPRINT("Failed writing SystemPartition value, Status=%lx\n", Status);
     }

/* Prepare for second data writing... */
@@ -811,10 +811,10 @@
                            0,
                            REG_SZ,
                            OsLoaderPathName->Buffer,
-                           OsLoaderPathName->Length + sizeof(WCHAR));
+                           OsLoaderPathName->Length + sizeof(UNICODE_NULL));
     if (!NT_SUCCESS(Status))
     {
-        DPRINT("Failed writing OsLoaderPath value!\n");
+        DPRINT("Failed writing OsLoaderPath value, Status=%lx\n", Status);
     }

/* We're finally done! */
@@ -870,22 +870,22 @@
 {
     NTSTATUS Status;
     PCM_RESOURCE_LIST ResourceList;
-    
+
     DPRINT1("IoReportResourceUsage is halfplemented!\n");
-    
+
     if (!DriverList && !DeviceList)
         return STATUS_INVALID_PARAMETER;
-    
+
     if (DeviceList)
         ResourceList = DeviceList;
     else
         ResourceList = DriverList;
-    
+
     Status = IopDetectResourceConflict(ResourceList, FALSE, NULL);
     if (Status == STATUS_CONFLICTING_ADDRESSES)
     {
         *ConflictDetected = TRUE;
-        
+
         if (!OverrideConflict)
         {
             DPRINT1("Denying an attempt to claim resources currently in use by another device!\n");
@@ -902,9 +902,9 @@
     }

/* TODO: Claim resources in registry */
-    
+
     *ConflictDetected = FALSE;
-    
+
     return STATUS_SUCCESS;
 }