[PATCH v3 1/2] ntoskrnl.exe: Implement IoRegisterDeviceInterface
Dmitry Timoshkov
dmitry at baikal.ru
Thu Oct 4 12:07:08 CDT 2018
Aric Stewart <aric at codeweavers.com> wrote:
> + do
> + {
> + rc = SetupDiEnumDeviceInfo( infoset, index, &devInfo );
> + if (IsEqualGUID( &devInfo.ClassGuid, class_guid ))
Without an rc check IsEqualGUID() will use bogus data.
> + {
> + BOOL check;
> + check = SetupDiGetDeviceInstanceIdW( infoset, &devInfo, id, size, &required );
> + if (check && strcmpW( id, instance_id ) == 0)
> + break;
> + }
> + index++;
> + } while (rc);
There are other places in the patch that don't check the API result
before using bogus data in case of an error.
--
Dmitry.
More information about the wine-devel
mailing list