Re: [PATCH v7 0/7] MR7472: winebth.sys part 6: Add support for device inquiry/scanning.

[Elizabeth Figura (＠zfigura)]

Elizabeth Figura (@zfigura) commented about dlls/bluetoothapis/main.c:
> - if (list->numOfDevices == 1)
> - {
> -
>    *devices = list;
> -
>    return TRUE;
> - }
> -
> - size = sizeof( *list ) + (list->numOfDevices - 1) * sizeof( list->deviceList[0] );
> - ptr = realloc( list,  size );
> - if (!ptr)
> - {
> -
>    free( list );
> -
>    SetLastError( ERROR_OUTOFMEMORY );
> -
>    return FALSE;
> - }
> - list = ptr;
> - success = DeviceIoControl( radio, IOCTL_BTH_GET_DEVICE_INFO, list, sizeof( *list ), list, size, &bytes, NULL );

Can't the number of devices change between the two calls? I would assume we don't want to fail in that case; we should probably be performing a loop.