September 2007 - Wine-Commits

Juan Lang : setupapi: When freeing an interface, delete it from the registry if its device is a phantom.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 7de0f1d583ec4a910db198becbb82f11639129d5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7de0f1d583ec4a910db198bec… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:08:51 2007 -0700 setupapi: When freeing an interface, delete it from the registry if its device is a phantom. --- dlls/setupapi/devinst.c | 9 +++++++++ 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 9fe76e2..a5356e5 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -152,6 +152,15 @@ static void SETUPDI_FreeInterfaceInstances(struct InterfaceInstances *instances) struct InterfaceInfo *ifaceInfo = (struct InterfaceInfo *)instances->instances[i].Reserved; + if (ifaceInfo->device && ifaceInfo->device->Reserved) + { + struct DeviceInfo *devInfo = + (struct DeviceInfo *)ifaceInfo->device->Reserved; + + if (devInfo->phantom) + SetupDiDeleteDeviceInterfaceRegKey(devInfo->set, + &instances->instances[i], 0); + } HeapFree(GetProcessHeap(), 0, ifaceInfo->referenceString); HeapFree(GetProcessHeap(), 0, ifaceInfo->symbolicLink); }

1 0

Juan Lang : setupapi: Implement SetupDiDeleteDeviceInterfaceRegKey.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: dadcb201d9bd35fdceccd6d9d6c968cb41617f3b URL: http://source.winehq.org/git/wine.git/?a=commit;h=dadcb201d9bd35fdceccd6d9d… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:08:28 2007 -0700 setupapi: Implement SetupDiDeleteDeviceInterfaceRegKey. --- dlls/setupapi/devinst.c | 99 ++++++++++++++++++++++++++++++++++--------- dlls/setupapi/setupapi.spec | 1 + 2 files changed, 80 insertions(+), 20 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index a1a060c..9fe76e2 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -2224,6 +2224,33 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyA( return key; } +static PWSTR SETUPDI_GetInstancePath(struct InterfaceInfo *ifaceInfo) +{ + static const WCHAR hash[] = {'#',0}; + PWSTR instancePath = NULL; + + if (ifaceInfo->referenceString) + { + instancePath = HeapAlloc(GetProcessHeap(), 0, + (lstrlenW(ifaceInfo->referenceString) + 2) * sizeof(WCHAR)); + if (instancePath) + { + lstrcpyW(instancePath, hash); + lstrcatW(instancePath, ifaceInfo->referenceString); + } + else + SetLastError(ERROR_OUTOFMEMORY); + } + else + { + instancePath = HeapAlloc(GetProcessHeap(), 0, + (lstrlenW(hash) + 1) * sizeof(WCHAR)); + if (instancePath) + lstrcpyW(instancePath, hash); + } + return instancePath; +} + /*********************************************************************** * SetupDiCreateDeviceInterfaceRegKeyW (SETUPAPI.@) */ @@ -2271,28 +2298,10 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW( if (!(l = RegCreateKeyExW(interfacesKey, bracedGuidString, 0, NULL, 0, samDesired, NULL, &parent, NULL))) { - static const WCHAR hash[] = {'#',0}; struct InterfaceInfo *ifaceInfo = (struct InterfaceInfo *)DeviceInterfaceData->Reserved; - LPCWSTR instancePath = NULL; - LPWSTR referencePath = NULL; + PWSTR instancePath = SETUPDI_GetInstancePath(ifaceInfo); - if (ifaceInfo->referenceString) - { - referencePath = HeapAlloc(GetProcessHeap(), 0, - (lstrlenW(ifaceInfo->referenceString) + 2) * - sizeof(WCHAR)); - if (referencePath) - { - lstrcpyW(referencePath, hash); - lstrcatW(referencePath, ifaceInfo->referenceString); - instancePath = referencePath; - } - else - SetLastError(ERROR_OUTOFMEMORY); - } - else - instancePath = hash; if (instancePath) { LONG l; @@ -2307,7 +2316,7 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW( else if (InfHandle) FIXME("INF section installation unsupported\n"); } - HeapFree(GetProcessHeap(), 0, referencePath); + HeapFree(GetProcessHeap(), 0, instancePath); RegCloseKey(parent); } else @@ -2320,6 +2329,56 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW( } /*********************************************************************** + * SetupDiDeleteDeviceInterfaceRegKey (SETUPAPI.@) + */ +BOOL WINAPI SetupDiDeleteDeviceInterfaceRegKey( + HDEVINFO DeviceInfoSet, + PSP_DEVICE_INTERFACE_DATA DeviceInterfaceData, + DWORD Reserved) +{ + struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; + HKEY parent; + BOOL ret = FALSE; + + TRACE("%p %p %d\n", DeviceInfoSet, DeviceInterfaceData, Reserved); + + if (!DeviceInfoSet || DeviceInfoSet == (HDEVINFO)INVALID_HANDLE_VALUE || + set->magic != SETUP_DEVICE_INFO_SET_MAGIC) + { + SetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + if (!DeviceInterfaceData || + DeviceInterfaceData->cbSize != sizeof(SP_DEVICE_INTERFACE_DATA) || + !DeviceInterfaceData->Reserved) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } + parent = SetupDiOpenClassRegKeyExW(&DeviceInterfaceData->InterfaceClassGuid, + KEY_ALL_ACCESS, DIOCR_INTERFACE, NULL, NULL); + if (parent != INVALID_HANDLE_VALUE) + { + struct InterfaceInfo *ifaceInfo = + (struct InterfaceInfo *)DeviceInterfaceData->Reserved; + PWSTR instancePath = SETUPDI_GetInstancePath(ifaceInfo); + + if (instancePath) + { + LONG l = RegDeleteKeyW(parent, instancePath); + + if (l) + SetLastError(l); + else + ret = TRUE; + HeapFree(GetProcessHeap(), 0, instancePath); + } + RegCloseKey(parent); + } + return ret; +} + +/*********************************************************************** * SetupDiEnumDeviceInterfaces (SETUPAPI.@) * * PARAMS diff --git a/dlls/setupapi/setupapi.spec b/dlls/setupapi/setupapi.spec index 5ba940e..f9589b1 100644 --- a/dlls/setupapi/setupapi.spec +++ b/dlls/setupapi/setupapi.spec @@ -295,6 +295,7 @@ @ stub SetupDiDeleteDevRegKey @ stub SetupDiDeleteDeviceInfo @ stub SetupDiDeleteDeviceInterfaceData +@ stdcall SetupDiDeleteDeviceInterfaceRegKey(ptr ptr long) @ stub SetupDiDeleteDeviceRegKey @ stub SetupDiDestroyClassImageList @ stdcall SetupDiDestroyDeviceInfoList(long)

