6 Mar
2025
6 Mar
'25
10:58 p.m.
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?
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/7472#note_97039