Re: [PATCH v5 1/2] ntoskrnl.exe: Implement IoRegisterDeviceInterface

[Zebediah Figura]

> - required = 0;
> - SetupDiGetDeviceInterfaceDetailW( infoset, &infoData, NULL, 0,

&required, NULL );
> - if (required == 0)
> - {
> -
>    ERR( "Failed to get required size for interface\n" );
> -
>    return STATUS_UNSUCCESSFUL;
> - }
> -
> - required += sizeof(WCHAR);
> -
> - data = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY , required +

sizeof(SP_DEVICE_INTERFACE_DETAIL_DATA_W) );

I think this is redundant, SetupDiGetDeviceInterfaceDetail() will return
the required length including the size of the struct.

Otherwise this patch looks correct to me.