Hi Christian, thanks for working on setupapi, it could use some help. Huw already commnented on your patch, I have a couple more:
+ if (!list) { + goto cleanup; + }
The style in this file has opening braces on lines of their own, e.g.: if (hInf == INVALID_HANDLE_VALUE) { return FALSE; } Please stick with that.
+ if (MachineName != NULL) {
You should probably just fail if the machine name isn't NULL, since you don't use it and you don't connect to a remote machine anyway.
--Juan
__________________________________________________ Do You Yahoo!? Tired of spam? Yahoo! Mail has the best spam protection around http://mail.yahoo.com