@me. I get the impression that the function used to return non-zero on invalid parameter, so altering it to return 0 would cause apps that previously got lucky (passed invalid parameters, but no noticable side effects) to run their exit sequence without the user asking them to. But yeah, something like return FALSE with GetLastError() being ERROR_INVALID_PARAMETER vs ERROR_SUCCESS or something would seem like a better work-around otherwise.
↧