Elizabeth Figura (@zfigura) commented about dlls/bluetoothapis/main.c:
- BOOL success;
- TRACE( "(%p, %p)\n", find, info );
- /* This method doesn't perform any validation for info, for some reason. */
- if (!find)
- {
SetLastError( ERROR_INVALID_HANDLE );
return FALSE;
- }
- success = device_find_set_next( find, info );
- if (!success)
SetLastError( ERROR_NO_MORE_ITEMS );
- return success;
Other functions seem to explicitly SetLastError( ERROR_SUCCESS ), should we be doing that here too?