1 0

Juan Lang : setupapi: Implement SetupDiCreateDeviceInterfaceRegKeyA/W.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 21b7e1faf66e9a04473cf97c49a10d4545fa8be7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=21b7e1faf66e9a04473cf97c4… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:07:16 2007 -0700 setupapi: Implement SetupDiCreateDeviceInterfaceRegKeyA/W. --- dlls/setupapi/devinst.c | 105 +++++++++++++++++++++++++++++++++++++++++++++-- 1 files changed, 101 insertions(+), 4 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 9593b1c..6d90d15 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -2171,9 +2171,27 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyA( HINF InfHandle, PCSTR InfSectionName) { - FIXME("%p %p %d %08x %p %p\n", DeviceInfoSet, DeviceInterfaceData, Reserved, + HKEY key; + PWSTR InfSectionNameW = NULL; + + TRACE("%p %p %d %08x %p %p\n", DeviceInfoSet, DeviceInterfaceData, Reserved, samDesired, InfHandle, InfSectionName); - return INVALID_HANDLE_VALUE; + if (InfHandle) + { + if (!InfSectionName) + { + SetLastError(ERROR_INVALID_PARAMETER); + return INVALID_HANDLE_VALUE; + } + InfSectionNameW = MultiByteToUnicode(InfSectionName, CP_ACP); + if (!InfSectionNameW) + return INVALID_HANDLE_VALUE; + } + key = SetupDiCreateDeviceInterfaceRegKeyW(DeviceInfoSet, + DeviceInterfaceData, Reserved, samDesired, InfHandle, + InfSectionNameW); + MyFree(InfSectionNameW); + return key; } /*********************************************************************** @@ -2187,9 +2205,88 @@ HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW( HINF InfHandle, PCWSTR InfSectionName) { - FIXME("%p %p %d %08x %p %p\n", DeviceInfoSet, DeviceInterfaceData, Reserved, + struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; + HKEY key = INVALID_HANDLE_VALUE, interfacesKey; + LONG l; + + TRACE("%p %p %d %08x %p %p\n", DeviceInfoSet, DeviceInterfaceData, Reserved, samDesired, InfHandle, InfSectionName); - return INVALID_HANDLE_VALUE; + + if (!DeviceInfoSet || DeviceInfoSet == (HDEVINFO)INVALID_HANDLE_VALUE || + set->magic != SETUP_DEVICE_INFO_SET_MAGIC) + { + SetLastError(ERROR_INVALID_HANDLE); + return INVALID_HANDLE_VALUE; + } + if (!DeviceInterfaceData || + DeviceInterfaceData->cbSize != sizeof(SP_DEVICE_INTERFACE_DATA) || + !DeviceInterfaceData->Reserved) + { + SetLastError(ERROR_INVALID_PARAMETER); + return INVALID_HANDLE_VALUE; + } + if (InfHandle && !InfSectionName) + { + SetLastError(ERROR_INVALID_PARAMETER); + return INVALID_HANDLE_VALUE; + } + if (!(l = RegCreateKeyExW(HKEY_LOCAL_MACHINE, DeviceClasses, 0, NULL, 0, + samDesired, NULL, &interfacesKey, NULL))) + { + HKEY parent; + WCHAR bracedGuidString[39]; + + SETUPDI_GuidToString(&DeviceInterfaceData->InterfaceClassGuid, + bracedGuidString); + if (!(l = RegCreateKeyExW(interfacesKey, bracedGuidString, 0, NULL, 0, + samDesired, NULL, &parent, NULL))) + { + static const WCHAR hash[] = {'#',0}; + struct InterfaceInfo *ifaceInfo = + (struct InterfaceInfo *)DeviceInterfaceData->Reserved; + LPCWSTR instancePath = NULL; + LPWSTR referencePath = NULL; + + if (ifaceInfo->referenceString) + { + referencePath = HeapAlloc(GetProcessHeap(), 0, + (lstrlenW(ifaceInfo->referenceString) + 2) * + sizeof(WCHAR)); + if (referencePath) + { + lstrcpyW(referencePath, hash); + lstrcatW(referencePath, ifaceInfo->referenceString); + instancePath = referencePath; + } + else + SetLastError(ERROR_OUTOFMEMORY); + } + else + instancePath = hash; + if (instancePath) + { + LONG l; + + l = RegCreateKeyExW(parent, instancePath, 0, NULL, 0, + samDesired, NULL, &key, NULL); + if (l) + { + SetLastError(l); + key = INVALID_HANDLE_VALUE; + } + else if (InfHandle) + FIXME("INF section installation unsupported\n"); + } + HeapFree(GetProcessHeap(), 0, referencePath); + RegCloseKey(parent); + } + else + SetLastError(l); + RegCloseKey(interfacesKey); + } + else + SetLastError(l); + return key; } /***********************************************************************

1 0

Juan Lang : setupapi: Store pointer to an interface' s device in the interface instance.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 5ce7fcae8e6526bdd5e1e5520adb24fd6c8c5c44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5ce7fcae8e6526bdd5e1e5520… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:06:18 2007 -0700 setupapi: Store pointer to an interface's device in the interface instance. --- dlls/setupapi/devinst.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index cb1605f..57c142b 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -104,8 +104,9 @@ struct DeviceInfoSet /* Pointed to by SP_DEVICE_INTERFACE_DATA's Reserved member */ struct InterfaceInfo { - LPWSTR referenceString; - LPWSTR symbolicLink; + LPWSTR referenceString; + LPWSTR symbolicLink; + PSP_DEVINFO_DATA device; }; /* A device may have multiple instances of the same interface, so this holds @@ -258,10 +259,11 @@ static LPWSTR SETUPDI_CreateSymbolicLinkPath(LPCWSTR instanceId, * the device, if it doesn't already exist in the device. If iface is not * NULL, returns a pointer to the newly added (or already existing) interface. */ -static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, +static BOOL SETUPDI_AddInterfaceInstance(PSP_DEVINFO_DATA DeviceInfoData, const GUID *InterfaceClassGuid, LPCWSTR ReferenceString, SP_DEVICE_INTERFACE_DATA **ifaceData) { + struct DeviceInfo *devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; BOOL newInterface = FALSE, ret; struct InterfaceInstances *iface = NULL; @@ -313,6 +315,7 @@ static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, if (ifaceInfo) { ret = TRUE; + ifaceInfo->device = DeviceInfoData; ifaceInfo->symbolicLink = SETUPDI_CreateSymbolicLinkPath( devInfo->instanceId, InterfaceClassGuid, ReferenceString); @@ -1736,7 +1739,6 @@ end: static void SETUPDI_AddDeviceInterfaces(SP_DEVINFO_DATA *dev, HKEY key, const GUID *interface) { - struct DeviceInfo *devInfo = (struct DeviceInfo *)dev->Reserved; DWORD i, len; WCHAR subKeyName[MAX_PATH]; LONG l = ERROR_SUCCESS; @@ -1751,7 +1753,7 @@ static void SETUPDI_AddDeviceInterfaces(SP_DEVINFO_DATA *dev, HKEY key, SP_DEVICE_INTERFACE_DATA *iface = NULL; /* The subkey name is the reference string, with a '#' prepended */ - SETUPDI_AddInterfaceInstance(devInfo, interface, subKeyName + 1, + SETUPDI_AddInterfaceInstance(dev, interface, subKeyName + 1, &iface); l = RegOpenKeyExW(key, subKeyName, 0, KEY_READ, &subKey); if (!l) @@ -2113,7 +2115,6 @@ BOOL WINAPI SetupDiCreateDeviceInterfaceW( PSP_DEVICE_INTERFACE_DATA DeviceInterfaceData) { struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; - struct DeviceInfo *devInfo; SP_DEVICE_INTERFACE_DATA *iface = NULL; BOOL ret; @@ -2142,8 +2143,7 @@ BOOL WINAPI SetupDiCreateDeviceInterfaceW( SetLastError(ERROR_INVALID_USER_BUFFER); return FALSE; } - devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; - if ((ret = SETUPDI_AddInterfaceInstance(devInfo, InterfaceClassGuid, + if ((ret = SETUPDI_AddInterfaceInstance(DeviceInfoData, InterfaceClassGuid, ReferenceString, &iface))) { if (DeviceInterfaceData)

1 0

Juan Lang : setupapi: Return the interface' s device from SetupDiGetDeviceInterfaceDetailA/W.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 443cfda4a6c98a43ad039187448f8835d14ab657 URL: http://source.winehq.org/git/wine.git/?a=commit;h=443cfda4a6c98a43ad0391874… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:06:46 2007 -0700 setupapi: Return the interface's device from SetupDiGetDeviceInterfaceDetailA/W. --- dlls/setupapi/devinst.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 57c142b..9593b1c 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -2401,6 +2401,8 @@ BOOL WINAPI SetupDiGetDeviceInterfaceDetailA( NULL, NULL); else DeviceInterfaceDetailData->DevicePath[0] = '\0'; + if (DeviceInfoData && DeviceInfoData->cbSize == sizeof(SP_DEVINFO_DATA)) + memcpy(DeviceInfoData, info->device, sizeof(SP_DEVINFO_DATA)); ret = TRUE; } else @@ -2467,6 +2469,8 @@ BOOL WINAPI SetupDiGetDeviceInterfaceDetailW( lstrcpyW(DeviceInterfaceDetailData->DevicePath, info->symbolicLink); else DeviceInterfaceDetailData->DevicePath[0] = '\0'; + if (DeviceInfoData && DeviceInfoData->cbSize == sizeof(SP_DEVINFO_DATA)) + memcpy(DeviceInfoData, info->device, sizeof(SP_DEVINFO_DATA)); ret = TRUE; } else

1 0

Juan Lang : setupapi: Save interface to registry when creating it.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: e69520ab6e1191308fa98272822e0f13bb0086ea URL: http://source.winehq.org/git/wine.git/?a=commit;h=e69520ab6e1191308fa982728… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:08:01 2007 -0700 setupapi: Save interface to registry when creating it. --- dlls/setupapi/devinst.c | 13 ++++++++++++- 1 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index acaa8ae..a1a060c 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -336,6 +336,8 @@ static BOOL SETUPDI_AddInterfaceInstance(PSP_DEVINFO_DATA DeviceInfoData, ifaceInfo->referenceString = NULL; if (ret) { + HKEY key; + iface->cInstances++; instance->cbSize = sizeof(SP_DEVICE_INTERFACE_DATA); @@ -346,7 +348,16 @@ static BOOL SETUPDI_AddInterfaceInstance(PSP_DEVINFO_DATA DeviceInfoData, if (newInterface) memcpy(&iface->guid, InterfaceClassGuid, sizeof(GUID)); - /* FIXME: now create this homeboy in the registry */ + key = SetupDiCreateDeviceInterfaceRegKeyW(devInfo->set, + instance, 0, KEY_WRITE, NULL, NULL); + if (key != INVALID_HANDLE_VALUE) + { + RegSetValueExW(key, SymbolicLink, 0, REG_SZ, + (BYTE *)ifaceInfo->symbolicLink, + lstrlenW(ifaceInfo->symbolicLink) * + sizeof(WCHAR)); + RegCloseKey(key); + } if (ifaceData) *ifaceData = instance; }

1 0

Juan Lang : setupapi: Store pointer to set in device, and use it to make sure that a device is a member of a set.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: d0a2866a90d420d90cf8f45879206ea944d3aeea URL: http://source.winehq.org/git/wine.git/?a=commit;h=d0a2866a90d420d90cf8f4587… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:07:37 2007 -0700 setupapi: Store pointer to set in device, and use it to make sure that a device is a member of a set. --- dlls/setupapi/devinst.c | 33 ++++++++++++++++++++++++++------- 1 files changed, 26 insertions(+), 7 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 6d90d15..acaa8ae 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -124,10 +124,11 @@ struct InterfaceInstances /* Pointed to by SP_DEVINFO_DATA's Reserved member */ struct DeviceInfo { - HKEY key; - BOOL phantom; - LPWSTR instanceId; - struct list interfaces; + struct DeviceInfoSet *set; + HKEY key; + BOOL phantom; + LPWSTR instanceId; + struct list interfaces; }; static void SETUPDI_GuidToString(const GUID *guid, LPWSTR guidStr) @@ -386,14 +387,15 @@ static BOOL SETUPDI_SetInterfaceSymbolicLink(SP_DEVICE_INTERFACE_DATA *iface, return ret; } -static struct DeviceInfo *SETUPDI_AllocateDeviceInfo(LPCWSTR instanceId, - BOOL phantom) +static struct DeviceInfo *SETUPDI_AllocateDeviceInfo(struct DeviceInfoSet *set, + LPCWSTR instanceId, BOOL phantom) { struct DeviceInfo *devInfo = HeapAlloc(GetProcessHeap(), 0, sizeof(struct DeviceInfo)); if (devInfo) { + devInfo->set = set; devInfo->instanceId = HeapAlloc(GetProcessHeap(), 0, (lstrlenW(instanceId) + 1) * sizeof(WCHAR)); if (devInfo->instanceId) @@ -470,7 +472,7 @@ static BOOL SETUPDI_AddDeviceToSet(struct DeviceInfoSet *set, SP_DEVINFO_DATA **dev) { BOOL ret = FALSE; - struct DeviceInfo *devInfo = SETUPDI_AllocateDeviceInfo(instanceId, + struct DeviceInfo *devInfo = SETUPDI_AllocateDeviceInfo(set, instanceId, phantom); TRACE("%p, %s, %d, %s, %d\n", set, debugstr_guid(guid), devInst, @@ -1364,6 +1366,11 @@ BOOL WINAPI SetupDiRegisterDeviceInfo( return FALSE; } devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; + if (devInfo->set != set) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } if (devInfo->phantom) { devInfo->phantom = FALSE; @@ -1504,6 +1511,11 @@ BOOL WINAPI SetupDiGetDeviceInstanceIdW( return FALSE; } devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; + if (devInfo->set != set) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } TRACE("instance ID: %s\n", debugstr_w(devInfo->instanceId)); if (DeviceInstanceIdSize < lstrlenW(devInfo->instanceId) + 1) { @@ -2115,6 +2127,7 @@ BOOL WINAPI SetupDiCreateDeviceInterfaceW( PSP_DEVICE_INTERFACE_DATA DeviceInterfaceData) { struct DeviceInfoSet *set = (struct DeviceInfoSet *)DeviceInfoSet; + struct DeviceInfo *devInfo; SP_DEVICE_INTERFACE_DATA *iface = NULL; BOOL ret; @@ -2138,6 +2151,12 @@ BOOL WINAPI SetupDiCreateDeviceInterfaceW( SetLastError(ERROR_INVALID_PARAMETER); return FALSE; } + devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; + if (devInfo->set != set) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } if (!InterfaceClassGuid) { SetLastError(ERROR_INVALID_USER_BUFFER);

1 0

Juan Lang : setupapi: Fix prototype of SetupDiCreateDeviceInterfaceRegKeyA/ W.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 4bfca01da0b9d36500e09c63b7e00ce6239b5688 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4bfca01da0b9d36500e09c63b… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:03:31 2007 -0700 setupapi: Fix prototype of SetupDiCreateDeviceInterfaceRegKeyA/W. --- include/setupapi.h | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/setupapi.h b/include/setupapi.h index e4a0fc3..d52790d 100644 --- a/include/setupapi.h +++ b/include/setupapi.h @@ -1456,8 +1456,8 @@ BOOL WINAPI SetupDiCreateDeviceInfoW(HDEVINFO, PCWSTR, CONST GUID*, PCWSTR, BOOL WINAPI SetupDiCreateDeviceInterfaceA(HDEVINFO, PSP_DEVINFO_DATA, const GUID *, PCSTR, DWORD, PSP_DEVICE_INTERFACE_DATA); BOOL WINAPI SetupDiCreateDeviceInterfaceW(HDEVINFO, PSP_DEVINFO_DATA, const GUID *, PCWSTR, DWORD, PSP_DEVICE_INTERFACE_DATA); #define SetupDiCreateDeviceInterface WINELIB_NAME_AW(SetupDiCreateDeviceInterface) -HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyA(HDEVINFO, PSP_DEVINFO_DATA, DWORD, REGSAM, HINF, PCSTR); -HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW(HDEVINFO, PSP_DEVINFO_DATA, DWORD, REGSAM, HINF, PCWSTR); +HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyA(HDEVINFO, PSP_DEVICE_INTERFACE_DATA, DWORD, REGSAM, HINF, PCSTR); +HKEY WINAPI SetupDiCreateDeviceInterfaceRegKeyW(HDEVINFO, PSP_DEVICE_INTERFACE_DATA, DWORD, REGSAM, HINF, PCWSTR); #define SetupDiCreateDeviceInterfaceRegKey WINELIB_NAME_AW(SetupDiCreateDeviceInterfaceRegKey) BOOL WINAPI SetupDiCreateDevRegKeyA(HDEVINFO, PSP_DEVINFO_DATA, DWORD, DWORD, DWORD, HINF, PCSTR); BOOL WINAPI SetupDiCreateDevRegKeyW(HDEVINFO, PSP_DEVINFO_DATA, DWORD, DWORD, DWORD, HINF, PCWSTR);

1 0

Juan Lang : setupapi: Create symbolic link value when interface is created.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: df3cb0630df098cab0aa7f422c40ff813e91bd93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df3cb0630df098cab0aa7f422… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 11:56:45 2007 -0700 setupapi: Create symbolic link value when interface is created. --- dlls/setupapi/devinst.c | 63 ++++++++++++++++++++++++++++++++-------- dlls/setupapi/tests/devinst.c | 2 - 2 files changed, 50 insertions(+), 15 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index c71d17f..08a78f3 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -129,6 +129,18 @@ struct DeviceInfo struct InterfaceInstances *interfaces; }; +static void SETUPDI_GuidToString(const GUID *guid, LPWSTR guidStr) +{ + static const WCHAR fmt[] = {'{','%','0','8','X','-','%','0','4','X','-', + '%','0','4','X','-','%','0','2','X','%','0','2','X','-','%','0','2', + 'X','%','0','2','X','%','0','2','X','%','0','2','X','%','0','2','X','%', + '0','2','X','}',0}; + + sprintfW(guidStr, fmt, guid->Data1, guid->Data2, guid->Data3, + guid->Data4[0], guid->Data4[1], guid->Data4[2], guid->Data4[3], + guid->Data4[4], guid->Data4[5], guid->Data4[6], guid->Data4[7]); +} + static void SETUPDI_FreeInterfaceInstances(struct InterfaceInstances *instances) { DWORD i; @@ -205,6 +217,41 @@ static BOOL SETUPDI_FindInterfaceInstance( return found; } +static LPWSTR SETUPDI_CreateSymbolicLinkPath(LPCWSTR instanceId, + const GUID *InterfaceClassGuid, LPCWSTR ReferenceString) +{ + static const WCHAR fmt[] = {'\\','\\','?','\\','%','s','#','%','s',0}; + WCHAR guidStr[39]; + DWORD len; + LPWSTR ret; + + SETUPDI_GuidToString(InterfaceClassGuid, guidStr); + /* omit length of format specifiers, but include NULL terminator: */ + len = lstrlenW(fmt) - 4 + 1; + len += lstrlenW(instanceId) + lstrlenW(guidStr); + if (ReferenceString) + { + /* space for a hash between string and reference string: */ + len += lstrlenW(ReferenceString) + 1; + } + ret = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR)); + if (ret) + { + int printed = sprintfW(ret, fmt, instanceId, guidStr); + LPWSTR ptr; + + /* replace '\\' with '#' after the "\\\\?\\" beginning */ + for (ptr = strchrW(ret + 4, '\\'); ptr; ptr = strchrW(ptr + 1, '\\')) + *ptr = '#'; + if (ReferenceString) + { + ret[printed - 1] = '\\'; + lstrcpyW(ret + printed, ReferenceString); + } + } + return ret; +} + /* Adds an interface with the given interface class and reference string to * the device, if it doesn't already exist in the device. If iface is not * NULL, returns a pointer to the newly added (or already existing) interface. @@ -285,7 +332,9 @@ static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, if (ifaceInfo) { ret = TRUE; - ifaceInfo->symbolicLink = NULL; + ifaceInfo->symbolicLink = SETUPDI_CreateSymbolicLinkPath( + devInfo->instanceId, InterfaceClassGuid, + ReferenceString); if (ReferenceString) { ifaceInfo->referenceString = @@ -425,18 +474,6 @@ static void SETUPDI_FreeDeviceInfo(struct DeviceInfo *devInfo) HeapFree(GetProcessHeap(), 0, devInfo); } -static void SETUPDI_GuidToString(const GUID *guid, LPWSTR guidStr) -{ - static const WCHAR fmt[] = {'{','%','0','8','X','-','%','0','4','X','-', - '%','0','4','X','-','%','0','2','X','%','0','2','X','-','%','0','2', - 'X','%','0','2','X','%','0','2','X','%','0','2','X','%','0','2','X','%', - '0','2','X','}',0}; - - sprintfW(guidStr, fmt, guid->Data1, guid->Data2, guid->Data3, - guid->Data4[0], guid->Data4[1], guid->Data4[2], guid->Data4[3], - guid->Data4[4], guid->Data4[5], guid->Data4[6], guid->Data4[7]); -} - /* Adds a device with GUID guid and identifer devInst to set. Allocates a * struct DeviceInfo, and points the returned device info's Reserved member * to it. "Phantom" devices are deleted from the registry when closed. diff --git a/dlls/setupapi/tests/devinst.c b/dlls/setupapi/tests/devinst.c index d8a434f..0b58e4e 100644 --- a/dlls/setupapi/tests/devinst.c +++ b/dlls/setupapi/tests/devinst.c @@ -482,7 +482,6 @@ static void testGetDeviceInterfaceDetail(void) SetLastError(0xdeadbeef); ret = pSetupDiGetDeviceInterfaceDetailA(set, &interfaceData, detail, size, &size, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_USER_BUFFER, "Expected ERROR_INVALID_USER_BUFFER, got %08x\n", GetLastError()); detail->cbSize = sizeof(SP_DEVICE_INTERFACE_DETAIL_DATA_A); @@ -490,7 +489,6 @@ static void testGetDeviceInterfaceDetail(void) size, &size, NULL); ok(ret, "SetupDiGetDeviceInterfaceDetailA failed: %d\n", GetLastError()); - todo_wine ok(!lstrcmpiA(path, detail->DevicePath), "Unexpected path %s\n", detail->DevicePath); HeapFree(GetProcessHeap(), 0, buf);

1 0

Juan Lang : setupapi: Convert device's interfaces to a standard list.
by Alexandre Julliard Sept. 24, 2007

Sept. 24, 2007

Module: wine Branch: master Commit: 44ac73effe3328da51fdafd3b062b27f5184f2ca URL: http://source.winehq.org/git/wine.git/?a=commit;h=44ac73effe3328da51fdafd3b… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Sep 21 17:04:09 2007 -0700 setupapi: Convert device's interfaces to a standard list. --- dlls/setupapi/devinst.c | 107 ++++++++++++++++++----------------------------- 1 files changed, 41 insertions(+), 66 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index 08a78f3..2236896 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -33,6 +33,7 @@ #include "winnls.h" #include "setupapi.h" #include "wine/debug.h" +#include "wine/list.h" #include "wine/unicode.h" #include "cfgmgr32.h" #include "initguid.h" @@ -116,17 +117,16 @@ struct InterfaceInstances DWORD cInstances; DWORD cInstancesAllocated; SP_DEVICE_INTERFACE_DATA *instances; + struct list entry; }; /* Pointed to by SP_DEVINFO_DATA's Reserved member */ struct DeviceInfo { - HKEY key; - BOOL phantom; - LPWSTR instanceId; - DWORD cInterfaces; - DWORD cInterfacesAllocated; - struct InterfaceInstances *interfaces; + HKEY key; + BOOL phantom; + LPWSTR instanceId; + struct list interfaces; }; static void SETUPDI_GuidToString(const GUID *guid, LPWSTR guidStr) @@ -157,27 +157,29 @@ static void SETUPDI_FreeInterfaceInstances(struct InterfaceInstances *instances) } /* Finds the interface with interface class InterfaceClassGuid in the device. - * Returns TRUE if found, and updates interfaceIndex to the index of the - * device's interfaces member where the given interface was found. + * Returns TRUE if found, and updates *interface to point to device's + * interfaces member where the given interface was found. * Returns FALSE if not found. */ static BOOL SETUPDI_FindInterface(const struct DeviceInfo *devInfo, - const GUID *InterfaceClassGuid, DWORD *interfaceIndex) + const GUID *InterfaceClassGuid, struct InterfaceInstances **interface) { BOOL found = FALSE; - DWORD i; + struct InterfaceInstances *iface; TRACE("%s\n", debugstr_guid(InterfaceClassGuid)); - for (i = 0; !found && i < devInfo->cInterfaces; i++) + LIST_FOR_EACH_ENTRY(iface, &devInfo->interfaces, struct InterfaceInstances, + entry) { - if (IsEqualGUID(&devInfo->interfaces[i].guid, InterfaceClassGuid)) + if (IsEqualGUID(&iface->guid, InterfaceClassGuid)) { - *interfaceIndex = i; + *interface = iface; found = TRUE; + break; } } - TRACE("returning %d (%d)\n", found, found ? *interfaceIndex : 0); + TRACE("returning %d (%p)\n", found, found ? *interface : NULL); return found; } @@ -261,42 +263,21 @@ static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, SP_DEVICE_INTERFACE_DATA **ifaceData) { BOOL newInterface = FALSE, ret; - DWORD interfaceIndex = 0; struct InterfaceInstances *iface = NULL; TRACE("%p %s %s %p\n", devInfo, debugstr_guid(InterfaceClassGuid), debugstr_w(ReferenceString), iface); - if (!(ret = SETUPDI_FindInterface(devInfo, InterfaceClassGuid, - &interfaceIndex))) + if (!(ret = SETUPDI_FindInterface(devInfo, InterfaceClassGuid, &iface))) { - if (!devInfo->cInterfacesAllocated) - { - devInfo->interfaces = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, - sizeof(struct InterfaceInstances)); - if (devInfo->interfaces) - { - iface = &devInfo->interfaces[devInfo->cInterfacesAllocated++]; - newInterface = TRUE; - } - } - else if (devInfo->cInterfaces == devInfo->cInterfacesAllocated) + iface = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, + sizeof(struct InterfaceInstances)); + if (iface) { - devInfo->interfaces = HeapReAlloc(GetProcessHeap(), - HEAP_ZERO_MEMORY, devInfo->interfaces, - (devInfo->cInterfacesAllocated + 1) * - sizeof(struct InterfaceInstances)); - if (devInfo->interfaces) - { - iface = &devInfo->interfaces[devInfo->cInterfacesAllocated++]; - newInterface = TRUE; - } + list_add_tail(&devInfo->interfaces, &iface->entry); + newInterface = TRUE; } - else - iface = &devInfo->interfaces[devInfo->cInterfaces]; } - else - iface = &devInfo->interfaces[interfaceIndex]; if (iface) { DWORD instanceIndex = 0; @@ -359,11 +340,8 @@ static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, instance->Flags = SPINT_ACTIVE; /* FIXME */ instance->Reserved = (ULONG_PTR)ifaceInfo; if (newInterface) - { memcpy(&iface->guid, InterfaceClassGuid, sizeof(GUID)); - devInfo->cInterfaces++; - } /* FIXME: now create this homeboy in the registry */ if (ifaceData) *ifaceData = instance; @@ -376,8 +354,7 @@ static BOOL SETUPDI_AddInterfaceInstance(struct DeviceInfo *devInfo, else { if (ifaceData) - *ifaceData = - &devInfo->interfaces[interfaceIndex].instances[instanceIndex]; + *ifaceData = &iface->instances[instanceIndex]; } } else @@ -436,8 +413,7 @@ static struct DeviceInfo *SETUPDI_AllocateDeviceInfo(LPCWSTR instanceId, (LPBYTE)&phantom, sizeof(phantom)); RegCloseKey(enumKey); } - devInfo->cInterfaces = devInfo->cInterfacesAllocated = 0; - devInfo->interfaces = NULL; + list_init(&devInfo->interfaces); } else { @@ -450,7 +426,7 @@ static struct DeviceInfo *SETUPDI_AllocateDeviceInfo(LPCWSTR instanceId, static void SETUPDI_FreeDeviceInfo(struct DeviceInfo *devInfo) { - DWORD i; + struct InterfaceInstances *iface, *next; if (devInfo->key != INVALID_HANDLE_VALUE) RegCloseKey(devInfo->key); @@ -468,9 +444,13 @@ static void SETUPDI_FreeDeviceInfo(struct DeviceInfo *devInfo) } } HeapFree(GetProcessHeap(), 0, devInfo->instanceId); - for (i = 0; i < devInfo->cInterfaces; i++) - SETUPDI_FreeInterfaceInstances(&devInfo->interfaces[i]); - HeapFree(GetProcessHeap(), 0, devInfo->interfaces); + LIST_FOR_EACH_ENTRY_SAFE(iface, next, &devInfo->interfaces, + struct InterfaceInstances, entry) + { + list_remove(&iface->entry); + SETUPDI_FreeInterfaceInstances(iface); + HeapFree(GetProcessHeap(), 0, iface); + } HeapFree(GetProcessHeap(), 0, devInfo); } @@ -2238,13 +2218,12 @@ BOOL WINAPI SetupDiEnumDeviceInterfaces( { struct DeviceInfo *devInfo = (struct DeviceInfo *)DeviceInfoData->Reserved; - DWORD i; + struct InterfaceInstances *iface; - if ((ret = SETUPDI_FindInterface(devInfo, InterfaceClassGuid, &i))) + if ((ret = SETUPDI_FindInterface(devInfo, InterfaceClassGuid, &iface))) { - if (MemberIndex < devInfo->interfaces[i].cInstances) - memcpy(DeviceInterfaceData, - &devInfo->interfaces[i].instances[MemberIndex], + if (MemberIndex < iface->cInstances) + memcpy(DeviceInterfaceData, &iface->instances[MemberIndex], sizeof(SP_DEVICE_INTERFACE_DATA)); else { @@ -2265,22 +2244,18 @@ BOOL WINAPI SetupDiEnumDeviceInterfaces( { struct DeviceInfo *devInfo = (struct DeviceInfo *)set->devices[i].Reserved; - DWORD interfaceIndex; + struct InterfaceInstances *iface; - if (SETUPDI_FindInterface(devInfo, InterfaceClassGuid, - &interfaceIndex)) + if (SETUPDI_FindInterface(devInfo, InterfaceClassGuid, &iface)) { - struct InterfaceInstances *interface = - &devInfo->interfaces[interfaceIndex]; - - if (cEnumerated + interface->cInstances < MemberIndex + 1) - cEnumerated += interface->cInstances; + if (cEnumerated + iface->cInstances < MemberIndex + 1) + cEnumerated += iface->cInstances; else { DWORD instanceIndex = MemberIndex - cEnumerated; memcpy(DeviceInterfaceData, - &interface->instances[instanceIndex], + &iface->instances[instanceIndex], sizeof(SP_DEVICE_INTERFACE_DATA)); cEnumerated += instanceIndex + 1; found = TRUE;

1 0