[janderwald] 73020: [USBAUDIO] - fix warnings & leaks based on Thomas feedback

Author: janderwald
Date: Sat Oct 22 12:34:03 2016
New Revision: 73020

URL: http://svn.reactos.org/svn/reactos?rev=73020&view=rev
Log:
[USBAUDIO]
- fix warnings & leaks based on Thomas feedback

Modified:
    trunk/reactos/drivers/usb/usbaudio/filter.c
    trunk/reactos/drivers/usb/usbaudio/pin.c

Modified: trunk/reactos/drivers/usb/usbaudio/filter.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/usb/usbaudio/filter...
==============================================================================
--- trunk/reactos/drivers/usb/usbaudio/filter.c	[iso-8859-1] (original)
+++ trunk/reactos/drivers/usb/usbaudio/filter.c	[iso-8859-1] Sat Oct 22 12:34:03 2016
@@ -1083,6 +1083,11 @@
         /* store result */
         *OutDescriptor = Descriptor;
     }
+    else
+    {
+        /* failed */
+        FreeFunction(Descriptor);
+    }

/* done */
     return Status;
@@ -1098,7 +1103,6 @@
     OUT PVOID *OutDescriptor)
 {
     NTSTATUS Status;
-    PUSB_STRING_DESCRIPTOR StringDescriptor;

/* retrieve descriptor */
     Status = USBAudioGetDescriptor(DeviceObject, USB_STRING_DESCRIPTOR_TYPE, DescriptorLength, DescriptorIndex, LanguageId, OutDescriptor);
@@ -1124,14 +1128,14 @@
     RtlInitUnicodeString(&DestinationString, L"\Registry\Machine\SYSTEM\CurrentControlSet\Control\MediaCategories\");

/* initialize object attributes */
-    InitializeObjectAttributes(&ObjectAttributes, &DestinationString, OBJ_CASE_INSENSITIVE | OBJ_OPENIF, NULL, NULL);
+    InitializeObjectAttributes(&ObjectAttributes, &DestinationString, OBJ_CASE_INSENSITIVE | OBJ_OPENIF | OBJ_KERNEL_HANDLE, NULL, NULL);

/* create the key */
     Status = ZwOpenKey(&Handle, KEY_ALL_ACCESS, &ObjectAttributes);
     if (NT_SUCCESS(Status))
     {
         /* initialize object attributes */
-        InitializeObjectAttributes(&ObjectAttributes, Name, OBJ_CASE_INSENSITIVE, Handle, NULL);
+        InitializeObjectAttributes(&ObjectAttributes, Name, OBJ_CASE_INSENSITIVE | OBJ_KERNEL_HANDLE, Handle, NULL);

Status = ZwCreateKey(OutHandle, KEY_ALL_ACCESS, &ObjectAttributes, 0, NULL, 0, NULL);
         ZwClose(Handle);
@@ -1228,8 +1232,8 @@
     if (!NT_SUCCESS(Status))
     {
         /* failed*/
-        //FreeFunction(ComponentId);
-        //return Status;
+        FreeFunction(ComponentId);
+        return Status;
     }
     FilterDescriptor->ComponentId = ComponentId;

@@ -1247,8 +1251,8 @@
     if (!NT_SUCCESS(Status))
     {
         /* failed*/
-        //FreeFunction(ComponentId);
-        //return Status;
+        FreeFunction(ComponentId);
+        return Status;
     }

/* lets create the filter */

Modified: trunk/reactos/drivers/usb/usbaudio/pin.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/usb/usbaudio/pin.c?...
==============================================================================
--- trunk/reactos/drivers/usb/usbaudio/pin.c	[iso-8859-1] (original)
+++ trunk/reactos/drivers/usb/usbaudio/pin.c	[iso-8859-1] Sat Oct 22 12:34:03 2016
@@ -789,7 +789,6 @@
     PPIN_CONTEXT PinContext;
     PLIST_ENTRY CurEntry;
     PIRP Irp;
-    PIO_STACK_LOCATION IoStack;
     PURB Urb;
     PUCHAR TransferBuffer, OutBuffer;
     ULONG Offset, Length;
@@ -827,7 +826,6 @@
         Irp = (PIRP)CONTAINING_RECORD(CurEntry, IRP, Tail.Overlay.ListEntry);

/* get urb from irp */
-        IoStack = IoGetNextIrpStackLocation(Irp);
         Urb = (PURB)Irp->Tail.Overlay.DriverContext[0];
         ASSERT(Urb);

@@ -882,6 +880,7 @@
         else
         {
             Status = KsStreamPointerAdvanceOffsets(LeadingStreamPointer, 0, Length, FALSE);
+            NT_ASSERT(NT_SUCCESS(Status));
             ASSERT(Length == Urb->UrbIsochronousTransfer.TransferBufferLength - Offset);
         }

@@ -1102,7 +1101,7 @@
     }

/* get pin descriptor */
-    PinDescriptor = &Filter->Descriptor->PinDescriptors[Pin->PinId];
+    PinDescriptor = (PKSPIN_DESCRIPTOR_EX)&Filter->Descriptor->PinDescriptors[Pin->PinId];

*DataSize = sizeof(KSDATAFORMAT_WAVEFORMATEX);
     if (DataBufferSize == 0)