Ros-diffs

ros-diffs@reactos.org

81390 discussions

[hpoussin] 43261: WINLDR: Move i386 specific code to its own file
by hpoussin＠svn.reactos.org 03 Oct '09

03 Oct '09

Author: hpoussin Date: Sat Oct 3 14:36:52 2009 New Revision: 43261 URL: http://svn.reactos.org/svn/reactos?rev=43261&view=rev Log: WINLDR: Move i386 specific code to its own file Added: trunk/reactos/boot/freeldr/freeldr/windows/i386/ (with props) trunk/reactos/boot/freeldr/freeldr/windows/i386/ntsetup.c (with props) Modified: trunk/reactos/boot/freeldr/freeldr/freeldr_arch.rbuild trunk/reactos/boot/freeldr/freeldr/windows/winldr.c trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c Modified: trunk/reactos/boot/freeldr/freeldr/freeldr_arch.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/freel… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/freeldr_arch.rbuild [iso-8859-1] (original) +++ trunk/reactos/boot/freeldr/freeldr/freeldr_arch.rbuild [iso-8859-1] Sat Oct 3 14:36:52 2009 @@ -100,4 +100,12 @@ </if> </directory> + + <directory name="windows"> + <if property="ARCH" value="i386"> + <directory name="i386"> + <file>ntsetup.c</file> + </directory> + </if> + </directory> </module> Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Sat Oct 3 14:36:52 2009 @@ -1,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ ------------------------------------------------------------------------------ --- svn:ignore (added) +++ svn:ignore Sat Oct 3 14:36:52 2009 @@ -1,0 +1,7 @@ +GNUmakefile +*.vcproj +*.user +*.cbp +*.ncb +*.suo +*.sln Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: trunk/reactos/boot/freeldr/freeldr/windows/i386/ntsetup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/windows/i386/ntsetup.c (added) +++ trunk/reactos/boot/freeldr/freeldr/windows/i386/ntsetup.c [iso-8859-1] Sat Oct 3 14:36:52 2009 @@ -1,0 +1,83 @@ +/* + * PROJECT: EFI Windows Loader + * LICENSE: GPL - See COPYING in the top level directory + * FILE: freeldr/windows/i386/ntsetup.c + * PURPOSE: i386-specific setup for Windows boot + * PROGRAMMERS: Aleksey Bragin (aleksey(a)reactos.org) + */ + +/* INCLUDES ***************************************************************/ + +#include <freeldr.h> +#include <debug.h> + +// this is needed for new IDT filling +#if 0 +extern ULONG_PTR i386DivideByZero; +extern ULONG_PTR i386DebugException; +extern ULONG_PTR i386NMIException; +extern ULONG_PTR i386Breakpoint; +extern ULONG_PTR i386Overflow; +extern ULONG_PTR i386BoundException; +extern ULONG_PTR i386InvalidOpcode; +extern ULONG_PTR i386FPUNotAvailable; +extern ULONG_PTR i386DoubleFault; +extern ULONG_PTR i386CoprocessorSegment; +extern ULONG_PTR i386InvalidTSS; +extern ULONG_PTR i386SegmentNotPresent; +extern ULONG_PTR i386StackException; +extern ULONG_PTR i386GeneralProtectionFault; +extern ULONG_PTR i386PageFault; // exc 14 +extern ULONG_PTR i386CoprocessorError; // exc 16 +extern ULONG_PTR i386AlignmentCheck; // exc 17 +#endif + +/* FUNCTIONS **************************************************************/ + +// Last step before going virtual +void WinLdrSetupForNt(PLOADER_PARAMETER_BLOCK LoaderBlock, + PVOID *GdtIdt, + ULONG *PcrBasePage, + ULONG *TssBasePage) +{ + ULONG TssSize; + ULONG TssPages; + ULONG_PTR Pcr = 0; + ULONG_PTR Tss = 0; + ULONG BlockSize, NumPages; + + LoaderBlock->u.I386.CommonDataArea = NULL; // Force No ABIOS support + LoaderBlock->u.I386.MachineType = MACHINE_TYPE_ISA; + + /* Allocate 2 pages for PCR */ + Pcr = (ULONG_PTR)MmAllocateMemoryWithType(2 * MM_PAGE_SIZE, LoaderStartupPcrPage); + *PcrBasePage = Pcr >> MM_PAGE_SHIFT; + + if (Pcr == 0) + { + UiMessageBox("Can't allocate PCR\n"); + return; + } + + /* Allocate TSS */ + TssSize = (sizeof(KTSS) + MM_PAGE_SIZE) & ~(MM_PAGE_SIZE - 1); + TssPages = TssSize / MM_PAGE_SIZE; + + Tss = (ULONG_PTR)MmAllocateMemoryWithType(TssSize, LoaderMemoryData); + + *TssBasePage = Tss >> MM_PAGE_SHIFT; + + /* Allocate space for new GDT + IDT */ + BlockSize = NUM_GDT*sizeof(KGDTENTRY) + NUM_IDT*sizeof(KIDTENTRY);//FIXME: Use GDT/IDT limits here? + NumPages = (BlockSize + MM_PAGE_SIZE - 1) >> MM_PAGE_SHIFT; + *GdtIdt = (PKGDTENTRY)MmAllocateMemoryWithType(NumPages * MM_PAGE_SIZE, LoaderMemoryData); + + if (*GdtIdt == NULL) + { + UiMessageBox("Can't allocate pages for GDT+IDT!\n"); + return; + } + + /* Zero newly prepared GDT+IDT */ + RtlZeroMemory(*GdtIdt, NumPages << MM_PAGE_SHIFT); +} Propchange: trunk/reactos/boot/freeldr/freeldr/windows/i386/ntsetup.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/reactos/boot/freeldr/freeldr/windows/winldr.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/windows/winldr.c [iso-8859-1] (original) +++ trunk/reactos/boot/freeldr/freeldr/windows/winldr.c [iso-8859-1] Sat Oct 3 14:36:52 2009 @@ -24,6 +24,12 @@ #include <ndk/ldrtypes.h> #include <debug.h> +// TODO: Move to .h +void WinLdrSetupForNt(PLOADER_PARAMETER_BLOCK LoaderBlock, + PVOID *GdtIdt, + ULONG *PcrBasePage, + ULONG *TssBasePage); + //FIXME: Do a better way to retrieve Arc disk information extern ULONG reactos_disk_count; extern ARC_DISK_SIGNATURE reactos_arc_disk_info[]; @@ -91,8 +97,6 @@ ULONG i, PathSeparator; PLOADER_PARAMETER_EXTENSION Extension; - LoaderBlock->u.I386.CommonDataArea = NULL; // Force No ABIOS support - /* Construct SystemRoot and ArcBoot from SystemPath */ PathSeparator = strstr(BootPath, "\\") - BootPath; strncpy(ArcBoot, BootPath, PathSeparator); @@ -203,54 +207,6 @@ if (LoaderBlock->SetupLdrBlock) LoaderBlock->SetupLdrBlock = PaToVa(LoaderBlock->SetupLdrBlock); -} - -// Last step before going virtual -void WinLdrSetupForNt(PLOADER_PARAMETER_BLOCK LoaderBlock, - PVOID *GdtIdt, - ULONG *PcrBasePage, - ULONG *TssBasePage) -{ - ULONG TssSize; - ULONG TssPages; - ULONG_PTR Pcr = 0; - ULONG_PTR Tss = 0; - ULONG BlockSize, NumPages; - - LoaderBlock->u.I386.CommonDataArea = NULL; //CommonDataArea; - LoaderBlock->u.I386.MachineType = 0; // ntldr sets this to 0 - - /* Allocate 2 pages for PCR */ - Pcr = (ULONG_PTR)MmAllocateMemoryWithType(2 * MM_PAGE_SIZE, LoaderStartupPcrPage); - *PcrBasePage = Pcr >> MM_PAGE_SHIFT; - - if (Pcr == 0) - { - UiMessageBox("Can't allocate PCR\n"); - return; - } - - /* Allocate TSS */ - TssSize = (sizeof(KTSS) + MM_PAGE_SIZE) & ~(MM_PAGE_SIZE - 1); - TssPages = TssSize / MM_PAGE_SIZE; - - Tss = (ULONG_PTR)MmAllocateMemoryWithType(TssSize, LoaderMemoryData); - - *TssBasePage = Tss >> MM_PAGE_SHIFT; - - /* Allocate space for new GDT + IDT */ - BlockSize = NUM_GDT*sizeof(KGDTENTRY) + NUM_IDT*sizeof(KIDTENTRY);//FIXME: Use GDT/IDT limits here? - NumPages = (BlockSize + MM_PAGE_SIZE - 1) >> MM_PAGE_SHIFT; - *GdtIdt = (PKGDTENTRY)MmAllocateMemoryWithType(NumPages * MM_PAGE_SIZE, LoaderMemoryData); - - if (*GdtIdt == NULL) - { - UiMessageBox("Can't allocate pages for GDT+IDT!\n"); - return; - } - - /* Zero newly prepared GDT+IDT */ - RtlZeroMemory(*GdtIdt, NumPages << MM_PAGE_SHIFT); } BOOLEAN Modified: trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/boot/freeldr/freeldr/windo… ============================================================================== --- trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c [iso-8859-1] (original) +++ trunk/reactos/boot/freeldr/freeldr/windows/wlmemory.c [iso-8859-1] Sat Oct 3 14:36:52 2009 @@ -77,27 +77,6 @@ } GDTIDT; #pragma pack(4) -// this is needed for new IDT filling -#if 0 -extern ULONG_PTR i386DivideByZero; -extern ULONG_PTR i386DebugException; -extern ULONG_PTR i386NMIException; -extern ULONG_PTR i386Breakpoint; -extern ULONG_PTR i386Overflow; -extern ULONG_PTR i386BoundException; -extern ULONG_PTR i386InvalidOpcode; -extern ULONG_PTR i386FPUNotAvailable; -extern ULONG_PTR i386DoubleFault; -extern ULONG_PTR i386CoprocessorSegment; -extern ULONG_PTR i386InvalidTSS; -extern ULONG_PTR i386SegmentNotPresent; -extern ULONG_PTR i386StackException; -extern ULONG_PTR i386GeneralProtectionFault; -extern ULONG_PTR i386PageFault; // exc 14 -extern ULONG_PTR i386CoprocessorError; // exc 16 -extern ULONG_PTR i386AlignmentCheck; // exc 17 -#endif - /* GLOBALS ***************************************************************/ PHARDWARE_PTE PDE;

1 0

[dchapyshev] 43260: - Implement delete of the program information from the registry - Store settings information in HKEY_LOCAL_MACHINE instead HKEY_CURRENT_USER
by dchapyshev＠svn.reactos.org 03 Oct '09

03 Oct '09

Author: dchapyshev Date: Sat Oct 3 11:34:19 2009 New Revision: 43260 URL: http://svn.reactos.org/svn/reactos?rev=43260&view=rev Log: - Implement delete of the program information from the registry - Store settings information in HKEY_LOCAL_MACHINE instead HKEY_CURRENT_USER Modified: trunk/reactos/base/applications/rapps/installed.c trunk/reactos/base/applications/rapps/lang/bg-BG.rc trunk/reactos/base/applications/rapps/lang/de-DE.rc trunk/reactos/base/applications/rapps/lang/en-US.rc trunk/reactos/base/applications/rapps/lang/es-ES.rc trunk/reactos/base/applications/rapps/lang/ja-JP.rc trunk/reactos/base/applications/rapps/lang/no-NO.rc trunk/reactos/base/applications/rapps/lang/pl-PL.rc trunk/reactos/base/applications/rapps/lang/ru-RU.rc trunk/reactos/base/applications/rapps/lang/sk-SK.rc trunk/reactos/base/applications/rapps/lang/uk-UA.rc trunk/reactos/base/applications/rapps/rapps.h trunk/reactos/base/applications/rapps/resource.h trunk/reactos/base/applications/rapps/winmain.c Modified: trunk/reactos/base/applications/rapps/installed.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/in… ============================================================================== --- trunk/reactos/base/applications/rapps/installed.c [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/installed.c [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -136,17 +136,17 @@ ShowInstalledAppInfo(INT Index) { WCHAR szText[MAX_PATH], szInfo[MAX_PATH]; - HKEY hKey = (HKEY) ListViewGetlParam(Index); - - if (!hKey) return FALSE; - - GetApplicationString(hKey, L"DisplayName", szText); + PINSTALLED_INFO Info = ListViewGetlParam(Index); + + if (!Info || !Info->hSubKey) return FALSE; + + GetApplicationString(Info->hSubKey, L"DisplayName", szText); NewRichEditText(szText, CFE_BOLD); InsertRichEditText(L"\n", 0); #define GET_INFO(a, b, c, d) \ - if (GetApplicationString(hKey, a, szInfo)) \ + if (GetApplicationString(Info->hSubKey, a, szInfo)) \ { \ LoadStringW(hInst, b, szText, sizeof(szText) / sizeof(WCHAR)); \ InsertRichEditText(szText, c); \ @@ -175,32 +175,70 @@ } +VOID +RemoveAppFromRegistry(INT Index) +{ + PINSTALLED_INFO Info; + WCHAR szFullName[MAX_PATH] = L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall\\"; + WCHAR szMsgText[MAX_STR_LEN], szMsgTitle[MAX_STR_LEN]; + INT ItemIndex = SendMessage(hListView, LVM_GETNEXTITEM, -1, LVNI_FOCUSED); + + if (!IS_INSTALLED_ENUM(SelectedEnumType)) + return; + + Info = ListViewGetlParam(Index); + if (!Info || !Info->hSubKey || (ItemIndex == -1)) return; + + if (!LoadStringW(hInst, IDS_APP_REG_REMOVE, szMsgText, sizeof(szMsgText) / sizeof(WCHAR)) || + !LoadStringW(hInst, IDS_INFORMATION, szMsgTitle, sizeof(szMsgTitle) / sizeof(WCHAR))) + return; + + if (MessageBoxW(hMainWnd, szMsgText, szMsgTitle, MB_YESNO | MB_ICONQUESTION) == IDYES) + { + wcsncat(szFullName, Info->szKeyName, MAX_PATH - wcslen(szFullName)); + + if (RegDeleteKeyW(Info->hRootKey, szFullName) == ERROR_SUCCESS) + { + (VOID) ListView_DeleteItem(hListView, ItemIndex); + return; + } + + if (!LoadStringW(hInst, IDS_UNABLE_TO_REMOVE, szMsgText, sizeof(szMsgText) / sizeof(WCHAR))) + return; + + MessageBoxW(hMainWnd, szMsgText, NULL, MB_OK | MB_ICONERROR); + } +} + + BOOL EnumInstalledApplications(INT EnumType, BOOL IsUserKey, APPENUMPROC lpEnumProc) { DWORD dwSize = MAX_PATH, dwType, dwValue; BOOL bIsSystemComponent, bIsUpdate; - WCHAR pszName[MAX_PATH]; WCHAR pszParentKeyName[MAX_PATH]; WCHAR pszDisplayName[MAX_PATH]; - HKEY hKey, hSubKey; + INSTALLED_INFO Info; + HKEY hKey; LONG ItemIndex = 0; - if (RegOpenKeyW(IsUserKey ? HKEY_CURRENT_USER : HKEY_LOCAL_MACHINE, + Info.hRootKey = IsUserKey ? HKEY_CURRENT_USER : HKEY_LOCAL_MACHINE; + + if (RegOpenKeyW(Info.hRootKey, L"Software\\Microsoft\\Windows\\CurrentVersion\\Uninstall", &hKey) != ERROR_SUCCESS) { return FALSE; } - while (RegEnumKeyExW(hKey, ItemIndex, pszName, &dwSize, NULL, NULL, NULL, NULL) == ERROR_SUCCESS) - { - if (RegOpenKeyW(hKey, pszName, &hSubKey) == ERROR_SUCCESS) + while (RegEnumKeyExW(hKey, ItemIndex, Info.szKeyName, &dwSize, NULL, NULL, NULL, NULL) == ERROR_SUCCESS) + { + if (RegOpenKeyW(hKey, Info.szKeyName, &Info.hSubKey) == ERROR_SUCCESS) { dwType = REG_DWORD; dwSize = sizeof(DWORD); - if (RegQueryValueExW(hSubKey, + if (RegQueryValueExW(Info.hSubKey, L"SystemComponent", NULL, &dwType, @@ -216,7 +254,7 @@ dwType = REG_SZ; dwSize = MAX_PATH; - bIsUpdate = (RegQueryValueExW(hSubKey, + bIsUpdate = (RegQueryValueExW(Info.hSubKey, L"ParentKeyName", NULL, &dwType, @@ -224,7 +262,7 @@ &dwSize) == ERROR_SUCCESS); dwSize = MAX_PATH; - if (RegQueryValueExW(hSubKey, + if (RegQueryValueExW(Info.hSubKey, L"DisplayName", NULL, &dwType, @@ -240,7 +278,7 @@ ((EnumType == ENUM_APPLICATIONS) && (!bIsUpdate)) || /* Applications only */ ((EnumType == ENUM_UPDATES) && (bIsUpdate))) /* Updates only */ { - if (!lpEnumProc(ItemIndex, pszDisplayName, pszName, (LPARAM)hSubKey)) + if (!lpEnumProc(ItemIndex, pszDisplayName, Info)) break; } } Modified: trunk/reactos/base/applications/rapps/lang/bg-BG.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/bg-BG.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/bg-BG.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "&Ñëàãàíå", ID_INSTALL MENUITEM "&Ìàõàíå",ID_UNINSTALL MENUITEM "&Ïðîìÿíà", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "Î&ïðåñíÿâàíå", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Ñëàãàíå", ID_INSTALL MENUITEM "&Ìàõàíå", ID_UNINSTALL MENUITEM "&Ïðîìÿíà", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "Î&ïðåñíÿâàíå", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/de-DE.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/de-DE.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "&Installieren", ID_INSTALL MENUITEM "&Deinstallieren",ID_UNINSTALL MENUITEM "&Ändern", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Aktualisieren", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Installieren", ID_INSTALL MENUITEM "&Deinstallieren", ID_UNINSTALL MENUITEM "&Ändern", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Aktualisieren", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/en-US.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/en-US.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "&Install", ID_INSTALL MENUITEM "&Uninstall",ID_UNINSTALL MENUITEM "&Modify", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Refresh", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Install", ID_INSTALL MENUITEM "&Uninstall", ID_UNINSTALL MENUITEM "&Modify", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Refresh", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/es-ES.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/es-ES.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/es-ES.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "&Instalar", ID_INSTALL MENUITEM "&Desinstalar",ID_UNINSTALL MENUITEM "&Modificar", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Refrescar", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Instalar", ID_INSTALL MENUITEM "&Desinstalar", ID_UNINSTALL MENUITEM "&Modificar", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Refrescar", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/ja-JP.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/ja-JP.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/ja-JP.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "CXg[(&I)", ID_INSTALL MENUITEM "ACXg[(&U)",ID_UNINSTALL MENUITEM "ÏX(&M)", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "XV(&R)", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "CXg[(&I)", ID_INSTALL MENUITEM "ACXg[(&U)", ID_UNINSTALL MENUITEM "ÏX(&M)", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "XV(&R)", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "vOÌ_E[hÉgp·étH_ðIðµÄ¾³¢:" IDS_CHOOSE_FOLDER_ERROR "wè³ê½tH_Í¶ÝµÜ¹ñ!" IDS_USER_NOT_ADMIN """ReactOS AvP[V }l[W["" ðN®·éÉÍÇÒ ÀÅ éKvª èÜ·!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/no-NO.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/no-NO.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/no-NO.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -13,6 +13,8 @@ MENUITEM "&Installere", ID_INSTALL MENUITEM "&Avinstallere",ID_UNINSTALL MENUITEM "&Endre", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Oppdatere", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Installere", ID_INSTALL MENUITEM "&Avinstallere", ID_UNINSTALL MENUITEM "&Endre", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Oppdater", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/pl-PL.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/pl-PL.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/pl-PL.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -15,6 +15,8 @@ MENUITEM "&Instaluj", ID_INSTALL MENUITEM "&Odinstaluj",ID_UNINSTALL MENUITEM "&Modyfikuj", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "O&dwie¿", ID_REFRESH END @@ -41,6 +43,8 @@ MENUITEM "&Instaluj", ID_INSTALL MENUITEM "&odinstaluj", ID_UNINSTALL MENUITEM "&Modyfikuj", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Odwie¿", ID_REFRESH END @@ -185,4 +189,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/ru-RU.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/ru-RU.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/ru-RU.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -10,9 +10,11 @@ END POPUP "&Ïðîãðàììû" BEGIN - MENUITEM "&Óñòàíîâèòü", ID_INSTALL + MENUITEM "Ó&ñòàíîâèòü", ID_INSTALL MENUITEM "&Óäàëèòü", ID_UNINSTALL MENUITEM "&Èçìåíèòü", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "Ó&äàëèòü èç ðååñòðà", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Îáíîâèòü", ID_REFRESH END @@ -39,6 +41,8 @@ MENUITEM "&Óñòàíîâèòü", ID_INSTALL MENUITEM "&Óäàëèòü", ID_UNINSTALL MENUITEM "&Èçìåíèòü", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "Ó&äàëèòü èç ðååñòðà", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Îáíîâèòü", ID_REFRESH END @@ -183,4 +187,7 @@ IDS_CHOOSE_FOLDER_TEXT "Âûáåðèòå ïàïêó, êîòîðàÿ áóäåò èñïîëüçîâàòüñÿ äëÿ çàãðóçêè ïðîãðàìì:" IDS_CHOOSE_FOLDER_ERROR "Âû óêàçàëè íåñóùåñòâóþùóþ ïàïêó!" IDS_USER_NOT_ADMIN "Âû äîëæíû áûòü àäìèíèñòðàòîðîì äëÿ çàïóñêà ""Ìåíåäæåðà ïðèëîæåíèé ReactOS""!" + IDS_APP_REG_REMOVE "Âû äåéñòâèòåëüíî õîòèòå óäàëèòü äàííûå îá óñòàíîâëåííîé ïðîãðàììå èç ðååñòðà?" + IDS_INFORMATION "Èíôîðìàöèÿ" + IDS_UNABLE_TO_REMOVE "Íå óäàëîñü óäàëèòü äàííûå î ïðîãðàììå èç ðååñòðà!" END Modified: trunk/reactos/base/applications/rapps/lang/sk-SK.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/sk-SK.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/sk-SK.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -18,6 +18,8 @@ MENUITEM "&Intalova", ID_INSTALL MENUITEM "O&dintalova", ID_UNINSTALL MENUITEM "&Zmeni", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Obnovi", ID_REFRESH END @@ -44,6 +46,8 @@ MENUITEM "&Intalova", ID_INSTALL MENUITEM "O&dintalova", ID_UNINSTALL MENUITEM "&Zmeni", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Obnovi", ID_REFRESH END @@ -188,4 +192,7 @@ IDS_CHOOSE_FOLDER_TEXT "Vyberte prieèinok, ktorý sa pouije pre sahovanie programov:" IDS_CHOOSE_FOLDER_ERROR "Zvolili ste si neexistujúci prieèinok!" IDS_USER_NOT_ADMIN "Mali by ste by administrátor pre spustenie ""Manaéra aplikácií systému ReactOS""!" + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" END Modified: trunk/reactos/base/applications/rapps/lang/uk-UA.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/uk-UA.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/uk-UA.rc [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -22,6 +22,8 @@ MENUITEM "&Âèäàëèòè",ID_UNINSTALL MENUITEM "&Çì³íèòè", ID_MODIFY MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE + MENUITEM SEPARATOR MENUITEM "&Îíîâèòè", ID_REFRESH END POPUP "Äîâ³äêà" @@ -47,6 +49,8 @@ MENUITEM "&Âñòàíîâèòè", ID_INSTALL MENUITEM "&Âèäàëèòè", ID_UNINSTALL MENUITEM "&Çì³íèòè", ID_MODIFY + MENUITEM SEPARATOR + MENUITEM "&Remove from Registry", ID_REGREMOVE MENUITEM SEPARATOR MENUITEM "&Îíîâèòè", ID_REFRESH END @@ -191,4 +195,7 @@ IDS_CHOOSE_FOLDER_TEXT "Choose a folder which will be used for downloading of programs:" IDS_CHOOSE_FOLDER_ERROR "You have specified a nonexistent folder!" IDS_USER_NOT_ADMIN "You should be administrator for start ""ReactOS Applications Manager""!" -END + IDS_APP_REG_REMOVE "Are you sure you want to delete the data on the installed program from the registry?" + IDS_INFORMATION "Information" + IDS_UNABLE_TO_REMOVE "Unable to remove data on the program from the registry!" +END Modified: trunk/reactos/base/applications/rapps/rapps.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/ra… ============================================================================== --- trunk/reactos/base/applications/rapps/rapps.h [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/rapps.h [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -71,6 +71,14 @@ typedef struct { + HKEY hRootKey; + HKEY hSubKey; + WCHAR szKeyName[MAX_PATH]; + +} INSTALLED_INFO, *PINSTALLED_INFO; + +typedef struct +{ BOOL bSaveWndPos; BOOL bUpdateAtStart; BOOL bLogEnabled; @@ -95,12 +103,13 @@ BOOL InstallApplication(INT Index); /* installed.c */ -typedef BOOL (CALLBACK *APPENUMPROC)(INT ItemIndex, LPWSTR lpName, LPWSTR lpKeyName, LPARAM lParam); +typedef BOOL (CALLBACK *APPENUMPROC)(INT ItemIndex, LPWSTR lpName, INSTALLED_INFO Info); BOOL EnumInstalledApplications(INT EnumType, BOOL IsUserKey, APPENUMPROC lpEnumProc); BOOL GetApplicationString(HKEY hKey, LPWSTR lpKeyName, LPWSTR lpString); BOOL ShowInstalledAppInfo(INT Index); BOOL UninstallApplication(INT Index, BOOL bModify); BOOL IsInstalledApplication(LPWSTR lpRegName, BOOL IsUserKey); +VOID RemoveAppFromRegistry(INT Index); /* winmain.c */ extern HWND hMainWnd; Modified: trunk/reactos/base/applications/rapps/resource.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/re… ============================================================================== --- trunk/reactos/base/applications/rapps/resource.h [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/resource.h [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -69,6 +69,7 @@ #define ID_COPY_LINK 557 #define ID_SETTINGS 558 #define ID_REFRESH 559 +#define ID_REGREMOVE 560 /* Strings */ #define IDS_APPTITLE 100 @@ -87,6 +88,9 @@ #define IDS_CHOOSE_FOLDER_TEXT 113 #define IDS_CHOOSE_FOLDER_ERROR 114 #define IDS_USER_NOT_ADMIN 115 +#define IDS_APP_REG_REMOVE 116 +#define IDS_INFORMATION 117 +#define IDS_UNABLE_TO_REMOVE 118 /* Tooltips */ #define IDS_TOOLTIP_INSTALL 200 Modified: trunk/reactos/base/applications/rapps/winmain.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/wi… ============================================================================== --- trunk/reactos/base/applications/rapps/winmain.c [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/winmain.c [iso-8859-1] Sat Oct 3 11:34:19 2009 @@ -15,6 +15,7 @@ INT SelectedEnumType = ENUM_ALL_COMPONENTS; SETTINGS_INFO SettingsInfo; + VOID FillDafaultSettings(PSETTINGS_INFO pSettingsInfo) { @@ -37,7 +38,7 @@ HKEY hKey; DWORD dwSize; - if (RegOpenKeyExW(HKEY_CURRENT_USER, L"Software\\ReactOS\\rapps", 0, KEY_READ, &hKey) == ERROR_SUCCESS) + if (RegOpenKeyExW(HKEY_LOCAL_MACHINE, L"Software\\ReactOS\\rapps", 0, KEY_READ, &hKey) == ERROR_SUCCESS) { dwSize = sizeof(SETTINGS_INFO); if (RegQueryValueExW(hKey, L"Settings", NULL, NULL, (LPBYTE)&SettingsInfo, &dwSize) == ERROR_SUCCESS) @@ -70,7 +71,7 @@ SettingsInfo.Maximized = (IsZoomed(hwnd) || (wp.flags & WPF_RESTORETOMAXIMIZED)); } - if (RegCreateKeyExW(HKEY_CURRENT_USER, L"Software\\ReactOS\\rapps", 0, NULL, + if (RegCreateKeyExW(HKEY_LOCAL_MACHINE, L"Software\\ReactOS\\rapps", 0, NULL, REG_OPTION_NON_VOLATILE, KEY_WRITE, NULL, &hKey, NULL) == ERROR_SUCCESS) { RegSetValueEx(hKey, L"Settings", 0, REG_BINARY, (LPBYTE)&SettingsInfo, sizeof(SETTINGS_INFO)); @@ -82,31 +83,40 @@ FreeInstalledAppList(VOID) { INT Count = ListView_GetItemCount(hListView) - 1; - HKEY hKey; + PINSTALLED_INFO Info; while (Count >= 0) { - hKey = ListViewGetlParam(Count); - if (hKey) - RegCloseKey(hKey); + Info = ListViewGetlParam(Count); + if (Info) + { + RegCloseKey(Info->hSubKey); + HeapFree(GetProcessHeap(), 0, Info); + } Count--; } } BOOL CALLBACK -EnumInstalledAppProc(INT ItemIndex, LPWSTR lpName, LPWSTR lpKeyName, LPARAM lParam) -{ +EnumInstalledAppProc(INT ItemIndex, LPWSTR lpName, INSTALLED_INFO Info) +{ + PINSTALLED_INFO ItemInfo; WCHAR szText[MAX_PATH]; INT Index; - Index = ListViewAddItem(ItemIndex, 0, lpName, lParam); + ItemInfo = HeapAlloc(GetProcessHeap(), 0, sizeof(INSTALLED_INFO)); + if (!ItemInfo) return FALSE; + + *ItemInfo = Info; + + Index = ListViewAddItem(ItemIndex, 0, lpName, (LPARAM)ItemInfo); /* Get version info */ - GetApplicationString((HKEY)lParam, L"DisplayVersion", szText); + GetApplicationString((HKEY)ItemInfo->hSubKey, L"DisplayVersion", szText); ListView_SetItemText(hListView, Index, 1, szText); /* Get comments */ - GetApplicationString((HKEY)lParam, L"Comments", szText); + GetApplicationString((HKEY)ItemInfo->hSubKey, L"Comments", szText); ListView_SetItemText(hListView, Index, 2, szText); return TRUE; @@ -399,6 +409,10 @@ UpdateApplicationsList(-1); break; + case ID_REGREMOVE: + RemoveAppFromRegistry(-1); + break; + case ID_REFRESH: UpdateApplicationsList(-1); break;

1 0

[jimtabor] 43259: - Prevent kernel bug check in win32k when calling a hook proc when thread is in cleanup. - This is not a hack and not a fix. Please retest related bug report applications. Tested Firefox 3.5 and the new putty. Need new debug print outs. ATM Ff 3.5 send out WH_JOURNALRECORD hook calls when the thread is in cleanup. Look like this: Thread is in cleanup and trying to call hook 0 - Reference bug 4298.
by jimtabor＠svn.reactos.org 03 Oct '09

03 Oct '09

Author: jimtabor Date: Sat Oct 3 04:30:47 2009 New Revision: 43259 URL: http://svn.reactos.org/svn/reactos?rev=43259&view=rev Log: - Prevent kernel bug check in win32k when calling a hook proc when thread is in cleanup. - This is not a hack and not a fix. Please retest related bug report applications. Tested Firefox 3.5 and the new putty. Need new debug print outs. ATM Ff 3.5 send out WH_JOURNALRECORD hook calls when the thread is in cleanup. Look like this: Thread is in cleanup and trying to call hook 0 - Reference bug 4298. Modified: trunk/reactos/subsystems/win32/win32k/main/dllmain.c trunk/reactos/subsystems/win32/win32k/ntuser/callback.c trunk/reactos/subsystems/win32/win32k/ntuser/window.c Modified: trunk/reactos/subsystems/win32/win32k/main/dllmain.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ma… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/main/dllmain.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/main/dllmain.c [iso-8859-1] Sat Oct 3 04:30:47 2009 @@ -16,8 +16,7 @@ * along with this program; if not, write to the Free Software * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. */ -/* $Id$ - * +/* * Entry Point for win32k.sys */ @@ -285,6 +284,7 @@ DPRINT("Destroying W32 thread TID:%d at IRQ level: %lu\n", Thread->Cid.UniqueThread, KeGetCurrentIrql()); Win32Thread->IsExiting = TRUE; + Win32Thread->TIF_flags |= TIF_INCLEANUP; HOOK_DestroyThreadHooks(Thread); UnregisterThreadHotKeys(Thread); /* what if this co_ func crash in umode? what will clean us up then? */ Modified: trunk/reactos/subsystems/win32/win32k/ntuser/callback.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/callback.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/callback.c [iso-8859-1] Sat Oct 3 04:30:47 2009 @@ -326,6 +326,14 @@ UNICODE_STRING ClassName; PANSI_STRING asWindowName; PANSI_STRING asClassName; + PTHREADINFO pti; + + pti = PsGetCurrentThreadWin32Thread(); + if (pti->TIF_flags & TIF_INCLEANUP) + { + DPRINT1("Thread is in cleanup and trying to call hook %d\n", Code); + return 0; + } ArgumentLength = sizeof(HOOKPROC_CALLBACK_ARGUMENTS) - sizeof(WCHAR) + ModuleName->Length; Modified: trunk/reactos/subsystems/win32/win32k/ntuser/window.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/window.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/window.c [iso-8859-1] Sat Oct 3 04:30:47 2009 @@ -425,6 +425,7 @@ co_IntSendMessage(Window->hSelf, WM_NCDESTROY, 0, 0); } MsqRemoveTimersWindow(ThreadData->MessageQueue, Window->hSelf); + HOOK_DestroyThreadHooks(ThreadData->pEThread); // This is needed here too! /* flush the message queue */ MsqRemoveWindowMessagesFromQueue(Window);

1 0

[janderwald] 43258: - Add KSPROPSETID_Audio guid - Allocate data format after successful initialization of IPinWaveCyclic - Store Miniport Adapter in subdevice descriptor - Implement GUID_NULL handler for topology nodes - Implement setting / retrieving properties for topology nodes
by janderwald＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: janderwald Date: Fri Oct 2 23:55:53 2009 New Revision: 43258 URL: http://svn.reactos.org/svn/reactos?rev=43258&view=rev Log: - Add KSPROPSETID_Audio guid - Allocate data format after successful initialization of IPinWaveCyclic - Store Miniport Adapter in subdevice descriptor - Implement GUID_NULL handler for topology nodes - Implement setting / retrieving properties for topology nodes Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/guids.cpp trunk/reactos/drivers/wdm/audio/backpln/portcls/interfaces.hpp trunk/reactos/drivers/wdm/audio/backpln/portcls/pin_wavecyclic.cpp trunk/reactos/drivers/wdm/audio/backpln/portcls/port_topology.cpp trunk/reactos/drivers/wdm/audio/backpln/portcls/port_wavecyclic.cpp trunk/reactos/drivers/wdm/audio/backpln/portcls/undoc.cpp Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/guids.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/guids.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/guids.cpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -82,6 +82,7 @@ const GUID IID_IAllocatorMXF = {0xa5f0d62cL, 0xb30f, 0x11d2, {0xb7, 0xa3, 0x00, 0x60, 0x08, 0x33, 0x16, 0xc1}}; +const GUID KSPROPSETID_Audio = {0x45FFAAA0L, 0x6E1B, 0x11D0, {0xBC, 0xF2, 0x44, 0x45, 0x53, 0x54, 0x00, 0x00}}; /// /// undocumented guids Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/interfaces.hpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/interfaces.hpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/interfaces.hpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -161,6 +161,8 @@ DEFINE_ABSTRACT_IRPTARGET() }; +typedef IIrpTarget *PIRPTARGET; + /***************************************************************************** * ISubdevice ***************************************************************************** @@ -212,6 +214,7 @@ LIST_ENTRY SymbolicLinkList; LIST_ENTRY PhysicalConnectionList; UNICODE_STRING RefString; + PUNKNOWN UnknownMiniport; }SUBDEVICE_DESCRIPTOR, *PSUBDEVICE_DESCRIPTOR; #undef INTERFACE Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/pin_wavecyclic.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/pin_wavecyclic.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/pin_wavecyclic.cpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -913,12 +913,6 @@ DPRINT("CPortPinWaveCyclic::Init entered Size %u\n", DataFormat->FormatSize); - m_Format = (PKSDATAFORMAT)ExAllocatePoolWithTag(NonPagedPool, DataFormat->FormatSize, TAG_PORTCLASS); - if (!m_Format) - return STATUS_INSUFFICIENT_RESOURCES; - - RtlMoveMemory(m_Format, DataFormat, DataFormat->FormatSize); - Status = NewIrpQueue(&m_IrpQueue); if (!NT_SUCCESS(Status)) return Status; @@ -943,7 +937,7 @@ NonPagedPool, ConnectDetails->PinId, Capture, - m_Format, + DataFormat, &m_DmaChannel, &m_ServiceGroup); #if 0 @@ -999,6 +993,13 @@ return Status; } + m_Format = (PKSDATAFORMAT)ExAllocatePoolWithTag(NonPagedPool, DataFormat->FormatSize, TAG_PORTCLASS); + if (!m_Format) + return STATUS_INSUFFICIENT_RESOURCES; + + RtlMoveMemory(m_Format, DataFormat, DataFormat->FormatSize); + + Port->AddRef(); Filter->AddRef(); Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/port_topology.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/port_topology.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/port_topology.cpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -273,6 +273,12 @@ DPRINT("IPortTopology_fnInit success\n"); + if (NT_SUCCESS(Status)) + { + // store for node property requests + m_SubDeviceDescriptor->UnknownMiniport = UnknownMiniport; + } + return STATUS_SUCCESS; } @@ -455,7 +461,7 @@ DPRINT("CreatePinWorkerRoutine called\n"); // create the pin Status = WorkerContext->Filter->NewIrpTarget(&Pin, - NULL, + KSSTRING_Pin, NULL, NonPagedPool, DeviceObject, Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/port_wavecyclic.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/port_wavecyclic.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/port_wavecyclic.cpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -298,6 +298,9 @@ return Status; } + // store for node property requests + m_SubDeviceDescriptor->UnknownMiniport = UnknownMiniport; + // check if it supports IPinCount interface Status = UnknownMiniport->QueryInterface(IID_IPinCount, (PVOID*)&PinCount); if (NT_SUCCESS(Status)) Modified: trunk/reactos/drivers/wdm/audio/backpln/portcls/undoc.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/backpln/… ============================================================================== --- trunk/reactos/drivers/wdm/audio/backpln/portcls/undoc.cpp [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/backpln/portcls/undoc.cpp [iso-8859-1] Fri Oct 2 23:55:53 2009 @@ -51,8 +51,133 @@ IN PKSPROPERTY_SET PropertySet, IN PSUBDEVICE_DESCRIPTOR SubDeviceDescriptor) { + NTSTATUS Status; + PIO_STACK_LOCATION IoStack; + PKSP_NODE Property; + PPCNODE_DESCRIPTOR Node; + PPCPROPERTY_ITEM PropertyItem; + ULONG Index; + LPGUID Buffer; + //PULONG Flags; + PPCPROPERTY_REQUEST PropertyRequest; + KSPROPERTY_ITEM_IRP_STORAGE(Irp) = (PKSPROPERTY_ITEM)SubDeviceDescriptor; - return KsPropertyHandler(Irp, PropertySetCount, PropertySet); + + /* try first KsPropertyHandler */ + Status = KsPropertyHandler(Irp, PropertySetCount, PropertySet); + + // get current irp stack location + IoStack = IoGetCurrentIrpStackLocation(Irp); + + // access property + Property = (PKSP_NODE)IoStack->Parameters.DeviceIoControl.Type3InputBuffer; + + // check if this a GUID_NULL request + if (Status == STATUS_NOT_FOUND) + { + if (IoStack->Parameters.DeviceIoControl.InputBufferLength < sizeof(KSP_NODE)) + return Status; + + // check if its a request for a topology node + if (IsEqualGUIDAligned(Property->Property.Set, GUID_NULL) && Property->Property.Id == 0 && Property->Property.Flags == (KSPROPERTY_TYPE_SETSUPPORT | KSPROPERTY_TYPE_TOPOLOGY)) + { + if (Property->NodeId >= SubDeviceDescriptor->DeviceDescriptor->NodeCount) + { + // request is out of bounds + Irp->IoStatus.Information = 0; + return STATUS_INVALID_PARAMETER; + } + + Node = (PPCNODE_DESCRIPTOR)((ULONG_PTR)SubDeviceDescriptor->DeviceDescriptor->Nodes + (Property->NodeId * SubDeviceDescriptor->DeviceDescriptor->NodeSize)); + + PC_ASSERT(Node->AutomationTable); + PC_ASSERT(Node->AutomationTable->PropertyCount); + PC_ASSERT(Node->AutomationTable->PropertyItemSize); + + Irp->IoStatus.Information = sizeof(GUID) * Node->AutomationTable->PropertyCount; + if (IoStack->Parameters.DeviceIoControl.OutputBufferLength < sizeof(GUID) * Node->AutomationTable->PropertyCount) + { + // buffer too small + return STATUS_MORE_ENTRIES; + } + + PropertyItem = (PCPROPERTY_ITEM*)Node->AutomationTable->Properties; + Buffer = (LPGUID)Irp->UserBuffer; + + for(Index = 0; Index < Node->AutomationTable->PropertyCount; Index++) + { + RtlMoveMemory(Buffer, PropertyItem->Set, sizeof(GUID)); + Buffer++; + + PropertyItem = (PPCPROPERTY_ITEM)((ULONG_PTR)PropertyItem + Node->AutomationTable->PropertyItemSize); + } + return STATUS_SUCCESS; + } + else /*if (Property->Property.Flags == (KSPROPERTY_TYPE_BASICSUPPORT | KSPROPERTY_TYPE_TOPOLOGY) || + Property->Property.Flags == (KSPROPERTY_TYPE_GET | KSPROPERTY_TYPE_TOPOLOGY) || + Property->Property.Flags == (KSPROPERTY_TYPE_SET | KSPROPERTY_TYPE_TOPOLOGY)) */ + { + if (Property->NodeId >= SubDeviceDescriptor->DeviceDescriptor->NodeCount) + { + // request is out of bounds + Irp->IoStatus.Information = 0; + return STATUS_INVALID_PARAMETER; + } + + Node = (PPCNODE_DESCRIPTOR)((ULONG_PTR)SubDeviceDescriptor->DeviceDescriptor->Nodes + (Property->NodeId * SubDeviceDescriptor->DeviceDescriptor->NodeSize)); + + PC_ASSERT(Node->AutomationTable); + PC_ASSERT(Node->AutomationTable->PropertyCount); + PC_ASSERT(Node->AutomationTable->PropertyItemSize); + + PropertyItem = (PCPROPERTY_ITEM*)Node->AutomationTable->Properties; + //Flags = (PULONG)Irp->UserBuffer; + + for(Index = 0; Index < Node->AutomationTable->PropertyCount; Index++) + { + if (IsEqualGUIDAligned(*PropertyItem->Set, Property->Property.Set) && PropertyItem->Id == Property->Property.Id) + { + PropertyRequest = (PPCPROPERTY_REQUEST)AllocateItem(NonPagedPool, sizeof(PCPROPERTY_REQUEST), TAG_PORTCLASS); + if (!PropertyRequest) + return STATUS_INSUFFICIENT_RESOURCES; + + PC_ASSERT(SubDeviceDescriptor->UnknownMiniport); + PropertyRequest->MajorTarget = SubDeviceDescriptor->UnknownMiniport; + //PropertyRequest->MinorTarget = (PUNKNOWN)0xABADCAFE; + PropertyRequest->Irp = Irp; + PropertyRequest->Node = Property->NodeId; + PropertyRequest->PropertyItem = PropertyItem; + PropertyRequest->Verb = Property->Property.Flags; + PropertyRequest->InstanceSize = IoStack->Parameters.DeviceIoControl.InputBufferLength - sizeof(KSNODEPROPERTY); + PropertyRequest->Instance = (PVOID)((ULONG_PTR)IoStack->Parameters.DeviceIoControl.Type3InputBuffer + sizeof(KSNODEPROPERTY)); + PropertyRequest->ValueSize = IoStack->Parameters.DeviceIoControl.OutputBufferLength; + PropertyRequest->Value = Irp->UserBuffer; + + Status = PropertyItem->Handler(PropertyRequest); + + if (Status != STATUS_PENDING) + { + //DPRINT1("Status %x ValueSize %u + + Irp->IoStatus.Information = PropertyRequest->ValueSize; + ExFreePool(PropertyRequest); + } + return Status; + } + PropertyItem = (PPCPROPERTY_ITEM)((ULONG_PTR)PropertyItem + Node->AutomationTable->PropertyItemSize); + } + } +#if 0 + else + { + UNICODE_STRING GuidString; + RtlStringFromGUID(Property->Property.Set, &GuidString); + DPRINT1("Id %u Flags %x Set %S\n", Property->Property.Id, Property->Property.Flags, GuidString.Buffer); + DbgBreakPoint(); + } +#endif + } + return Status; } VOID @@ -102,6 +227,39 @@ { UNIMPLEMENTED; return STATUS_NOT_IMPLEMENTED; +} + +VOID +DumpFilterDescriptor( + IN PPCFILTER_DESCRIPTOR FilterDescription) +{ + ULONG Index; + PPCPROPERTY_ITEM PropertyItem; + UNICODE_STRING GuidString; + + DPRINT1("======================\n"); + DPRINT1("Descriptor Automation Table%p\n",FilterDescription->AutomationTable); + + if (FilterDescription->AutomationTable) + { + DPRINT1("FilterPropertiesCount %u FilterPropertySize %u Expected %u\n", FilterDescription->AutomationTable->PropertyCount, FilterDescription->AutomationTable->PropertyItemSize, sizeof(PCPROPERTY_ITEM)); + if (FilterDescription->AutomationTable->PropertyCount) + { + PropertyItem = (PPCPROPERTY_ITEM)FilterDescription->AutomationTable->Properties; + + for(Index = 0; Index < FilterDescription->AutomationTable->PropertyCount; Index++) + { + RtlStringFromGUID(*PropertyItem->Set, &GuidString); + DPRINT1("Index %u GUID %S Id %u Flags %x\n", Index, GuidString.Buffer, PropertyItem->Id, PropertyItem->Flags); + + PropertyItem = (PPCPROPERTY_ITEM)((ULONG_PTR)PropertyItem + FilterDescription->AutomationTable->PropertyItemSize); + } + } + } + + + DPRINT1("======================\n"); + DbgBreakPoint(); } NTSTATUS @@ -147,6 +305,9 @@ { /// FIXME /// handle driver properties + + //DumpFilterDescriptor(FilterDescription); + Descriptor->FilterPropertySet = (PKSPROPERTY_SET)AllocateItem(NonPagedPool, sizeof(KSPROPERTY_SET) * FilterPropertiesCount, TAG_PORTCLASS); if (! Descriptor->FilterPropertySet) goto cleanup;

1 0

[fireball] 43257: [fastfat_new] - Don't leak VCB lock, prevents hang when trying to open another file.
by fireball＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: fireball Date: Fri Oct 2 19:07:37 2009 New Revision: 43257 URL: http://svn.reactos.org/svn/reactos?rev=43257&view=rev Log: [fastfat_new] - Don't leak VCB lock, prevents hang when trying to open another file. Modified: trunk/reactos/drivers/filesystems/fastfat_new/create.c Modified: trunk/reactos/drivers/filesystems/fastfat_new/create.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/create.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/create.c [iso-8859-1] Fri Oct 2 19:07:37 2009 @@ -527,6 +527,9 @@ since we're opening files here */ if (EndBackslash) { + /* Unlock VCB */ + FatReleaseVcb(IrpContext, Vcb); + /* Complete the request */ Iosb.Status = STATUS_OBJECT_NAME_INVALID; FatCompleteRequest(IrpContext, Irp, Iosb.Status); @@ -552,6 +555,9 @@ Irp->IoStatus.Information = Iosb.Information; } + /* Unlock VCB */ + FatReleaseVcb(IrpContext, Vcb); + /* Complete the request */ FatCompleteRequest(IrpContext, Irp, Iosb.Status); @@ -564,7 +570,6 @@ { PFAT_IRP_CONTEXT IrpContext; NTSTATUS Status; - //PVOLUME_DEVICE_OBJECT VolumeDO = (PVOLUME_DEVICE_OBJECT)DeviceObject; DPRINT1("FatCreate()\n");

1 0

[fireball] 43256: [fastfat_new] - Fail with an error if file can't be opened. - Implement FatCreateDcb, actually create DCBs for all parsed directories in the path. - Set file size in an advanced FSRTL header for a file. - Implement a small helper function for setting full names in FCB/DCB like it's done in the reference driver.
by fireball＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: fireball Date: Fri Oct 2 15:59:51 2009 New Revision: 43256 URL: http://svn.reactos.org/svn/reactos?rev=43256&view=rev Log: [fastfat_new] - Fail with an error if file can't be opened. - Implement FatCreateDcb, actually create DCBs for all parsed directories in the path. - Set file size in an advanced FSRTL header for a file. - Implement a small helper function for setting full names in FCB/DCB like it's done in the reference driver. Modified: trunk/reactos/drivers/filesystems/fastfat_new/create.c trunk/reactos/drivers/filesystems/fastfat_new/dir.c trunk/reactos/drivers/filesystems/fastfat_new/fastfat.h trunk/reactos/drivers/filesystems/fastfat_new/fatstruc.h trunk/reactos/drivers/filesystems/fastfat_new/fcb.c Modified: trunk/reactos/drivers/filesystems/fastfat_new/create.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/create.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/create.c [iso-8859-1] Fri Oct 2 15:59:51 2009 @@ -63,6 +63,7 @@ CHAR AnsiNameBuf[512]; PFCB Fcb; NTSTATUS Status; + FF_FILE *FileHandle; /* Check for create file option and fail */ if (CreateDisposition == FILE_CREATE) @@ -72,9 +73,6 @@ } // TODO: Check more params - - /* Create a new FCB for this file */ - Fcb = FatCreateFcb(IrpContext, Vcb, ParentDcb); /* Convert the name to ANSI */ AnsiName.Buffer = AnsiNameBuf; @@ -88,7 +86,16 @@ } /* Open the file with FullFAT */ - Fcb->FatHandle = FF_Open(Vcb->Ioman, AnsiName.Buffer, FF_MODE_READ, NULL); + FileHandle = FF_Open(Vcb->Ioman, AnsiName.Buffer, FF_MODE_READ, NULL); + + if (!FileHandle) + { + Iosb.Status = STATUS_OBJECT_NAME_NOT_FOUND; // FIXME: A shortcut for now + return Iosb; + } + + /* Create a new FCB for this file */ + Fcb = FatCreateFcb(IrpContext, Vcb, ParentDcb, FileHandle); // TODO: Check if overwrite is needed @@ -505,7 +512,13 @@ /* Break if came to the end */ if (!RemainingPart.Length) break; - // TODO: Create a DCB for this entry + /* Create a DCB for this entry */ + ParentDcb = FatCreateDcb(IrpContext, + Vcb, + ParentDcb); + + /* Set its name */ + FatSetFullNameInFcb(ParentDcb, &FirstName); } // TODO: Try to open directory Modified: trunk/reactos/drivers/filesystems/fastfat_new/dir.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/dir.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/dir.c [iso-8859-1] Fri Oct 2 15:59:51 2009 @@ -123,5 +123,46 @@ //FatCheckFreeDirentBitmap( IrpContext, Dcb ); } +PFCB +NTAPI +FatCreateDcb(IN PFAT_IRP_CONTEXT IrpContext, + IN PVCB Vcb, + IN PFCB ParentDcb) +{ + PFCB Fcb; + + /* Allocate it and zero it */ + Fcb = ExAllocatePoolWithTag(NonPagedPool, sizeof(FCB), TAG_FCB); + RtlZeroMemory(Fcb, sizeof(FCB)); + + /* Set node types */ + Fcb->Header.NodeTypeCode = FAT_NTC_DCB; + Fcb->Header.NodeByteSize = sizeof(FCB); + Fcb->Condition = FcbGood; + + /* Initialize resources */ + Fcb->Header.Resource = &Fcb->Resource; + ExInitializeResourceLite(Fcb->Header.Resource); + + Fcb->Header.PagingIoResource = &Fcb->PagingIoResource; + ExInitializeResourceLite(Fcb->Header.PagingIoResource); + + /* Initialize mutexes */ + Fcb->Header.FastMutex = &Fcb->HeaderMutex; + ExInitializeFastMutex(&Fcb->HeaderMutex); + FsRtlSetupAdvancedHeader(&Fcb->Header, &Fcb->HeaderMutex); + + /* Insert into parent's DCB list */ + InsertHeadList(&ParentDcb->Dcb.ParentDcbList, &Fcb->ParentDcbLinks); + + /* Set backlinks */ + Fcb->ParentFcb = ParentDcb; + Fcb->Vcb = Vcb; + + /* Initialize parent dcb list */ + InitializeListHead(&Fcb->Dcb.ParentDcbList); + + return Fcb; +} /* EOF */ Modified: trunk/reactos/drivers/filesystems/fastfat_new/fastfat.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/fastfat.h [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/fastfat.h [iso-8859-1] Fri Oct 2 15:59:51 2009 @@ -73,6 +73,11 @@ VOID NTAPI FatCreateRootDcb(IN PFAT_IRP_CONTEXT IrpContext, IN PVCB Vcb); + +PFCB NTAPI +FatCreateDcb(IN PFAT_IRP_CONTEXT IrpContext, + IN PVCB Vcb, + IN PFCB ParentDcb); /* -------------------------------------------------------- create.c */ @@ -264,7 +269,8 @@ FatCreateFcb( IN PFAT_IRP_CONTEXT IrpContext, IN PVCB Vcb, - IN PFCB ParentDcb); + IN PFCB ParentDcb, + IN FF_FILE *FileHandle); NTSTATUS FatOpenFcb( @@ -282,6 +288,10 @@ PCCB NTAPI FatCreateCcb(); +VOID NTAPI +FatSetFullNameInFcb(PFCB Fcb, + PUNICODE_STRING Name); + /* ------------------------------------------------------------ rw.c */ NTSTATUS NTAPI Modified: trunk/reactos/drivers/filesystems/fastfat_new/fatstruc.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/fatstruc.h [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/fatstruc.h [iso-8859-1] Fri Oct 2 15:59:51 2009 @@ -275,6 +275,8 @@ WCHAR ShortNameBuffer[0xc]; /* Full file name */ UNICODE_STRING FullFileName; + /* Long name with exact case */ + UNICODE_STRING ExactCaseLongName; /* A copy of fat attribute byte */ UCHAR DirentFatFlags; /* File basic info */ Modified: trunk/reactos/drivers/filesystems/fastfat_new/fcb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/fcb.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/fcb.c [iso-8859-1] Fri Oct 2 15:59:51 2009 @@ -11,6 +11,8 @@ #define NDEBUG #include "fastfat.h" +#define TAG_FILENAME 'fBnF' + /* FUNCTIONS ****************************************************************/ FSRTL_COMPARISON_RESULT @@ -109,7 +111,8 @@ NTAPI FatCreateFcb(IN PFAT_IRP_CONTEXT IrpContext, IN PVCB Vcb, - IN PFCB ParentDcb) + IN PFCB ParentDcb, + IN FF_FILE *FileHandle) { PFCB Fcb; @@ -141,6 +144,11 @@ Fcb->ParentFcb = ParentDcb; Fcb->Vcb = Vcb; + /* Set file handle and sizes */ + Fcb->Header.FileSize.LowPart = FileHandle->Filesize; + Fcb->Header.ValidDataLength.LowPart = FileHandle->Filesize; + Fcb->FatHandle = FileHandle; + return Fcb; } @@ -161,4 +169,76 @@ return Ccb; } +VOID +NTAPI +FatSetFullNameInFcb(PFCB Fcb, + PUNICODE_STRING Name) +{ + PUNICODE_STRING ParentName; + + /* Make sure this FCB's name wasn't already set */ + ASSERT(Fcb->FullFileName.Buffer == NULL); + + /* First of all, check exact case name */ + if (Fcb->ExactCaseLongName.Buffer) + { + ASSERT(Fcb->ExactCaseLongName.Length != 0); + + /* Use exact case name */ + Name = &Fcb->ExactCaseLongName; + } + + /* Treat root dir different */ + if (FatNodeType(Fcb->ParentFcb) == FAT_NTC_ROOT_DCB) + { + /* Set lengths */ + Fcb->FullFileName.MaximumLength = sizeof(WCHAR) + Name->Length; + Fcb->FullFileName.Length = Fcb->FullFileName.MaximumLength; + + /* Allocate a buffer */ + Fcb->FullFileName.Buffer = FsRtlAllocatePoolWithTag(PagedPool, + Fcb->FullFileName.Length, + TAG_FILENAME); + + /* Prefix with a backslash */ + Fcb->FullFileName.Buffer[0] = L'\\'; + + /* Copy the name here */ + RtlCopyMemory(&Fcb->FullFileName.Buffer[1], + &Name->Buffer[0], + Name->Length ); + } + else + { + ParentName = &Fcb->ParentFcb->FullFileName; + + /* Check if parent's name is set */ + if (!ParentName->Buffer) + return; + + /* Set lengths */ + Fcb->FullFileName.MaximumLength = + ParentName->Length + sizeof(WCHAR) + Name->Length; + Fcb->FullFileName.Length = Fcb->FullFileName.MaximumLength; + + /* Allocate a buffer */ + Fcb->FullFileName.Buffer = FsRtlAllocatePoolWithTag(PagedPool, + Fcb->FullFileName.Length, + TAG_FILENAME ); + + /* Copy parent's name here */ + RtlCopyMemory(&Fcb->FullFileName.Buffer[0], + &ParentName->Buffer[0], + ParentName->Length ); + + /* Add a backslash */ + Fcb->FullFileName.Buffer[ParentName->Length / sizeof(WCHAR)] = L'\\'; + + /* Copy given name here */ + RtlCopyMemory(&Fcb->FullFileName.Buffer[(ParentName->Length / sizeof(WCHAR)) + 1], + &Name->Buffer[0], + Name->Length ); + } +} + /* EOF */

1 0

[fireball] 43255: [fullfat] - Substitute more "free" to FF_Free function.
by fireball＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: fireball Date: Fri Oct 2 13:56:03 2009 New Revision: 43255 URL: http://svn.reactos.org/svn/reactos?rev=43255&view=rev Log: [fullfat] - Substitute more "free" to FF_Free function. Modified: trunk/reactos/lib/3rdparty/fullfat/ff_file.c Modified: trunk/reactos/lib/3rdparty/fullfat/ff_file.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/3rdparty/fullfat/ff_fi… ============================================================================== --- trunk/reactos/lib/3rdparty/fullfat/ff_file.c [iso-8859-1] (original) +++ trunk/reactos/lib/3rdparty/fullfat/ff_file.c [iso-8859-1] Fri Oct 2 13:56:03 2009 @@ -222,7 +222,7 @@ if(Object.Attrib == FF_FAT_ATTR_DIR) { if(!(pFile->Mode & FF_MODE_DIR)) { // Not the object, File Not Found! - free(pFile); + FF_Free(pFile); if(pError) { *pError = FF_ERR_FILE_OBJECT_IS_A_DIR; } @@ -233,7 +233,7 @@ //---------- Ensure Read-Only files don't get opened for Writing. if((pFile->Mode & FF_MODE_WRITE) || (pFile->Mode & FF_MODE_APPEND)) { if((Object.Attrib & FF_FAT_ATTR_READONLY)) { - free(pFile); + FF_Free(pFile); if(pError) { *pError = FF_ERR_FILE_IS_READ_ONLY; } @@ -276,7 +276,7 @@ if(pFileChain->ObjectCluster == pFile->ObjectCluster) { // File is already open! DON'T ALLOW IT! FF_ReleaseSemaphore(pIoman->pSemaphore); - free(pFile); + FF_Free(pFile); if(pError) { *pError = FF_ERR_FILE_ALREADY_OPEN; } @@ -294,7 +294,7 @@ return pFile; }else { - free(pFile); + FF_Free(pFile); if(pError) { *pError = FF_ERR_FILE_NOT_FOUND; } @@ -305,7 +305,7 @@ *pError = FF_ERR_FILE_INVALID_PATH; } - free(pFile); + FF_Free(pFile); return (FF_FILE *)NULL; } @@ -1405,7 +1405,7 @@ FF_ReleaseSemaphore(pFile->pIoman->pSemaphore); // If file written, flush to disk - free(pFile); + FF_Free(pFile); // Simply free the pointer! return FF_ERR_NONE; }

1 0

[janderwald] 43254: - Rewrite Wave API to enumerate wave out / in devices at startup - Improves speedup of application minus short delay in system boot - Enumerate controls for source and destination lines
by janderwald＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: janderwald Date: Fri Oct 2 12:57:24 2009 New Revision: 43254 URL: http://svn.reactos.org/svn/reactos?rev=43254&view=rev Log: - Rewrite Wave API to enumerate wave out / in devices at startup - Improves speedup of application minus short delay in system boot - Enumerate controls for source and destination lines Added: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/sup.c (with props) trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wave.c (with props) Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/control.c trunk/reactos/drivers/wdm/audio/legacy/wdmaud/entry.c trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.h trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.rbuild Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/control.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/control.c [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/control.c [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -19,378 +19,51 @@ const GUID KSDATAFORMAT_SPECIFIER_WAVEFORMATEX = {0x05589f81L, 0xc356, 0x11ce, {0xbf, 0x01, 0x00, 0xaa, 0x00, 0x55, 0x59, 0x5a}}; const GUID KSPROPSETID_Topology = {0x720D4AC0L, 0x7533, 0x11D0, {0xA5, 0xD6, 0x28, 0xDB, 0x04, 0xC1, 0x00, 0x00}}; -NTSTATUS -SetIrpIoStatus( - IN PIRP Irp, - IN NTSTATUS Status, - IN ULONG Length) -{ - Irp->IoStatus.Information = Length; - Irp->IoStatus.Status = Status; - IoCompleteRequest(Irp, IO_NO_INCREMENT); - return Status; - -} - -NTSTATUS -GetFilterIdAndPinId( - IN PDEVICE_OBJECT DeviceObject, - IN PWDMAUD_DEVICE_INFO DeviceInfo, - IN PWDMAUD_CLIENT ClientInfo, - IN PULONG FilterId, - IN PULONG PinId) -{ - KSP_PIN Pin; - ULONG Count, BytesReturned, Index, SubIndex, Result, NumPins; + +NTSTATUS +WdmAudControlOpen( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo) +{ + if (DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) + { + return WdmAudControlOpenMixer(DeviceObject, Irp, DeviceInfo, ClientInfo); + } + + if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE || DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) + { + return WdmAudControlOpenWave(DeviceObject, Irp, DeviceInfo, ClientInfo); + } + + return SetIrpIoStatus(Irp, STATUS_NOT_SUPPORTED, sizeof(WDMAUD_DEVICE_INFO)); +} + +NTSTATUS +WdmAudControlDeviceType( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo) +{ + ULONG Result = 0; NTSTATUS Status; - KSPIN_COMMUNICATION Communication; - KSPIN_DATAFLOW DataFlow; PWDMAUD_DEVICE_EXTENSION DeviceExtension; - if (DeviceInfo->DeviceType != WAVE_OUT_DEVICE_TYPE && DeviceInfo->DeviceType != WAVE_IN_DEVICE_TYPE) - { - DPRINT1("FIXME: Unsupported device type %x\n", DeviceInfo->DeviceType); - return STATUS_UNSUCCESSFUL; - } - - Pin.Property.Set = KSPROPSETID_Sysaudio; - Pin.Property.Id = KSPROPERTY_SYSAUDIO_DEVICE_COUNT; - Pin.Property.Flags = KSPROPERTY_TYPE_GET; - DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSPROPERTY), (PVOID)&Count, sizeof(ULONG), &BytesReturned); - if (!NT_SUCCESS(Status)) - return STATUS_UNSUCCESSFUL; - - Result = 0; - for(Index = 0; Index < Count; Index++) - { - /* query number of pins */ - Pin.Reserved = Index; // see sysaudio - Pin.Property.Flags = KSPROPERTY_TYPE_GET; - Pin.Property.Set = KSPROPSETID_Pin; - Pin.Property.Id = KSPROPERTY_PIN_CTYPES; - Pin.PinId = 0; - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&NumPins, sizeof(ULONG), &BytesReturned); - if (NT_SUCCESS(Status)) - { - /* enumerate now all pins */ - for(SubIndex = 0; SubIndex < NumPins; SubIndex++) - { - Pin.PinId = SubIndex; - Pin.Property.Id = KSPROPERTY_PIN_COMMUNICATION; - Communication = KSPIN_COMMUNICATION_NONE; - - /* get pin communication type */ - KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&Communication, sizeof(KSPIN_COMMUNICATION), &BytesReturned); - - Pin.Property.Id = KSPROPERTY_PIN_DATAFLOW; - DataFlow = 0; - - /* get pin dataflow type */ - KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&DataFlow, sizeof(KSPIN_DATAFLOW), &BytesReturned); - - if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE) - { - if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_IN) - { - if(DeviceInfo->DeviceIndex == Result) - { - /* found the index */ - *FilterId = Index; - *PinId = SubIndex; - return STATUS_SUCCESS; - } - - Result++; - } - } - else if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) - { - if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_OUT) - { - if(DeviceInfo->DeviceIndex == Result) - { - /* found the index */ - *FilterId = Index; - *PinId = SubIndex; - return STATUS_SUCCESS; - } - Result++; - } - } - } - } - } - - return STATUS_UNSUCCESSFUL; -} - -NTSTATUS -WdmAudControlOpen( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - IN PWDMAUD_DEVICE_INFO DeviceInfo, - IN PWDMAUD_CLIENT ClientInfo) -{ - SYSAUDIO_INSTANCE_INFO InstanceInfo; - PWDMAUD_DEVICE_EXTENSION DeviceExtension; - ULONG BytesReturned; - NTSTATUS Status; - ACCESS_MASK DesiredAccess = 0; - HANDLE PinHandle; - KSPIN_CONNECT * PinConnect; - ULONG Length, Index; - KSDATAFORMAT_WAVEFORMATEX * DataFormat; - ULONG FilterId; - ULONG PinId; - ULONG FreeIndex; - if (DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) { - return WdmAudControlOpenMixer(DeviceObject, Irp, DeviceInfo, ClientInfo); - } - - if (DeviceInfo->DeviceType != WAVE_OUT_DEVICE_TYPE && DeviceInfo->DeviceType != WAVE_IN_DEVICE_TYPE) - { - DPRINT1("FIXME: only waveout / wavein devices are supported\n"); - return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); - } - - if (DeviceInfo->u.WaveFormatEx.wFormatTag != WAVE_FORMAT_PCM) - { - DPRINT("FIXME: Only WAVE_FORMAT_PCM is supported RequestFormat %x\n", DeviceInfo->u.WaveFormatEx.wFormatTag); - return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); - } - - Status = GetFilterIdAndPinId(DeviceObject, DeviceInfo, ClientInfo, &FilterId, &PinId); - if (!NT_SUCCESS(Status)) - { - DPRINT1("Invalid device index %u\n", DeviceInfo->DeviceIndex); - return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); - } - - /* close pin handle which uses same virtual audio device id and pin id */ - FreeIndex = MAXULONG; - for(Index = 0; Index < ClientInfo->NumPins; Index++) - { - if (ClientInfo->hPins[Index].FilterId == FilterId && ClientInfo->hPins[Index].PinId == PinId && ClientInfo->hPins[Index].Handle && ClientInfo->hPins[Index].Type == DeviceInfo->DeviceType) - { - ZwClose(ClientInfo->hPins[Index].Handle); - ClientInfo->hPins[Index].Handle = NULL; - FreeIndex = Index; - } - } - - - Length = sizeof(KSDATAFORMAT_WAVEFORMATEX) + sizeof(KSPIN_CONNECT); - PinConnect = ExAllocatePool(NonPagedPool, Length); - if (!PinConnect) - { - /* no memory */ - return SetIrpIoStatus(Irp, STATUS_NO_MEMORY, 0); - } - - DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE || - DeviceInfo->DeviceType == MIDI_IN_DEVICE_TYPE || - DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) - { - DesiredAccess |= GENERIC_READ; - } - - if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE || - DeviceInfo->DeviceType == MIDI_OUT_DEVICE_TYPE || - DeviceInfo->DeviceType == AUX_DEVICE_TYPE || - DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) - { - DesiredAccess |= GENERIC_WRITE; - } - - PinConnect->Interface.Set = KSINTERFACESETID_Standard; - PinConnect->Interface.Id = KSINTERFACE_STANDARD_STREAMING; - PinConnect->Interface.Flags = 0; - PinConnect->Medium.Set = KSMEDIUMSETID_Standard; - PinConnect->Medium.Id = KSMEDIUM_TYPE_ANYINSTANCE; - PinConnect->Medium.Flags = 0; - PinConnect->PinToHandle = NULL; - PinConnect->PinId = PinId; - PinConnect->Priority.PriorityClass = KSPRIORITY_NORMAL; - PinConnect->Priority.PrioritySubClass = 1; - - - DataFormat = (KSDATAFORMAT_WAVEFORMATEX*) (PinConnect + 1); - DataFormat->WaveFormatEx.wFormatTag = DeviceInfo->u.WaveFormatEx.wFormatTag; - DataFormat->WaveFormatEx.nChannels = DeviceInfo->u.WaveFormatEx.nChannels; - DataFormat->WaveFormatEx.nSamplesPerSec = DeviceInfo->u.WaveFormatEx.nSamplesPerSec; - DataFormat->WaveFormatEx.nBlockAlign = DeviceInfo->u.WaveFormatEx.nBlockAlign; - DataFormat->WaveFormatEx.nAvgBytesPerSec = DeviceInfo->u.WaveFormatEx.nAvgBytesPerSec; - DataFormat->WaveFormatEx.wBitsPerSample = DeviceInfo->u.WaveFormatEx.wBitsPerSample; - DataFormat->WaveFormatEx.cbSize = 0; - DataFormat->DataFormat.FormatSize = sizeof(KSDATAFORMAT) + sizeof(WAVEFORMATEX); - DataFormat->DataFormat.Flags = 0; - DataFormat->DataFormat.Reserved = 0; - DataFormat->DataFormat.MajorFormat = KSDATAFORMAT_TYPE_AUDIO; - - DataFormat->DataFormat.SubFormat = KSDATAFORMAT_SUBTYPE_PCM; - DataFormat->DataFormat.Specifier = KSDATAFORMAT_SPECIFIER_WAVEFORMATEX; - DataFormat->DataFormat.SampleSize = 4; - - /* setup property request */ - InstanceInfo.Property.Set = KSPROPSETID_Sysaudio; - InstanceInfo.Property.Id = KSPROPERTY_SYSAUDIO_INSTANCE_INFO; - InstanceInfo.Property.Flags = KSPROPERTY_TYPE_SET; - InstanceInfo.Flags = 0; - InstanceInfo.DeviceNumber = FilterId; - - /* first open the virtual device */ - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&InstanceInfo, sizeof(SYSAUDIO_INSTANCE_INFO), NULL, 0, &BytesReturned); - - if (!NT_SUCCESS(Status)) - { - /* failed */ - ExFreePool(PinConnect); - return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); - } - - /* now create the pin */ - Status = KsCreatePin(DeviceExtension->hSysAudio, PinConnect, DesiredAccess, &PinHandle); - - /* free create info */ - ExFreePool(PinConnect); - - if (NT_SUCCESS(Status)) - { - PWDMAUD_HANDLE Handels; - - if (FreeIndex != MAXULONG) - { - /* re-use a free index */ - ClientInfo->hPins[Index].Handle = PinHandle; - ClientInfo->hPins[Index].FilterId = FilterId; - ClientInfo->hPins[Index].PinId = PinId; - ClientInfo->hPins[Index].Type = DeviceInfo->DeviceType; - - DeviceInfo->hDevice = PinHandle; - return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); - } - - Handels = ExAllocatePool(NonPagedPool, sizeof(WDMAUD_HANDLE) * (ClientInfo->NumPins+1)); - - if (Handels) - { - if (ClientInfo->NumPins) - { - RtlMoveMemory(Handels, ClientInfo->hPins, sizeof(WDMAUD_HANDLE) * ClientInfo->NumPins); - ExFreePool(ClientInfo->hPins); - } - - ClientInfo->hPins = Handels; - ClientInfo->hPins[ClientInfo->NumPins].Handle = PinHandle; - ClientInfo->hPins[ClientInfo->NumPins].Type = DeviceInfo->DeviceType; - ClientInfo->hPins[ClientInfo->NumPins].FilterId = FilterId; - ClientInfo->hPins[ClientInfo->NumPins].PinId = PinId; - ClientInfo->NumPins++; - } - DeviceInfo->hDevice = PinHandle; - } - else - { - DeviceInfo->hDevice = NULL; - } - - return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); -} - -NTSTATUS -WdmAudControlDeviceType( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp, - IN PWDMAUD_DEVICE_INFO DeviceInfo, - IN PWDMAUD_CLIENT ClientInfo) -{ - KSP_PIN Pin; - ULONG Count, BytesReturned, Index, SubIndex, Result, NumPins; - NTSTATUS Status; - KSPIN_COMMUNICATION Communication; - KSPIN_DATAFLOW DataFlow; - PWDMAUD_DEVICE_EXTENSION DeviceExtension; - - DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - if (DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) - { - DeviceInfo->DeviceCount = DeviceExtension->MixerInfoCount; - return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); - } - - if (DeviceInfo->DeviceType != WAVE_OUT_DEVICE_TYPE && DeviceInfo->DeviceType != WAVE_IN_DEVICE_TYPE) - { - DPRINT("FIXME: Unsupported device type %x\n", DeviceInfo->DeviceType); - DeviceInfo->DeviceCount = 0; - return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); - } - - Pin.Property.Set = KSPROPSETID_Sysaudio; - Pin.Property.Id = KSPROPERTY_SYSAUDIO_DEVICE_COUNT; - Pin.Property.Flags = KSPROPERTY_TYPE_GET; - - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSPROPERTY), (PVOID)&Count, sizeof(ULONG), &BytesReturned); - if (!NT_SUCCESS(Status)) - { - DPRINT1("KSPROPERTY_SYSAUDIO_DEVICE_COUNT failed with %x\n", Status); - return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); - } - Result = 0; - /* now enumerate all available filters */ - for(Index = 0; Index < Count; Index++) - { - /* query number of pins */ - Pin.Reserved = Index; // see sysaudio - Pin.Property.Flags = KSPROPERTY_TYPE_GET; - Pin.Property.Set = KSPROPSETID_Pin; - Pin.Property.Id = KSPROPERTY_PIN_CTYPES; - Pin.PinId = 0; - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&NumPins, sizeof(ULONG), &BytesReturned); - if (NT_SUCCESS(Status)) - { - /* enumerate now all pins */ - for(SubIndex = 0; SubIndex < NumPins; SubIndex++) - { - Pin.PinId = SubIndex; - Pin.Property.Id = KSPROPERTY_PIN_COMMUNICATION; - Communication = KSPIN_COMMUNICATION_NONE; - - /* get pin communication type */ - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&Communication, sizeof(KSPIN_COMMUNICATION), &BytesReturned); - if (!NT_SUCCESS(Status)) - continue; - - Pin.Property.Id = KSPROPERTY_PIN_DATAFLOW; - DataFlow = 0; - - /* get pin dataflow type */ - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&DataFlow, sizeof(KSPIN_DATAFLOW), &BytesReturned); - if (!NT_SUCCESS(Status)) - continue; - - if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE) - { - if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_IN) - Result++; - } - else if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) - { - if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_OUT) - Result++; - } - } - } + Result = DeviceExtension->MixerInfoCount; + } + else if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE) + { + Result = DeviceExtension->WaveOutDeviceCount; + } + else if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) + { + Result = DeviceExtension->WaveInDeviceCount; } /* store result count */ @@ -436,253 +109,6 @@ return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); } -ULONG -CheckFormatSupport( - IN PKSDATARANGE_AUDIO DataRangeAudio, - ULONG SampleFrequency, - ULONG Mono8Bit, - ULONG Stereo8Bit, - ULONG Mono16Bit, - ULONG Stereo16Bit) -{ - ULONG Result = 0; - - if (DataRangeAudio->MinimumSampleFrequency <= SampleFrequency && DataRangeAudio->MaximumSampleFrequency >= SampleFrequency) - { - if (DataRangeAudio->MinimumBitsPerSample <= 8 && DataRangeAudio->MaximumBitsPerSample >= 8) - { - Result |= Mono8Bit; - if (DataRangeAudio->MaximumChannels >= 2) - { - Result |= Stereo8Bit; - } - } - - if (DataRangeAudio->MinimumBitsPerSample <= 16 && DataRangeAudio->MaximumBitsPerSample >= 16) - { - Result |= Mono16Bit; - if (DataRangeAudio->MaximumChannels >= 2) - { - Result |= Stereo8Bit; - } - } - } - return Result; - -} - -PKEY_VALUE_PARTIAL_INFORMATION -ReadKeyValue( - IN HANDLE hSubKey, - IN PUNICODE_STRING KeyName) -{ - NTSTATUS Status; - ULONG Length; - PKEY_VALUE_PARTIAL_INFORMATION PartialInformation; - - /* now query MatchingDeviceId key */ - Status = ZwQueryValueKey(hSubKey, KeyName, KeyValuePartialInformation, NULL, 0, &Length); - - /* check for success */ - if (Status != STATUS_BUFFER_TOO_SMALL) - return NULL; - - /* allocate a buffer for key data */ - PartialInformation = ExAllocatePool(NonPagedPool, Length); - - if (!PartialInformation) - return NULL; - - - /* now query MatchingDeviceId key */ - Status = ZwQueryValueKey(hSubKey, KeyName, KeyValuePartialInformation, PartialInformation, Length, &Length); - - /* check for success */ - if (!NT_SUCCESS(Status)) - { - ExFreePool(PartialInformation); - return NULL; - } - - if (PartialInformation->Type != REG_SZ) - { - /* invalid key type */ - ExFreePool(PartialInformation); - return NULL; - } - - return PartialInformation; -} - - -NTSTATUS -CompareProductName( - IN HANDLE hSubKey, - IN LPWSTR PnpName, - IN ULONG ProductNameSize, - OUT LPWSTR ProductName) -{ - PKEY_VALUE_PARTIAL_INFORMATION PartialInformation; - UNICODE_STRING DriverDescName = RTL_CONSTANT_STRING(L"DriverDesc"); - UNICODE_STRING MatchingDeviceIdName = RTL_CONSTANT_STRING(L"MatchingDeviceId"); - ULONG Length; - LPWSTR DeviceName; - - /* read MatchingDeviceId value */ - PartialInformation = ReadKeyValue(hSubKey, &MatchingDeviceIdName); - - if (!PartialInformation) - return STATUS_UNSUCCESSFUL; - - - /* extract last '&' */ - DeviceName = wcsrchr((LPWSTR)PartialInformation->Data, L'&'); - ASSERT(DeviceName); - /* terminate it */ - DeviceName[0] = L'\0'; - - Length = wcslen((LPWSTR)PartialInformation->Data); - - DPRINT("DeviceName %S PnpName %S Length %u\n", (LPWSTR)PartialInformation->Data, PnpName, Length); - - if (_wcsnicmp((LPWSTR)PartialInformation->Data, &PnpName[4], Length)) - { - ExFreePool(PartialInformation); - return STATUS_NO_MATCH; - } - - /* free buffer */ - ExFreePool(PartialInformation); - - /* read DriverDescName value */ - PartialInformation = ReadKeyValue(hSubKey, &DriverDescName); - - if (!PartialInformation) - { - /* failed to read driver desc key */ - return STATUS_UNSUCCESSFUL; - } - - /* copy key name */ - Length = min(ProductNameSize * sizeof(WCHAR), PartialInformation->DataLength); - RtlMoveMemory(ProductName, (PVOID)PartialInformation->Data, Length); - - /* zero terminate it */ - ProductName[ProductNameSize-1] = L'\0'; - - /* free buffer */ - ExFreePool(PartialInformation); - - return STATUS_SUCCESS; -} - - - -NTSTATUS -FindProductName( - IN LPWSTR PnpName, - IN ULONG ProductNameSize, - OUT LPWSTR ProductName) -{ - UNICODE_STRING KeyName = RTL_CONSTANT_STRING(L"\\Registry\\Machine\\System\\CurrentControlSet\\Control\\Class\\{4D36E96C-E325-11CE-BFC1-08002BE10318}"); - - UNICODE_STRING SubKeyName; - WCHAR SubKey[20]; - OBJECT_ATTRIBUTES ObjectAttributes; - HANDLE hKey, hSubKey; - NTSTATUS Status; - ULONG Length, Index; - PKEY_FULL_INFORMATION KeyInformation; - - for(Index = 0; Index < wcslen(PnpName); Index++) - { - if (PnpName[Index] == '#') - PnpName[Index] = L'\\'; - } - - - /* initialize key attributes */ - InitializeObjectAttributes(&ObjectAttributes, &KeyName, OBJ_CASE_INSENSITIVE | OBJ_OPENIF, NULL, NULL); - - /* open the key */ - Status = ZwOpenKey(&hKey, GENERIC_READ, &ObjectAttributes); - - /* check for success */ - if (!NT_SUCCESS(Status)) - return Status; - - /* query num of subkeys */ - Status = ZwQueryKey(hKey, KeyFullInformation, NULL, 0, &Length); - - if (Status != STATUS_BUFFER_TOO_SMALL) - { - DPRINT1("ZwQueryKey failed with %x\n", Status); - /* failed */ - ZwClose(hKey); - return Status; - } - - /* allocate key information struct */ - KeyInformation = ExAllocatePool(NonPagedPool, Length); - if (!KeyInformation) - { - /* no memory */ - ZwClose(hKey); - return STATUS_INSUFFICIENT_RESOURCES; - } - - /* query num of subkeys */ - Status = ZwQueryKey(hKey, KeyFullInformation, (PVOID)KeyInformation, Length, &Length); - - if (!NT_SUCCESS(Status)) - { - DPRINT1("ZwQueryKey failed with %x\n", Status); - ExFreePool(KeyInformation); - ZwClose(hKey); - return Status; - } - - /* now iterate through all subkeys */ - for(Index = 0; Index < KeyInformation->SubKeys; Index++) - { - /* subkeys are always in the format 0000-XXXX */ - swprintf(SubKey, L"%04u", Index); - - /* initialize subkey name */ - RtlInitUnicodeString(&SubKeyName, SubKey); - - /* initialize key attributes */ - InitializeObjectAttributes(&ObjectAttributes, &SubKeyName, OBJ_CASE_INSENSITIVE | OBJ_OPENIF, hKey, NULL); - - /* open the sub key */ - Status = ZwOpenKey(&hSubKey, GENERIC_READ, &ObjectAttributes); - - /* check for success */ - if (NT_SUCCESS(Status)) - { - /* compare product name */ - Status = CompareProductName(hSubKey, PnpName, ProductNameSize, ProductName); - - /* close subkey */ - ZwClose(hSubKey); - - if (NT_SUCCESS(Status)) - break; - } - } - - /* free buffer */ - ExFreePool(KeyInformation); - - /* close key */ - ZwClose(hKey); - - /* no matching key found */ - return Status; -} - - - NTSTATUS WdmAudCapabilities( IN PDEVICE_OBJECT DeviceObject, @@ -692,19 +118,6 @@ { PWDMAUD_DEVICE_EXTENSION DeviceExtension; NTSTATUS Status = STATUS_UNSUCCESSFUL; - KSP_PIN PinProperty; - KSCOMPONENTID ComponentId; - KSMULTIPLE_ITEM * MultipleItem; - ULONG BytesReturned; - PKSDATARANGE_AUDIO DataRangeAudio; - PKSDATARANGE DataRange; - ULONG Index; - ULONG wChannels = 0; - ULONG dwFormats = 0; - ULONG dwSupport = 0; - ULONG FilterId; - ULONG PinId; - WCHAR DeviceName[MAX_PATH]; DPRINT("WdmAudCapabilities entered\n"); @@ -713,114 +126,11 @@ if (DeviceInfo->DeviceType == MIXER_DEVICE_TYPE) { Status = WdmAudMixerCapabilities(DeviceObject, DeviceInfo, ClientInfo, DeviceExtension); - return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); - } - - - Status = GetFilterIdAndPinId(DeviceObject, DeviceInfo, ClientInfo, &FilterId, &PinId); - if (!NT_SUCCESS(Status)) - { - DPRINT1("Invalid device index provided %u\n", DeviceInfo->DeviceIndex); - return SetIrpIoStatus(Irp, STATUS_INVALID_PARAMETER, 0); - } - - PinProperty.PinId = FilterId; - PinProperty.Property.Set = KSPROPSETID_Sysaudio; - PinProperty.Property.Id = KSPROPERTY_SYSAUDIO_COMPONENT_ID; - PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; - - RtlZeroMemory(&ComponentId, sizeof(KSCOMPONENTID)); - - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)&ComponentId, sizeof(KSCOMPONENTID), &BytesReturned); - if (NT_SUCCESS(Status)) - { - DeviceInfo->u.WaveOutCaps.wMid = ComponentId.Manufacturer.Data1 - 0xd5a47fa7; - DeviceInfo->u.WaveOutCaps.vDriverVersion = MAKELONG(ComponentId.Version, ComponentId.Revision); - } - - /* retrieve pnp base name */ - PinProperty.PinId = FilterId; - PinProperty.Property.Set = KSPROPSETID_Sysaudio; - PinProperty.Property.Id = KSPROPERTY_SYSAUDIO_DEVICE_INTERFACE_NAME; - PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)DeviceName, sizeof(DeviceName), &BytesReturned); - if (NT_SUCCESS(Status)) - { - /* find product name */ - Status = FindProductName(DeviceName, MAXPNAMELEN, DeviceInfo->u.WaveOutCaps.szPname); - - /* check for success */ - if (!NT_SUCCESS(Status)) - { - DeviceInfo->u.WaveOutCaps.szPname[0] = L'\0'; - } - } - - PinProperty.Reserved = DeviceInfo->DeviceIndex; - PinProperty.PinId = PinId; - PinProperty.Property.Set = KSPROPSETID_Pin; - PinProperty.Property.Id = KSPROPERTY_PIN_DATARANGES; - PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; - - BytesReturned = 0; - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)NULL, 0, &BytesReturned); - if (Status != STATUS_BUFFER_TOO_SMALL) - { - return SetIrpIoStatus(Irp, Status, 0); - } - - MultipleItem = ExAllocatePool(NonPagedPool, BytesReturned); - if (!MultipleItem) - { - /* no memory */ - return SetIrpIoStatus(Irp, STATUS_NO_MEMORY, 0); - } - - Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)MultipleItem, BytesReturned, &BytesReturned); - if (!NT_SUCCESS(Status)) - { - ExFreePool(MultipleItem); - return SetIrpIoStatus(Irp, Status, 0); - } - - DataRange = (PKSDATARANGE) (MultipleItem + 1); - for(Index = 0; Index < MultipleItem->Count; Index++) - { - if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE || DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) - { - if (DataRange->FormatSize == sizeof(KSDATARANGE_AUDIO)) - { - DataRangeAudio = (PKSDATARANGE_AUDIO)DataRange; - - if (IsEqualGUIDAligned(&DataRangeAudio->DataRange.MajorFormat, &KSDATAFORMAT_TYPE_AUDIO) && - IsEqualGUIDAligned(&DataRangeAudio->DataRange.SubFormat, &KSDATAFORMAT_SUBTYPE_PCM) && - IsEqualGUIDAligned(&DataRangeAudio->DataRange.Specifier, &KSDATAFORMAT_SPECIFIER_WAVEFORMATEX)) - { - DPRINT("Min Sample %u Max Sample %u Min Bits %u Max Bits %u Max Channel %u\n", DataRangeAudio->MinimumSampleFrequency, DataRangeAudio->MaximumSampleFrequency, - DataRangeAudio->MinimumBitsPerSample, DataRangeAudio->MaximumBitsPerSample, DataRangeAudio->MaximumChannels); - - dwFormats |= CheckFormatSupport(DataRangeAudio, 11025, WAVE_FORMAT_1M08, WAVE_FORMAT_1S08, WAVE_FORMAT_1M16, WAVE_FORMAT_1S16); - dwFormats |= CheckFormatSupport(DataRangeAudio, 22050, WAVE_FORMAT_2M08, WAVE_FORMAT_2S08, WAVE_FORMAT_2M16, WAVE_FORMAT_2S16); - dwFormats |= CheckFormatSupport(DataRangeAudio, 44100, WAVE_FORMAT_4M08, WAVE_FORMAT_4S08, WAVE_FORMAT_4M16, WAVE_FORMAT_4S16); - dwFormats |= CheckFormatSupport(DataRangeAudio, 48000, WAVE_FORMAT_48M08, WAVE_FORMAT_48S08, WAVE_FORMAT_48M16, WAVE_FORMAT_48S16); - dwFormats |= CheckFormatSupport(DataRangeAudio, 96000, WAVE_FORMAT_96M08, WAVE_FORMAT_96S08, WAVE_FORMAT_96M16, WAVE_FORMAT_96S16); - - - wChannels = DataRangeAudio->MaximumChannels; - dwSupport = WAVECAPS_VOLUME; //FIXME get info from nodes - } - } - } - DataRange = (PKSDATARANGE)((PUCHAR)DataRange + DataRange->FormatSize); - } - - DeviceInfo->u.WaveOutCaps.dwFormats = dwFormats; - DeviceInfo->u.WaveOutCaps.dwSupport = dwSupport; - DeviceInfo->u.WaveOutCaps.wChannels = wChannels; - - ExFreePool(MultipleItem); + } + else if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE || DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE) + { + Status = WdmAudWaveCapabilities(DeviceObject, DeviceInfo, ClientInfo, DeviceExtension); + } return SetIrpIoStatus(Irp, Status, sizeof(WDMAUD_DEVICE_INFO)); } @@ -846,6 +156,7 @@ return STATUS_SUCCESS; } } + SetIrpIoStatus(Irp, STATUS_INVALID_PARAMETER, sizeof(WDMAUD_DEVICE_INFO)); return STATUS_INVALID_PARAMETER; } Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/entry.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/entry.c [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/entry.c [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -78,6 +78,8 @@ Status = WdmAudMixerInitialize(DeviceObject); DPRINT("WdmAudMixerInitialize Status %x\n", Status); + Status = WdmAudWaveInitialize(DeviceObject); + DPRINT("WdmAudWaveInitialize Status %x\n", Status); DeviceObject->Flags |= DO_DIRECT_IO | DO_POWER_PAGABLE; DeviceObject->Flags &= ~ DO_DEVICE_INITIALIZING; Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/mixer.c [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -21,6 +21,7 @@ const GUID KSNODETYPE_CHORUS = {0x20173F20L, 0xC559, 0x11D0, {0x8A, 0x2B, 0x00, 0xA0, 0xC9, 0x25, 0x5A, 0xC1}}; const GUID KSNODETYPE_REVERB = {0xEF0328E0L, 0xC558, 0x11D0, {0x8A, 0x2B, 0x00, 0xA0, 0xC9, 0x25, 0x5A, 0xC1}}; const GUID KSNODETYPE_SUPERMIX = {0xE573ADC0L, 0xC555, 0x11D0, {0x8A, 0x2B, 0x00, 0xA0, 0xC9, 0x25, 0x5A, 0xC1}}; +const GUID KSNODETYPE_SUM = {0xDA441A60L, 0xC556, 0x11D0, {0x8A, 0x2B, 0x00, 0xA0, 0xC9, 0x25, 0x5A, 0xC1}}; #define DESTINATION_LINE 0xFFFF0000 @@ -62,7 +63,7 @@ while(Entry != &MixerInfo->LineList) { MixerLineSrc = (LPMIXERLINE_EXT)CONTAINING_RECORD(Entry, MIXERLINE_EXT, Entry); - DPRINT("dwLineID %x dwLineID %x\n", MixerLineSrc->Line.dwLineID, dwLineID); + DPRINT1("dwLineID %x dwLineID %x\n", MixerLineSrc->Line.dwLineID, dwLineID); if (MixerLineSrc->Line.dwLineID == dwLineID) return MixerLineSrc; @@ -719,67 +720,369 @@ return Pins; } +PKSTOPOLOGY_CONNECTION +GetConnectionByIndex( + IN PKSMULTIPLE_ITEM MultipleItem, + IN ULONG Index) +{ + PKSTOPOLOGY_CONNECTION Descriptor; + + ASSERT(Index < MultipleItem->Count); + + Descriptor = (PKSTOPOLOGY_CONNECTION)(MultipleItem + 1); + return &Descriptor[Index]; +} + +LPGUID +GetNodeType( + IN PKSMULTIPLE_ITEM MultipleItem, + IN ULONG Index) +{ + LPGUID NodeType; + + ASSERT(Index < MultipleItem->Count); + + NodeType = (LPGUID)(MultipleItem + 1); + return &NodeType[Index]; +} + +NTSTATUS +GetControlsFromPinByConnectionIndex( + IN PKSMULTIPLE_ITEM NodeConnections, + IN PKSMULTIPLE_ITEM NodeTypes, + IN ULONG bUpDirection, + IN ULONG NodeConnectionIndex, + OUT PULONG Nodes) +{ + PKSTOPOLOGY_CONNECTION CurConnection; + LPGUID NodeType; + ULONG NodeIndex; + NTSTATUS Status; + ULONG NodeConnectionCount, Index; + PULONG NodeConnection; + + + /* get current connection */ + CurConnection = GetConnectionByIndex(NodeConnections, NodeConnectionIndex); + + if (bUpDirection) + NodeIndex = CurConnection->FromNode; + else + NodeIndex = CurConnection->ToNode; + + /* get target node type of current connection */ + NodeType = GetNodeType(NodeTypes, NodeIndex); + + if (IsEqualGUIDAligned(NodeType, &KSNODETYPE_SUM) || IsEqualGUIDAligned(NodeType, &KSNODETYPE_MUX)) + { + if (bUpDirection) + { + /* add the sum / mux node to destination line */ + //Nodes[NodeIndex] = TRUE; + } + + return STATUS_SUCCESS; + } + + /* now add the node */ + Nodes[NodeIndex] = TRUE; + + + /* get all node indexes referenced by that node */ + if (bUpDirection) + { + Status = GetNodeIndexes(NodeConnections, NodeIndex, TRUE, FALSE, &NodeConnectionCount, &NodeConnection); + } + else + { + Status = GetNodeIndexes(NodeConnections, NodeIndex, TRUE, TRUE, &NodeConnectionCount, &NodeConnection); + } + + if (NT_SUCCESS(Status)) + { + for(Index = 0; Index < NodeConnectionCount; Index++) + { + /* iterate recursively into the nodes */ + Status = GetControlsFromPinByConnectionIndex(NodeConnections, NodeTypes, bUpDirection, NodeConnection[Index], Nodes); + ASSERT(Status == STATUS_SUCCESS); + } + /* free node connection indexes */ + ExFreePool(NodeConnection); + } + + return Status; +} + +NTSTATUS +GetControlsFromPin( + IN PKSMULTIPLE_ITEM NodeConnections, + IN PKSMULTIPLE_ITEM NodeTypes, + IN ULONG PinId, + IN ULONG bUpDirection, + OUT PULONG Nodes) +{ + ULONG NodeConnectionCount, Index; + NTSTATUS Status; + PULONG NodeConnection; + + /* sanity check */ + ASSERT(PinId != (ULONG)-1); + + /* get all node indexes referenced by that pin */ + if (bUpDirection) + Status = GetNodeIndexes(NodeConnections, PinId, FALSE, FALSE, &NodeConnectionCount, &NodeConnection); + else + Status = GetNodeIndexes(NodeConnections, PinId, FALSE, TRUE, &NodeConnectionCount, &NodeConnection); + + for(Index = 0; Index < NodeConnectionCount; Index++) + { + /* get all associated controls */ + Status = GetControlsFromPinByConnectionIndex(NodeConnections, NodeTypes, bUpDirection, NodeConnection[Index], Nodes); + } + + ExFreePool(NodeConnection); + + return Status; +} + +NTSTATUS +AddMixerControl( + IN LPMIXER_INFO MixerInfo, + IN PFILE_OBJECT FileObject, + IN PKSMULTIPLE_ITEM NodeTypes, + IN ULONG NodeIndex, + OUT LPMIXERCONTROLW MixerControl) +{ + LPGUID NodeType; + KSP_NODE Node; + ULONG BytesReturned; + NTSTATUS Status; + LPWSTR Name; + + + /* initialize mixer control */ + MixerControl->cbStruct = sizeof(MIXERCONTROLW); + MixerControl->dwControlID = MixerInfo->ControlId; + + /* get node type */ + NodeType = GetNodeType(NodeTypes, NodeIndex); + /* store control type */ + MixerControl->dwControlType = GetControlTypeFromTopologyNode(NodeType); + + MixerControl->fdwControl = MIXERCONTROL_CONTROLF_UNIFORM; //FIXME + MixerControl->cMultipleItems = 0; //FIXME + + if (MixerControl->dwControlType == MIXERCONTROL_CONTROLTYPE_MUTE) + { + MixerControl->Bounds.dwMinimum = 0; + MixerControl->Bounds.dwMaximum = 1; + } + else if (MixerControl->dwControlType == MIXERCONTROL_CONTROLTYPE_VOLUME) + { + MixerControl->Bounds.dwMinimum = 0; + MixerControl->Bounds.dwMaximum = 0xFFFF; + MixerControl->Metrics.cSteps = 0xC0; //FIXME + } + + /* setup request to retrieve name */ + Node.NodeId = NodeIndex; + Node.Property.Id = KSPROPERTY_TOPOLOGY_NAME; + Node.Property.Flags = KSPROPERTY_TYPE_GET; + Node.Property.Set = KSPROPSETID_Topology; + Node.Reserved = 0; + + /* get node name size */ + Status = KsSynchronousIoControlDevice(FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Node, sizeof(KSP_NODE), NULL, 0, &BytesReturned); + + if (Status == STATUS_BUFFER_TOO_SMALL) + { + ASSERT(BytesReturned != 0); + Name = ExAllocatePool(NonPagedPool, BytesReturned); + if (!Name) + { + /* not enough memory */ + return STATUS_INSUFFICIENT_RESOURCES; + } + + /* get node name */ + Status = KsSynchronousIoControlDevice(FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Node, sizeof(KSP_NODE), (LPVOID)Name, BytesReturned, &BytesReturned); + if (NT_SUCCESS(Status)) + { + RtlMoveMemory(MixerControl->szShortName, Name, (min(MIXER_SHORT_NAME_CHARS, wcslen(Name)+1)) * sizeof(WCHAR)); + MixerControl->szShortName[MIXER_SHORT_NAME_CHARS-1] = L'\0'; + + RtlMoveMemory(MixerControl->szName, Name, (min(MIXER_LONG_NAME_CHARS, wcslen(Name)+1)) * sizeof(WCHAR)); + MixerControl->szName[MIXER_LONG_NAME_CHARS-1] = L'\0'; + } + + /* free name buffer */ + ExFreePool(Name); + } + + MixerInfo->ControlId++; + + DPRINT("Status %x Name %S\n", Status, MixerControl->szName); + return STATUS_SUCCESS; +} + NTSTATUS AddMixerSourceLine( IN OUT LPMIXER_INFO MixerInfo, IN PFILE_OBJECT FileObject, + IN PKSMULTIPLE_ITEM NodeConnections, + IN PKSMULTIPLE_ITEM NodeTypes, IN ULONG DeviceIndex, - IN ULONG PinId) + IN ULONG PinId, + IN ULONG bBridgePin, + IN ULONG bTargetPin) { LPMIXERLINE_EXT SrcLine, DstLine; NTSTATUS Status; KSP_PIN Pin; LPWSTR PinName; GUID NodeType; - ULONG BytesReturned; - - /* allocate src mixer line */ - SrcLine = (LPMIXERLINE_EXT)ExAllocatePool(NonPagedPool, sizeof(MIXERLINE_EXT)); - if (!SrcLine) - return STATUS_INSUFFICIENT_RESOURCES; - - /* zero struct */ - RtlZeroMemory(SrcLine, sizeof(MIXERLINE_EXT)); - - /* initialize mixer src line */ - SrcLine->DeviceIndex = DeviceIndex; - SrcLine->PinId = PinId; - SrcLine->Line.cbStruct = sizeof(MIXERLINEW); + ULONG BytesReturned, ControlCount, Index; + PULONG Nodes; + + if (!bTargetPin) + { + /* allocate src mixer line */ + SrcLine = (LPMIXERLINE_EXT)ExAllocatePool(NonPagedPool, sizeof(MIXERLINE_EXT)); + + if (!SrcLine) + return STATUS_INSUFFICIENT_RESOURCES; + + /* zero struct */ + RtlZeroMemory(SrcLine, sizeof(MIXERLINE_EXT)); + + } + else + { + ASSERT(!IsListEmpty(&MixerInfo->LineList)); + SrcLine = GetSourceMixerLineByLineId(MixerInfo, DESTINATION_LINE); + } /* get destination line */ DstLine = GetSourceMixerLineByLineId(MixerInfo, DESTINATION_LINE); - - /* initialize mixer destination line */ - SrcLine->Line.cbStruct = sizeof(MIXERLINEW); - SrcLine->Line.dwDestination = 0; - SrcLine->Line.dwSource = DstLine->Line.cConnections; - SrcLine->Line.dwLineID = (DstLine->Line.cConnections * 0x10000); - SrcLine->Line.fdwLine = MIXERLINE_LINEF_ACTIVE | MIXERLINE_LINEF_SOURCE; - SrcLine->Line.dwUser = 0; - SrcLine->Line.cChannels = DstLine->Line.cChannels; - SrcLine->Line.cConnections = 0; - SrcLine->Line.cControls = 1; //FIXME - - //HACK - SrcLine->LineControls = ExAllocatePool(NonPagedPool, SrcLine->Line.cControls * sizeof(MIXERCONTROLW)); - if (!SrcLine->LineControls) + ASSERT(DstLine); + + + if (!bTargetPin) + { + /* initialize mixer src line */ + SrcLine->DeviceIndex = DeviceIndex; + SrcLine->PinId = PinId; + SrcLine->Line.cbStruct = sizeof(MIXERLINEW); + + /* initialize mixer destination line */ + SrcLine->Line.cbStruct = sizeof(MIXERLINEW); + SrcLine->Line.dwDestination = 0; + SrcLine->Line.dwSource = DstLine->Line.cConnections; + SrcLine->Line.dwLineID = (DstLine->Line.cConnections * 0x10000); + SrcLine->Line.fdwLine = MIXERLINE_LINEF_ACTIVE | MIXERLINE_LINEF_SOURCE; + SrcLine->Line.dwUser = 0; + SrcLine->Line.cChannels = DstLine->Line.cChannels; + SrcLine->Line.cConnections = 0; + SrcLine->Line.Target.dwType = 1; + SrcLine->Line.Target.dwDeviceID = DstLine->Line.Target.dwDeviceID; + SrcLine->Line.Target.wMid = MixerInfo->MixCaps.wMid; + SrcLine->Line.Target.wPid = MixerInfo->MixCaps.wPid; + SrcLine->Line.Target.vDriverVersion = MixerInfo->MixCaps.vDriverVersion; + wcscpy(SrcLine->Line.Target.szPname, MixerInfo->MixCaps.szPname); + + } + + /* allocate a node arrary */ + Nodes = ExAllocatePool(NonPagedPool, sizeof(ULONG) * NodeTypes->Count); + + if (!Nodes) { /* not enough memory */ - ExFreePool(SrcLine); + if (!bTargetPin) + { + ExFreePool(SrcLine); + } return STATUS_INSUFFICIENT_RESOURCES; } - /* clear line controls */ - RtlZeroMemory(SrcLine->LineControls, sizeof(MIXERCONTROLW)); - - /* fill in pseudo mixer control */ - SrcLine->LineControls->dwControlID = 1; //FIXME - SrcLine->LineControls->cbStruct = sizeof(MIXERCONTROLW); - SrcLine->LineControls->fdwControl = 0; - SrcLine->LineControls->cMultipleItems = 0; - wcscpy(SrcLine->LineControls->szName, L"test"); - wcscpy(SrcLine->LineControls->szShortName, L"test"); - + /* clear nodes array */ + RtlZeroMemory(Nodes, sizeof(ULONG) * NodeTypes->Count); + + Status = GetControlsFromPin(NodeConnections, NodeTypes, PinId, bTargetPin, Nodes); + if (!NT_SUCCESS(Status)) + { + /* something went wrong */ + if (!bTargetPin) + { + ExFreePool(SrcLine); + } + ExFreePool(Nodes); + return Status; + } + + /* now count all nodes controlled by that pin */ + ControlCount = 0; + for(Index = 0; Index < NodeTypes->Count; Index++) + { + if (Nodes[Index]) + ControlCount++; + } + + /* now allocate the line controls */ + if (ControlCount) + { + SrcLine->LineControls = ExAllocatePool(NonPagedPool, sizeof(MIXERCONTROLW) * ControlCount); + + if (!SrcLine->LineControls) + { + /* no memory available */ + if (!bTargetPin) + { + ExFreePool(SrcLine); + } + ExFreePool(Nodes); + return STATUS_INSUFFICIENT_RESOURCES; + } + + SrcLine->NodeIds = ExAllocatePool(NonPagedPool, sizeof(ULONG) * ControlCount); + if (!SrcLine->NodeIds) + { + /* no memory available */ + ExFreePool(SrcLine->LineControls); + if (!bTargetPin) + { + ExFreePool(SrcLine); + } + ExFreePool(Nodes); + return STATUS_INSUFFICIENT_RESOURCES; + } + + /* zero line controls */ + RtlZeroMemory(SrcLine->LineControls, sizeof(MIXERCONTROLW) * ControlCount); + RtlZeroMemory(SrcLine->NodeIds, sizeof(ULONG) * ControlCount); + + ControlCount = 0; + for(Index = 0; Index < NodeTypes->Count; Index++) + { + if (Nodes[Index]) + { + /* store the node index for retrieving / setting details */ + SrcLine->NodeIds[ControlCount] = Index; + + Status = AddMixerControl(MixerInfo, FileObject, NodeTypes, Index, &SrcLine->LineControls[ControlCount]); + if (NT_SUCCESS(Status)) + { + /* increment control count on success */ + ControlCount++; + } + } + } + /* store control count */ + SrcLine->Line.cControls = ControlCount; + } + + /* release nodes array */ + ExFreePool(Nodes); /* get pin category */ Pin.PinId = PinId; @@ -831,17 +1134,12 @@ } } - SrcLine->Line.Target.dwType = 1; - SrcLine->Line.Target.dwDeviceID = DstLine->Line.Target.dwDeviceID; - SrcLine->Line.Target.wMid = MixerInfo->MixCaps.wMid; - SrcLine->Line.Target.wPid = MixerInfo->MixCaps.wPid; - SrcLine->Line.Target.vDriverVersion = MixerInfo->MixCaps.vDriverVersion; - wcscpy(SrcLine->Line.Target.szPname, MixerInfo->MixCaps.szPname); - - /* insert src line */ - InsertTailList(&MixerInfo->LineList, &SrcLine->Entry); - DstLine->Line.cConnections++; + if (!bTargetPin) + { + InsertTailList(&MixerInfo->LineList, &SrcLine->Entry); + DstLine->Line.cConnections++; + } return STATUS_SUCCESS; } @@ -851,8 +1149,12 @@ AddMixerSourceLines( IN OUT LPMIXER_INFO MixerInfo, IN PFILE_OBJECT FileObject, + IN PKSMULTIPLE_ITEM NodeConnections, + IN PKSMULTIPLE_ITEM NodeTypes, IN ULONG DeviceIndex, IN ULONG PinsCount, + IN ULONG BridgePinIndex, + IN ULONG TargetPinIndex, IN PULONG Pins) { ULONG Index; @@ -862,7 +1164,7 @@ { if (Pins[Index-1]) { - AddMixerSourceLine(MixerInfo, FileObject, DeviceIndex, Index-1); + AddMixerSourceLine(MixerInfo, FileObject, NodeConnections, NodeTypes, DeviceIndex, Index-1, (Index -1 == BridgePinIndex), (Index -1 == TargetPinIndex)); } } return Status; @@ -987,7 +1289,7 @@ PinsSrcRef[OutConnection->Pin] = TRUE; } - Status = AddMixerSourceLines(MixerInfo, FileObject, DeviceIndex, PinsRefCount, PinsSrcRef); + Status = AddMixerSourceLines(MixerInfo, FileObject, NodeConnections, NodeTypes, DeviceIndex, PinsRefCount, OutConnection->Pin, Index, PinsSrcRef); ExFreePool(MixerControls); ExFreePool(PinsSrcRef); @@ -1045,10 +1347,6 @@ if (!DestinationLine) return STATUS_INSUFFICIENT_RESOURCES; - /* initialize mixer info */ - MixerInfo->hMixer = hDevice; - MixerInfo->MixerFileObject = FileObject; - /* intialize mixer caps */ MixerInfo->MixCaps.wMid = MM_MICROSOFT; //FIXME MixerInfo->MixCaps.wPid = MM_PID_UNMAPPED; //FIXME @@ -1083,7 +1381,6 @@ DestinationLine->Line.dwUser = 0; DestinationLine->Line.dwComponentType = (bInput == 0 ? MIXERLINE_COMPONENTTYPE_DST_SPEAKERS : MIXERLINE_COMPONENTTYPE_DST_WAVEIN); DestinationLine->Line.cChannels = 2; //FIXME - DestinationLine->Line.cControls = 0; //FIXME wcscpy(DestinationLine->Line.szShortName, L"Summe"); //FIXME wcscpy(DestinationLine->Line.szName, L"Summe"); //FIXME DestinationLine->Line.Target.dwType = (bInput == 0 ? MIXERLINE_TARGETTYPE_WAVEOUT : MIXERLINE_TARGETTYPE_WAVEIN); @@ -1411,6 +1708,7 @@ { LPMIXERLINE_EXT MixerLineSrc; PWDMAUD_DEVICE_EXTENSION DeviceExtension; + ULONG Index; /* get device extension */ DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; @@ -1427,15 +1725,12 @@ ASSERT(MixerLineSrc); if (MixerLineSrc) { - RtlMoveMemory(DeviceInfo->u.MixControls.pamxctrl, MixerLineSrc->LineControls, min(MixerLineSrc->Line.cControls, DeviceInfo->u.MixControls.cControls) * sizeof(MIXERLINECONTROLSW)); + RtlMoveMemory(DeviceInfo->u.MixControls.pamxctrl, MixerLineSrc->LineControls, min(MixerLineSrc->Line.cControls, DeviceInfo->u.MixControls.cControls) * sizeof(MIXERCONTROLW)); } return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); } else if (DeviceInfo->Flags == MIXER_GETLINECONTROLSF_ONEBYTYPE) { - DPRINT1("dwLineID %u\n",DeviceInfo->u.MixControls.dwLineID); - UNIMPLEMENTED - //HACK if ((ULONG)DeviceInfo->hDevice >= DeviceExtension->MixerInfoCount) { /* invalid parameter */ @@ -1444,11 +1739,18 @@ MixerLineSrc = GetSourceMixerLineByLineId(&DeviceExtension->MixerInfo[(ULONG)DeviceInfo->hDevice], DeviceInfo->u.MixControls.dwLineID); ASSERT(MixerLineSrc); - if (MixerLineSrc) - { - RtlMoveMemory(DeviceInfo->u.MixControls.pamxctrl, MixerLineSrc->LineControls, min(MixerLineSrc->Line.cControls, DeviceInfo->u.MixControls.cControls) * sizeof(MIXERLINECONTROLSW)); - } - return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); + + Index = 0; + for(Index = 0; Index < MixerLineSrc->Line.cControls; Index++) + { + if (DeviceInfo->u.MixControls.dwControlType == MixerLineSrc->LineControls[Index].dwControlType) + { + RtlMoveMemory(DeviceInfo->u.MixControls.pamxctrl, &MixerLineSrc->LineControls[Index], sizeof(MIXERCONTROLW)); + return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); + } + } + DPRINT1("DeviceInfo->u.MixControls.dwControlType %x not found in Line %x\n", DeviceInfo->u.MixControls.dwControlType, DeviceInfo->u.MixControls.dwLineID); + return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, sizeof(WDMAUD_DEVICE_INFO)); } UNIMPLEMENTED; Added: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/sup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/sup.c (added) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/sup.c [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -1,0 +1,300 @@ +/* + * COPYRIGHT: See COPYING in the top level directory + * PROJECT: ReactOS Kernel Streaming + * FILE: drivers/wdm/audio/legacy/wdmaud/sup.c + * PURPOSE: System Audio graph builder + * PROGRAMMER: Andrew Greenwood + * Johannes Anderwald + */ +#include "wdmaud.h" + +NTSTATUS +SetIrpIoStatus( + IN PIRP Irp, + IN NTSTATUS Status, + IN ULONG Length) +{ + Irp->IoStatus.Information = Length; + Irp->IoStatus.Status = Status; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + return Status; + +} + +NTSTATUS +ClosePin( + IN PWDMAUD_CLIENT ClientInfo, + IN ULONG FilterId, + IN ULONG PinId, + IN SOUND_DEVICE_TYPE DeviceType) +{ + ULONG Index; + + for(Index = 0; Index < ClientInfo->NumPins; Index++) + { + if (ClientInfo->hPins[Index].FilterId == FilterId && ClientInfo->hPins[Index].PinId == PinId && ClientInfo->hPins[Index].Handle && ClientInfo->hPins[Index].Type == DeviceType) + { + if (ClientInfo->hPins[Index].Type != MIXER_DEVICE_TYPE) + { + ZwClose(ClientInfo->hPins[Index].Handle); + } + ClientInfo->hPins[Index].Handle = NULL; + return Index; + } + } + return MAXULONG; +} + +NTSTATUS +InsertPinHandle( + IN PWDMAUD_CLIENT ClientInfo, + IN ULONG FilterId, + IN ULONG PinId, + IN SOUND_DEVICE_TYPE DeviceType, + IN HANDLE PinHandle, + IN ULONG FreeIndex) +{ + PWDMAUD_HANDLE Handles; + + if (FreeIndex != MAXULONG) + { + /* re-use a free index */ + ClientInfo->hPins[FreeIndex].Handle = PinHandle; + ClientInfo->hPins[FreeIndex].FilterId = FilterId; + ClientInfo->hPins[FreeIndex].PinId = PinId; + ClientInfo->hPins[FreeIndex].Type = DeviceType; + + return STATUS_SUCCESS; + } + + Handles = ExAllocatePool(NonPagedPool, sizeof(WDMAUD_HANDLE) * (ClientInfo->NumPins+1)); + + if (!Handles) + return STATUS_INSUFFICIENT_RESOURCES; + + if (ClientInfo->NumPins) + { + RtlMoveMemory(Handles, ClientInfo->hPins, sizeof(WDMAUD_HANDLE) * ClientInfo->NumPins); + ExFreePool(ClientInfo->hPins); + } + + ClientInfo->hPins = Handles; + ClientInfo->hPins[ClientInfo->NumPins].Handle = PinHandle; + ClientInfo->hPins[ClientInfo->NumPins].Type = DeviceType; + ClientInfo->hPins[ClientInfo->NumPins].FilterId = FilterId; + ClientInfo->hPins[ClientInfo->NumPins].PinId = PinId; + ClientInfo->NumPins++; + + return STATUS_SUCCESS; +} + +PKEY_VALUE_PARTIAL_INFORMATION +ReadKeyValue( + IN HANDLE hSubKey, + IN PUNICODE_STRING KeyName) +{ + NTSTATUS Status; + ULONG Length; + PKEY_VALUE_PARTIAL_INFORMATION PartialInformation; + + /* now query MatchingDeviceId key */ + Status = ZwQueryValueKey(hSubKey, KeyName, KeyValuePartialInformation, NULL, 0, &Length); + + /* check for success */ + if (Status != STATUS_BUFFER_TOO_SMALL) + return NULL; + + /* allocate a buffer for key data */ + PartialInformation = ExAllocatePool(NonPagedPool, Length); + + if (!PartialInformation) + return NULL; + + + /* now query MatchingDeviceId key */ + Status = ZwQueryValueKey(hSubKey, KeyName, KeyValuePartialInformation, PartialInformation, Length, &Length); + + /* check for success */ + if (!NT_SUCCESS(Status)) + { + ExFreePool(PartialInformation); + return NULL; + } + + if (PartialInformation->Type != REG_SZ) + { + /* invalid key type */ + ExFreePool(PartialInformation); + return NULL; + } + + return PartialInformation; +} + + +NTSTATUS +CompareProductName( + IN HANDLE hSubKey, + IN LPWSTR PnpName, + IN ULONG ProductNameSize, + OUT LPWSTR ProductName) +{ + PKEY_VALUE_PARTIAL_INFORMATION PartialInformation; + UNICODE_STRING DriverDescName = RTL_CONSTANT_STRING(L"DriverDesc"); + UNICODE_STRING MatchingDeviceIdName = RTL_CONSTANT_STRING(L"MatchingDeviceId"); + ULONG Length; + LPWSTR DeviceName; + + /* read MatchingDeviceId value */ + PartialInformation = ReadKeyValue(hSubKey, &MatchingDeviceIdName); + + if (!PartialInformation) + return STATUS_UNSUCCESSFUL; + + + /* extract last '&' */ + DeviceName = wcsrchr((LPWSTR)PartialInformation->Data, L'&'); + ASSERT(DeviceName); + /* terminate it */ + DeviceName[0] = L'\0'; + + Length = wcslen((LPWSTR)PartialInformation->Data); + + DPRINT("DeviceName %S PnpName %S Length %u\n", (LPWSTR)PartialInformation->Data, PnpName, Length); + + if (_wcsnicmp((LPWSTR)PartialInformation->Data, &PnpName[4], Length)) + { + ExFreePool(PartialInformation); + return STATUS_NO_MATCH; + } + + /* free buffer */ + ExFreePool(PartialInformation); + + /* read DriverDescName value */ + PartialInformation = ReadKeyValue(hSubKey, &DriverDescName); + + if (!PartialInformation) + { + /* failed to read driver desc key */ + return STATUS_UNSUCCESSFUL; + } + + /* copy key name */ + Length = min(ProductNameSize * sizeof(WCHAR), PartialInformation->DataLength); + RtlMoveMemory(ProductName, (PVOID)PartialInformation->Data, Length); + + /* zero terminate it */ + ProductName[ProductNameSize-1] = L'\0'; + + /* free buffer */ + ExFreePool(PartialInformation); + + return STATUS_SUCCESS; +} + + + +NTSTATUS +FindProductName( + IN LPWSTR PnpName, + IN ULONG ProductNameSize, + OUT LPWSTR ProductName) +{ + UNICODE_STRING KeyName = RTL_CONSTANT_STRING(L"\\Registry\\Machine\\System\\CurrentControlSet\\Control\\Class\\{4D36E96C-E325-11CE-BFC1-08002BE10318}"); + + UNICODE_STRING SubKeyName; + WCHAR SubKey[20]; + OBJECT_ATTRIBUTES ObjectAttributes; + HANDLE hKey, hSubKey; + NTSTATUS Status; + ULONG Length, Index; + PKEY_FULL_INFORMATION KeyInformation; + + for(Index = 0; Index < wcslen(PnpName); Index++) + { + if (PnpName[Index] == '#') + PnpName[Index] = L'\\'; + } + + + /* initialize key attributes */ + InitializeObjectAttributes(&ObjectAttributes, &KeyName, OBJ_CASE_INSENSITIVE | OBJ_OPENIF, NULL, NULL); + + /* open the key */ + Status = ZwOpenKey(&hKey, GENERIC_READ, &ObjectAttributes); + + /* check for success */ + if (!NT_SUCCESS(Status)) + return Status; + + /* query num of subkeys */ + Status = ZwQueryKey(hKey, KeyFullInformation, NULL, 0, &Length); + + if (Status != STATUS_BUFFER_TOO_SMALL) + { + DPRINT1("ZwQueryKey failed with %x\n", Status); + /* failed */ + ZwClose(hKey); + return Status; + } + + /* allocate key information struct */ + KeyInformation = ExAllocatePool(NonPagedPool, Length); + if (!KeyInformation) + { + /* no memory */ + ZwClose(hKey); + return STATUS_INSUFFICIENT_RESOURCES; + } + + /* query num of subkeys */ + Status = ZwQueryKey(hKey, KeyFullInformation, (PVOID)KeyInformation, Length, &Length); + + if (!NT_SUCCESS(Status)) + { + DPRINT1("ZwQueryKey failed with %x\n", Status); + ExFreePool(KeyInformation); + ZwClose(hKey); + return Status; + } + + /* now iterate through all subkeys */ + for(Index = 0; Index < KeyInformation->SubKeys; Index++) + { + /* subkeys are always in the format 0000-XXXX */ + swprintf(SubKey, L"%04u", Index); + + /* initialize subkey name */ + RtlInitUnicodeString(&SubKeyName, SubKey); + + /* initialize key attributes */ + InitializeObjectAttributes(&ObjectAttributes, &SubKeyName, OBJ_CASE_INSENSITIVE | OBJ_OPENIF, hKey, NULL); + + /* open the sub key */ + Status = ZwOpenKey(&hSubKey, GENERIC_READ, &ObjectAttributes); + + /* check for success */ + if (NT_SUCCESS(Status)) + { + /* compare product name */ + Status = CompareProductName(hSubKey, PnpName, ProductNameSize, ProductName); + + /* close subkey */ + ZwClose(hSubKey); + + if (NT_SUCCESS(Status)) + break; + } + } + + /* free buffer */ + ExFreePool(KeyInformation); + + /* close key */ + ZwClose(hKey); + + /* no matching key found */ + return Status; +} + Propchange: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/sup.c ------------------------------------------------------------------------------ svn:eol-style = native Added: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wave.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wave.c (added) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wave.c [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -1,0 +1,756 @@ +/* + * COPYRIGHT: See COPYING in the top level directory + * PROJECT: ReactOS Kernel Streaming + * FILE: drivers/wdm/audio/legacy/wdmaud/wave.c + * PURPOSE: Wave Out enumeration + * PROGRAMMER: Andrew Greenwood + * Johannes Anderwald + */ +#include "wdmaud.h" + + +typedef struct +{ + ULONG SampleRate; + ULONG Bit8Mono; + ULONG Bit8Stereo; + ULONG Bit16Mono; + ULONG Bit16Stereo; +}AUDIO_RANGE; + +#define AUDIO_TEST_RANGE (5) + +static AUDIO_RANGE TestRange[AUDIO_TEST_RANGE] = +{ + { + 11025, + WAVE_FORMAT_1M08, + WAVE_FORMAT_1S08, + WAVE_FORMAT_1M16, + WAVE_FORMAT_1S16 + }, + { + 22050, + WAVE_FORMAT_2M08, + WAVE_FORMAT_2S08, + WAVE_FORMAT_2M16, + WAVE_FORMAT_2S16 + }, + { + 44100, + WAVE_FORMAT_4M08, + WAVE_FORMAT_4S08, + WAVE_FORMAT_4M16, + WAVE_FORMAT_4S16 + }, + { + 48000, + WAVE_FORMAT_48M08, + WAVE_FORMAT_48S08, + WAVE_FORMAT_48M16, + WAVE_FORMAT_48S16 + }, + { + 96000, + WAVE_FORMAT_96M08, + WAVE_FORMAT_96S08, + WAVE_FORMAT_96M16, + WAVE_FORMAT_96S16 + } +}; + +LPWAVE_INFO +AllocateWaveInfo() +{ + /* allocate wav info */ + LPWAVE_INFO WaveOutInfo = ExAllocatePool(NonPagedPool, sizeof(WAVE_INFO)); + if (!WaveOutInfo) + return NULL; + + /* zero wave info struct */ + RtlZeroMemory(WaveOutInfo, sizeof(WAVE_INFO)); + + return WaveOutInfo; +} + +PKSPIN_CONNECT +AllocatePinConnect( + ULONG DataFormatSize) +{ + PKSPIN_CONNECT Connect = ExAllocatePool(NonPagedPool, sizeof(KSPIN_CONNECT) + DataFormatSize); + if (!Connect) + return NULL; + + /* zero pin connect struct */ + RtlZeroMemory(Connect, sizeof(KSPIN_CONNECT) + DataFormatSize); + + return Connect; +} + +NTSTATUS +GetWaveInfoByIndexAndType( + IN PDEVICE_OBJECT DeviceObject, + IN ULONG DeviceIndex, + IN SOUND_DEVICE_TYPE DeviceType, + OUT LPWAVE_INFO *OutWaveInfo) +{ + PWDMAUD_DEVICE_EXTENSION DeviceExtension; + ULONG Index = 0; + PLIST_ENTRY Entry, ListHead; + LPWAVE_INFO WaveInfo; + + /* get device extension */ + DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + if (DeviceType == WAVE_IN_DEVICE_TYPE) + ListHead = &DeviceExtension->WaveInList; + else + ListHead = &DeviceExtension->WaveOutList; + + /* get first entry */ + Entry = ListHead->Flink; + + while(Entry != ListHead) + { + WaveInfo = (LPWAVE_INFO)CONTAINING_RECORD(Entry, WAVE_INFO, Entry); + + if (Index == DeviceIndex) + { + *OutWaveInfo = WaveInfo; + return STATUS_SUCCESS; + } + Index++; + Entry = Entry->Flink; + } + + return STATUS_NOT_FOUND; +} + + +VOID +InitializePinConnect( + IN OUT PKSPIN_CONNECT PinConnect, + IN ULONG PinId) +{ + PinConnect->Interface.Set = KSINTERFACESETID_Standard; + PinConnect->Interface.Id = KSINTERFACE_STANDARD_STREAMING; + PinConnect->Interface.Flags = 0; + PinConnect->Medium.Set = KSMEDIUMSETID_Standard; + PinConnect->Medium.Id = KSMEDIUM_TYPE_ANYINSTANCE; + PinConnect->Medium.Flags = 0; + PinConnect->PinToHandle = NULL; + PinConnect->PinId = PinId; + PinConnect->Priority.PriorityClass = KSPRIORITY_NORMAL; + PinConnect->Priority.PrioritySubClass = 1; +} + +VOID +InitializeDataFormat( + IN PKSDATAFORMAT_WAVEFORMATEX DataFormat, + LPWAVEFORMATEX WaveFormatEx) +{ + + DataFormat->WaveFormatEx.wFormatTag = WaveFormatEx->wFormatTag; + DataFormat->WaveFormatEx.nChannels = WaveFormatEx->nChannels; + DataFormat->WaveFormatEx.nSamplesPerSec = WaveFormatEx->nSamplesPerSec; + DataFormat->WaveFormatEx.nBlockAlign = WaveFormatEx->nBlockAlign; + DataFormat->WaveFormatEx.nAvgBytesPerSec = WaveFormatEx->nAvgBytesPerSec; + DataFormat->WaveFormatEx.wBitsPerSample = WaveFormatEx->wBitsPerSample; + DataFormat->WaveFormatEx.cbSize = 0; + DataFormat->DataFormat.FormatSize = sizeof(KSDATAFORMAT) + sizeof(WAVEFORMATEX); + DataFormat->DataFormat.Flags = 0; + DataFormat->DataFormat.Reserved = 0; + DataFormat->DataFormat.MajorFormat = KSDATAFORMAT_TYPE_AUDIO; + + DataFormat->DataFormat.SubFormat = KSDATAFORMAT_SUBTYPE_PCM; + DataFormat->DataFormat.Specifier = KSDATAFORMAT_SPECIFIER_WAVEFORMATEX; + DataFormat->DataFormat.SampleSize = 4; +} + + +NTSTATUS +AttachToVirtualAudioDevice( + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension, + IN ULONG VirtualDeviceId) +{ + ULONG BytesReturned; + SYSAUDIO_INSTANCE_INFO InstanceInfo; + + /* setup property request */ + InstanceInfo.Property.Set = KSPROPSETID_Sysaudio; + InstanceInfo.Property.Id = KSPROPERTY_SYSAUDIO_INSTANCE_INFO; + InstanceInfo.Property.Flags = KSPROPERTY_TYPE_SET; + InstanceInfo.Flags = 0; + InstanceInfo.DeviceNumber = VirtualDeviceId; + + /* attach to virtual device */ + return KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&InstanceInfo, sizeof(SYSAUDIO_INSTANCE_INFO), NULL, 0, &BytesReturned); + +} + +NTSTATUS +GetAudioPinDataRanges( + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension, + IN ULONG FilterId, + IN ULONG PinId, + IN OUT PKSMULTIPLE_ITEM * OutMultipleItem) +{ + KSP_PIN PinProperty; + ULONG BytesReturned = 0; + NTSTATUS Status; + PKSMULTIPLE_ITEM MultipleItem; + + /* retrieve size of data ranges buffer */ + PinProperty.Reserved = FilterId; + PinProperty.PinId = PinId; + PinProperty.Property.Set = KSPROPSETID_Pin; + PinProperty.Property.Id = KSPROPERTY_PIN_DATARANGES; + PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)NULL, 0, &BytesReturned); + if (Status != STATUS_MORE_ENTRIES) + { + return Status; + } + + MultipleItem = ExAllocatePool(NonPagedPool, BytesReturned); + if (!MultipleItem) + { + /* not enough memory */ + return STATUS_INSUFFICIENT_RESOURCES; + } + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)MultipleItem, BytesReturned, &BytesReturned); + if (!NT_SUCCESS(Status)) + { + /* failed */ + ExFreePool(MultipleItem); + return Status; + } + + /* save result */ + *OutMultipleItem = MultipleItem; + return Status; +} + +NTSTATUS +FindAudioDataRange( + PKSMULTIPLE_ITEM MultipleItem, + PKSDATARANGE_AUDIO * OutDataRangeAudio) +{ + ULONG Index; + PKSDATARANGE_AUDIO DataRangeAudio; + PKSDATARANGE DataRange; + + DataRange = (PKSDATARANGE) (MultipleItem + 1); + for(Index = 0; Index < MultipleItem->Count; Index++) + { + if (DataRange->FormatSize == sizeof(KSDATARANGE_AUDIO)) + { + DataRangeAudio = (PKSDATARANGE_AUDIO)DataRange; + if (IsEqualGUIDAligned(&DataRangeAudio->DataRange.MajorFormat, &KSDATAFORMAT_TYPE_AUDIO) && + IsEqualGUIDAligned(&DataRangeAudio->DataRange.SubFormat, &KSDATAFORMAT_SUBTYPE_PCM) && + IsEqualGUIDAligned(&DataRangeAudio->DataRange.Specifier, &KSDATAFORMAT_SPECIFIER_WAVEFORMATEX)) + { + DPRINT("Min Sample %u Max Sample %u Min Bits %u Max Bits %u Max Channel %u\n", DataRangeAudio->MinimumSampleFrequency, DataRangeAudio->MaximumSampleFrequency, + DataRangeAudio->MinimumBitsPerSample, DataRangeAudio->MaximumBitsPerSample, DataRangeAudio->MaximumChannels); + *OutDataRangeAudio = DataRangeAudio; + return STATUS_SUCCESS; + } + } + } + return STATUS_UNSUCCESSFUL; +} + +NTSTATUS +NTAPI +OpenWavePin( + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension, + IN ULONG FilterId, + IN ULONG PinId, + IN LPWAVEFORMATEX WaveFormatEx, + IN ACCESS_MASK DesiredAccess, + OUT PHANDLE PinHandle) +{ + PKSPIN_CONNECT PinConnect; + PKSDATAFORMAT_WAVEFORMATEX DataFormat; + NTSTATUS Status; + + /* allocate pin connect */ + PinConnect = AllocatePinConnect(sizeof(KSDATAFORMAT_WAVEFORMATEX)); + if (!PinConnect) + { + /* no memory */ + return STATUS_INSUFFICIENT_RESOURCES; + } + + /* initialize pin connect struct */ + InitializePinConnect(PinConnect, PinId); + + /* get offset to dataformat */ + DataFormat = (PKSDATAFORMAT_WAVEFORMATEX) (PinConnect + 1); + /* initialize with requested wave format */ + InitializeDataFormat(DataFormat, WaveFormatEx); + + /* first attach to the virtual device */ + Status = AttachToVirtualAudioDevice(DeviceExtension, FilterId); + + if (!NT_SUCCESS(Status)) + { + /* failed */ + ExFreePool(PinConnect); + return Status; + } + + /* now create the pin */ + Status = KsCreatePin(DeviceExtension->hSysAudio, PinConnect, DesiredAccess, PinHandle); + + /* free create info */ + ExFreePool(PinConnect); + + return Status; +} + +ULONG +GetPinInstanceCount( + PWDMAUD_DEVICE_EXTENSION DeviceExtension, + ULONG FilterId, + ULONG PinId) +{ + KSP_PIN PinRequest; + KSPIN_CINSTANCES PinInstances; + ULONG BytesReturned; + NTSTATUS Status; + + /* query the instance count */ + PinRequest.Reserved = FilterId; + PinRequest.PinId = PinId; + PinRequest.Property.Set = KSPROPSETID_Pin; + PinRequest.Property.Flags = KSPROPERTY_TYPE_GET; + PinRequest.Property.Id = KSPROPERTY_PIN_CINSTANCES; + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinRequest, sizeof(KSP_PIN), (PVOID)&PinInstances, sizeof(KSPIN_CINSTANCES), &BytesReturned); + ASSERT(Status == STATUS_SUCCESS); + return PinInstances.CurrentCount; +} + + +NTSTATUS +CheckSampleFormat( + PWDMAUD_DEVICE_EXTENSION DeviceExtension, + LPWAVE_INFO WaveInfo, + ULONG SampleRate, + ULONG NumChannels, + ULONG BitsPerSample) +{ + NTSTATUS Status = STATUS_SUCCESS; +#if 0 + + WAVEFORMATEX WaveFormat; + HANDLE PinHandle; + + /* clear wave format */ + RtlZeroMemory(&WaveFormat, sizeof(WAVEFORMATEX)); + + WaveFormat.wFormatTag = WAVE_FORMAT_PCM; + WaveFormat.nChannels = NumChannels; + WaveFormat.nSamplesPerSec = SampleRate; + WaveFormat.nAvgBytesPerSec = SampleRate * NumChannels * (BitsPerSample/8); + WaveFormat.nBlockAlign = (NumChannels * BitsPerSample) / 8; + WaveFormat.wBitsPerSample = BitsPerSample; + WaveFormat.cbSize = sizeof(WAVEFORMATEX); + + Status = OpenWavePin(DeviceExtension, WaveInfo->FilterId, WaveInfo->PinId, &WaveFormat, GENERIC_READ | GENERIC_WRITE, &PinHandle); + + if (NT_SUCCESS(Status)) + { + /* success */ + ZwClose(PinHandle); + + while(GetPinInstanceCount(DeviceExtension, WaveInfo->FilterId, WaveInfo->PinId)) + KeStallExecutionProcessor(5); + } + + DPRINT("SampleRate %u BitsPerSample %u NumChannels %u Status %x bInput %u\n", SampleRate, BitsPerSample, NumChannels, Status, WaveInfo->bInput); +#endif + + + return Status; +} + + +NTSTATUS +CheckFormat( + PWDMAUD_DEVICE_EXTENSION DeviceExtension, + PKSDATARANGE_AUDIO DataRangeAudio, + LPWAVE_INFO WaveInfo) +{ + ULONG Index, SampleFrequency; + ULONG Result = 0; + NTSTATUS Status; + + for(Index = 0; Index < AUDIO_TEST_RANGE; Index++) + { + SampleFrequency = TestRange[Index].SampleRate; + + if (DataRangeAudio->MinimumSampleFrequency <= SampleFrequency && DataRangeAudio->MaximumSampleFrequency >= SampleFrequency) + { + /* the audio adapter supports the sample frequency */ + if (DataRangeAudio->MinimumBitsPerSample <= 8 && DataRangeAudio->MaximumBitsPerSample >= 8) + { + /* check if pin supports the sample rate in 8-Bit Mono */ + Status = CheckSampleFormat(DeviceExtension, WaveInfo, SampleFrequency, 1, 8); + if (NT_SUCCESS(Status)) + { + Result |= TestRange[Index].Bit8Mono; + } + + if (DataRangeAudio->MaximumChannels > 1) + { + /* check if pin supports the sample rate in 8-Bit Stereo */ + Status = CheckSampleFormat(DeviceExtension, WaveInfo, SampleFrequency, 2, 8); + if (NT_SUCCESS(Status)) + { + Result |= TestRange[Index].Bit8Stereo; + } + } + } + + if (DataRangeAudio->MinimumBitsPerSample <= 16 && DataRangeAudio->MaximumBitsPerSample >= 16) + { + /* check if pin supports the sample rate in 16-Bit Mono */ + Status = CheckSampleFormat(DeviceExtension, WaveInfo, SampleFrequency, 1, 16); + if (NT_SUCCESS(Status)) + { + Result |= TestRange[Index].Bit16Mono; + } + + if (DataRangeAudio->MaximumChannels > 1) + { + /* check if pin supports the sample rate in 16-Bit Stereo */ + Status = CheckSampleFormat(DeviceExtension, WaveInfo, SampleFrequency, 2, 16); + if (NT_SUCCESS(Status)) + { + Result |= TestRange[Index].Bit16Stereo; + } + } + } + } + } + + + if (WaveInfo->bInput) + WaveInfo->u.InCaps.dwFormats = Result; + else + WaveInfo->u.OutCaps.dwFormats = Result; + + DPRINT("Format %x bInput %u\n", Result, WaveInfo->bInput); + + + return STATUS_SUCCESS; +} + +NTSTATUS +InitializeWaveInfo( + IN PDEVICE_OBJECT DeviceObject, + IN ULONG FilterId, + IN ULONG PinId, + IN ULONG bInput) +{ + KSP_PIN PinProperty; + KSCOMPONENTID ComponentId; + NTSTATUS Status; + ULONG BytesReturned; + WCHAR DeviceName[MAX_PATH]; + PWDMAUD_DEVICE_EXTENSION DeviceExtension; + PKSMULTIPLE_ITEM MultipleItem; + PKSDATARANGE_AUDIO DataRangeAudio; + LPWAVE_INFO WaveInfo = AllocateWaveInfo(); + + + if (!WaveInfo) + return STATUS_INSUFFICIENT_RESOURCES; + + /* initialize wave info */ + WaveInfo->bInput = bInput; + WaveInfo->FilterId = FilterId; + WaveInfo->PinId = PinId; + + /* setup request to return component id */ + PinProperty.PinId = FilterId; + PinProperty.Property.Set = KSPROPSETID_Sysaudio; + PinProperty.Property.Id = KSPROPERTY_SYSAUDIO_COMPONENT_ID; + PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; + + /* get device extension */ + DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + /* query sysaudio for component id */ + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)&ComponentId, sizeof(KSCOMPONENTID), &BytesReturned); + if (NT_SUCCESS(Status)) + { + if (bInput) + { + WaveInfo->u.InCaps.wMid = ComponentId.Manufacturer.Data1 - 0xd5a47fa7; + WaveInfo->u.InCaps.vDriverVersion = MAKELONG(ComponentId.Version, ComponentId.Revision); + } + else + { + WaveInfo->u.OutCaps.wMid = ComponentId.Manufacturer.Data1 - 0xd5a47fa7; + WaveInfo->u.OutCaps.vDriverVersion = MAKELONG(ComponentId.Version, ComponentId.Revision); + } + } + else + { + /* set up something useful */ + if (bInput) + { + WaveInfo->u.InCaps.wMid = MM_MICROSOFT; + WaveInfo->u.InCaps.wPid = MM_PID_UNMAPPED; + WaveInfo->u.InCaps.vDriverVersion = 1; + } + else + { + WaveInfo->u.OutCaps.wMid = MM_MICROSOFT; + WaveInfo->u.OutCaps.wPid = MM_PID_UNMAPPED; + WaveInfo->u.OutCaps.vDriverVersion = 1; + } + } + + /* retrieve pnp base name */ + PinProperty.PinId = FilterId; + PinProperty.Property.Set = KSPROPSETID_Sysaudio; + PinProperty.Property.Id = KSPROPERTY_SYSAUDIO_DEVICE_INTERFACE_NAME; + PinProperty.Property.Flags = KSPROPERTY_TYPE_GET; + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&PinProperty, sizeof(KSP_PIN), (PVOID)DeviceName, sizeof(DeviceName), &BytesReturned); + if (NT_SUCCESS(Status)) + { + /* find product name */ + if (bInput) + Status = FindProductName(DeviceName, MAXPNAMELEN, WaveInfo->u.OutCaps.szPname); + else + Status = FindProductName(DeviceName, MAXPNAMELEN, WaveInfo->u.InCaps.szPname); + + /* check for success */ + if (!NT_SUCCESS(Status)) + { + if (bInput) + WaveInfo->u.OutCaps.szPname[0] = L'\0'; + else + WaveInfo->u.InCaps.szPname[0] = L'\0'; + } + } + + Status = GetAudioPinDataRanges(DeviceExtension, FilterId, PinId, &MultipleItem); + if (NT_SUCCESS(Status)) + { + /* find a audio data range */ + Status = FindAudioDataRange(MultipleItem, &DataRangeAudio); + + if (NT_SUCCESS(Status)) + { + if (bInput) + { + WaveInfo->u.InCaps.wChannels = DataRangeAudio->MaximumChannels; + } + else + { + WaveInfo->u.OutCaps.wChannels = DataRangeAudio->MaximumChannels; + } + CheckFormat(DeviceExtension, DataRangeAudio, WaveInfo); + } + ExFreePool(MultipleItem); + } + + if (bInput) + { + InsertTailList(&DeviceExtension->WaveInList, &WaveInfo->Entry); + DeviceExtension->WaveInDeviceCount++; + } + else + { + InsertTailList(&DeviceExtension->WaveOutList, &WaveInfo->Entry); + DeviceExtension->WaveOutDeviceCount++; + } + + + return STATUS_SUCCESS; +} + + +NTSTATUS +NTAPI +WdmAudWaveInitialize( + IN PDEVICE_OBJECT DeviceObject) +{ + KSP_PIN Pin; + ULONG Count, BytesReturned, Index, SubIndex, Result, NumPins; + NTSTATUS Status; + KSPIN_COMMUNICATION Communication; + KSPIN_DATAFLOW DataFlow; + PWDMAUD_DEVICE_EXTENSION DeviceExtension; + + Pin.Property.Set = KSPROPSETID_Sysaudio; + Pin.Property.Id = KSPROPERTY_SYSAUDIO_DEVICE_COUNT; + Pin.Property.Flags = KSPROPERTY_TYPE_GET; + + DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + /* set wave count to zero */ + DeviceExtension->WaveInDeviceCount = 0; + DeviceExtension->WaveOutDeviceCount = 0; + + /* intialize list head */ + InitializeListHead(&DeviceExtension->WaveInList); + InitializeListHead(&DeviceExtension->WaveOutList); + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSPROPERTY), (PVOID)&Count, sizeof(ULONG), &BytesReturned); + if (!NT_SUCCESS(Status)) + return STATUS_UNSUCCESSFUL; + + Result = 0; + for(Index = 0; Index < Count; Index++) + { + /* query number of pins */ + Pin.Reserved = Index; // see sysaudio + Pin.Property.Flags = KSPROPERTY_TYPE_GET; + Pin.Property.Set = KSPROPSETID_Pin; + Pin.Property.Id = KSPROPERTY_PIN_CTYPES; + Pin.PinId = 0; + + Status = KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&NumPins, sizeof(ULONG), &BytesReturned); + if (NT_SUCCESS(Status)) + { + /* enumerate now all pins */ + for(SubIndex = 0; SubIndex < NumPins; SubIndex++) + { + Pin.PinId = SubIndex; + Pin.Property.Id = KSPROPERTY_PIN_COMMUNICATION; + Communication = KSPIN_COMMUNICATION_NONE; + + /* get pin communication type */ + KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&Communication, sizeof(KSPIN_COMMUNICATION), &BytesReturned); + + Pin.Property.Id = KSPROPERTY_PIN_DATAFLOW; + DataFlow = 0; + + /* get pin dataflow type */ + KsSynchronousIoControlDevice(DeviceExtension->FileObject, KernelMode, IOCTL_KS_PROPERTY, (PVOID)&Pin, sizeof(KSP_PIN), (PVOID)&DataFlow, sizeof(KSPIN_DATAFLOW), &BytesReturned); + + if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_IN) + { + /* found a wave out device */ + InitializeWaveInfo(DeviceObject, Index, SubIndex, FALSE); + } + else if (Communication == KSPIN_COMMUNICATION_SINK && DataFlow == KSPIN_DATAFLOW_OUT) + { + /* found a wave in device */ + InitializeWaveInfo(DeviceObject, Index, SubIndex, TRUE); + } + } + } + } + + return STATUS_SUCCESS; +} + +NTSTATUS +WdmAudControlOpenWave( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo) +{ + PWDMAUD_DEVICE_EXTENSION DeviceExtension; + LPWAVE_INFO WaveInfo; + NTSTATUS Status; + ACCESS_MASK DesiredAccess = 0; + HANDLE PinHandle; + ULONG FreeIndex; + + if (DeviceInfo->u.WaveFormatEx.wFormatTag != WAVE_FORMAT_PCM) + { + DPRINT("FIXME: Only WAVE_FORMAT_PCM is supported RequestFormat %x\n", DeviceInfo->u.WaveFormatEx.wFormatTag); + return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); + } + + /* get device extension */ + DeviceExtension = (PWDMAUD_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + /* find destination wave */ + Status = GetWaveInfoByIndexAndType(DeviceObject, DeviceInfo->DeviceIndex, DeviceInfo->DeviceType, &WaveInfo); + if (!NT_SUCCESS(Status)) + { + /* failed to find wave info */ + DbgBreakPoint(); + return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); + } + + /* close pin handle which uses same virtual audio device id and pin id */ + FreeIndex = ClosePin(ClientInfo, WaveInfo->FilterId, WaveInfo->PinId, DeviceInfo->DeviceType); + + /* get desired access */ + if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) + { + DesiredAccess |= GENERIC_READ; + } + else if (DeviceInfo->DeviceType == WAVE_OUT_DEVICE_TYPE) + { + DesiredAccess |= GENERIC_WRITE; + } + + /* now try open the pin */ + Status = OpenWavePin(DeviceExtension, WaveInfo->FilterId, WaveInfo->PinId, &DeviceInfo->u.WaveFormatEx, DesiredAccess, &PinHandle); + + if (!NT_SUCCESS(Status)) + { + /* failed to open the pin */ + return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); + } + + /* store the handle */ + Status = InsertPinHandle(ClientInfo, WaveInfo->FilterId, WaveInfo->PinId, DeviceInfo->DeviceType, PinHandle, FreeIndex); + if (!NT_SUCCESS(Status)) + { + /* failed to insert handle */ + ZwClose(PinHandle); + return SetIrpIoStatus(Irp, STATUS_UNSUCCESSFUL, 0); + } + + /* store pin handle */ + DeviceInfo->hDevice = PinHandle; + return SetIrpIoStatus(Irp, STATUS_SUCCESS, sizeof(WDMAUD_DEVICE_INFO)); +} + +NTSTATUS +WdmAudWaveCapabilities( + IN PDEVICE_OBJECT DeviceObject, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo, + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension) +{ + LPWAVE_INFO WaveInfo; + NTSTATUS Status; + + /* find destination wave */ + Status = GetWaveInfoByIndexAndType(DeviceObject, DeviceInfo->DeviceIndex, DeviceInfo->DeviceType, &WaveInfo); + if (!NT_SUCCESS(Status)) + { + /* failed to find wave info */ + DbgBreakPoint(); + return STATUS_UNSUCCESSFUL; + } + + if (DeviceInfo->DeviceType == WAVE_IN_DEVICE_TYPE) + { + RtlMoveMemory(&DeviceInfo->u.WaveInCaps, &WaveInfo->u.InCaps, sizeof(WAVEINCAPSW)); + } + else + { + RtlMoveMemory(&DeviceInfo->u.WaveOutCaps, &WaveInfo->u.OutCaps, sizeof(WAVEOUTCAPSW)); + } + + return STATUS_SUCCESS; +} + Propchange: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wave.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.h [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.h [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -40,19 +40,31 @@ ULONG DeviceIndex; MIXERLINEW Line; LPMIXERCONTROLW LineControls; + PULONG NodeIds; }MIXERLINE_EXT, *LPMIXERLINE_EXT; typedef struct { - HANDLE hMixer; - PFILE_OBJECT MixerFileObject; - MIXERCAPSW MixCaps; LIST_ENTRY LineList; - + ULONG ControlId; }MIXER_INFO, *LPMIXER_INFO; + + +typedef struct +{ + LIST_ENTRY Entry; + ULONG FilterId; + ULONG PinId; + ULONG bInput; + union + { + WAVEOUTCAPSW OutCaps; + WAVEINCAPSW InCaps; + }u; +}WAVE_INFO, *LPWAVE_INFO; typedef struct @@ -77,6 +89,12 @@ ULONG MixerInfoCount; LPMIXER_INFO MixerInfo; + ULONG WaveInDeviceCount; + LIST_ENTRY WaveInList; + + ULONG WaveOutDeviceCount; + LIST_ENTRY WaveOutList; + }WDMAUD_DEVICE_EXTENSION, *PWDMAUD_DEVICE_EXTENSION; @@ -87,6 +105,16 @@ }CONTEXT_WRITE, *PCONTEXT_WRITE; NTSTATUS +NTAPI +OpenWavePin( + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension, + IN ULONG FilterId, + IN ULONG PinId, + IN LPWAVEFORMATEX WaveFormatEx, + IN ACCESS_MASK DesiredAccess, + OUT PHANDLE PinHandle); + +NTSTATUS WdmAudRegisterDeviceInterface( IN PDEVICE_OBJECT PhysicalDeviceObject, IN PWDMAUD_DEVICE_EXTENSION DeviceExtension); @@ -119,6 +147,14 @@ IN PIRP Irp, IN PWDMAUD_DEVICE_INFO DeviceInfo, IN PWDMAUD_CLIENT ClientInfo); + +NTSTATUS +WdmAudControlOpenWave( + IN PDEVICE_OBJECT DeviceObject, + IN PIRP Irp, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo); + ULONG GetNumOfMixerDevices( @@ -149,6 +185,13 @@ IN PWDMAUD_DEVICE_EXTENSION DeviceExtension); NTSTATUS +WdmAudWaveCapabilities( + IN PDEVICE_OBJECT DeviceObject, + IN PWDMAUD_DEVICE_INFO DeviceInfo, + IN PWDMAUD_CLIENT ClientInfo, + IN PWDMAUD_DEVICE_EXTENSION DeviceExtension); + +NTSTATUS NTAPI WdmAudFrameSize( IN PDEVICE_OBJECT DeviceObject, @@ -192,5 +235,26 @@ WdmAudMixerInitialize( IN PDEVICE_OBJECT DeviceObject); +NTSTATUS +NTAPI +WdmAudWaveInitialize( + IN PDEVICE_OBJECT DeviceObject); + +NTSTATUS +ClosePin( + IN PWDMAUD_CLIENT ClientInfo, + IN ULONG FilterId, + IN ULONG PinId, + IN SOUND_DEVICE_TYPE DeviceType); + +NTSTATUS +InsertPinHandle( + IN PWDMAUD_CLIENT ClientInfo, + IN ULONG FilterId, + IN ULONG PinId, + IN SOUND_DEVICE_TYPE DeviceType, + IN HANDLE PinHandle, + IN ULONG FreeIndex); + #endif Modified: trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.rbuild URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/wdm/audio/legacy/w… ============================================================================== --- trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.rbuild [iso-8859-1] (original) +++ trunk/reactos/drivers/wdm/audio/legacy/wdmaud/wdmaud.rbuild [iso-8859-1] Fri Oct 2 12:57:24 2009 @@ -7,9 +7,12 @@ <library>ntoskrnl</library> <library>ks</library> <library>pseh</library> + <library>hal</library> <file>control.c</file> <file>deviface.c</file> <file>entry.c</file> <file>mixer.c</file> + <file>wave.c</file> + <file>sup.c</file> <file>wdmaud.rc</file> </module>

1 0

[janderwald] 43253: - Check if NodeId is out of bounds - Return correct status code
by janderwald＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: janderwald Date: Fri Oct 2 12:53:36 2009 New Revision: 43253 URL: http://svn.reactos.org/svn/reactos?rev=43253&view=rev Log: - Check if NodeId is out of bounds - Return correct status code Modified: trunk/reactos/drivers/ksfilter/ks/topology.c Modified: trunk/reactos/drivers/ksfilter/ks/topology.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/ksfilter/ks/topolo… ============================================================================== --- trunk/reactos/drivers/ksfilter/ks/topology.c [iso-8859-1] (original) +++ trunk/reactos/drivers/ksfilter/ks/topology.c [iso-8859-1] Fri Oct 2 12:53:36 2009 @@ -241,6 +241,13 @@ case KSPROPERTY_TOPOLOGY_NAME: Node = (KSP_NODE*)Property; + if (Node->NodeId >= Topology->TopologyNodesCount) + { + Irp->IoStatus.Information = 0; + Status = STATUS_INVALID_PARAMETER; + break; + } + Status = RtlStringFromGUID(&Topology->TopologyNodesNames[Node->NodeId], &GuidString); if (!NT_SUCCESS(Status)) { @@ -254,6 +261,7 @@ if (!KeyName.Buffer) { Irp->IoStatus.Information = 0; + Status = STATUS_INSUFFICIENT_RESOURCES; break; }

1 0

[fireball] 43252: [fastfat_new] - Implement querying standard information (important for getting correct file size). - Substitute CcMap/copy/Unpin by CcCopyRead in FatReadBlocks. - Take offset into account (seek) in file read operation. Fixes always reading files from the beginning.
by fireball＠svn.reactos.org 02 Oct '09

02 Oct '09

Author: fireball Date: Fri Oct 2 12:49:57 2009 New Revision: 43252 URL: http://svn.reactos.org/svn/reactos?rev=43252&view=rev Log: [fastfat_new] - Implement querying standard information (important for getting correct file size). - Substitute CcMap/copy/Unpin by CcCopyRead in FatReadBlocks. - Take offset into account (seek) in file read operation. Fixes always reading files from the beginning. Modified: trunk/reactos/drivers/filesystems/fastfat_new/finfo.c trunk/reactos/drivers/filesystems/fastfat_new/fullfat.c trunk/reactos/drivers/filesystems/fastfat_new/rw.c Modified: trunk/reactos/drivers/filesystems/fastfat_new/finfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/finfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/finfo.c [iso-8859-1] Fri Oct 2 12:49:57 2009 @@ -13,6 +13,49 @@ /* FUNCTIONS ****************************************************************/ +VOID +NTAPI +FatiQueryStandardInformation(IN PFAT_IRP_CONTEXT IrpContext, + IN PFCB Fcb, + IN PFILE_OBJECT FileObject, + IN OUT PFILE_STANDARD_INFORMATION Buffer, + IN OUT PLONG Length) +{ + /* Zero the buffer */ + RtlZeroMemory(Buffer, sizeof(FILE_STANDARD_INFORMATION)); + + /* Deduct the written length */ + *Length -= sizeof(FILE_STANDARD_INFORMATION); + + Buffer->NumberOfLinks = 1; + Buffer->DeletePending = FALSE; // FIXME + + /* Check if it's a dir or a file */ + if (FatNodeType(Fcb) == FAT_NTC_FCB) + { + Buffer->Directory = FALSE; + + Buffer->EndOfFile.LowPart = Fcb->FatHandle->Filesize; + Buffer->AllocationSize = Buffer->EndOfFile; + DPRINT1("Filesize %d, chain length %d\n", Fcb->FatHandle->Filesize, Fcb->FatHandle->iChainLength); + } + else + { + Buffer->Directory = TRUE; + } +} + +VOID +NTAPI +FatiQueryInternalInformation(IN PFAT_IRP_CONTEXT IrpContext, + IN PFCB Fcb, + IN PFILE_OBJECT FileObject, + IN OUT PFILE_INTERNAL_INFORMATION Buffer, + IN OUT PLONG Length) +{ + UNIMPLEMENTED; +} + NTSTATUS NTAPI FatiQueryInformation(IN PFAT_IRP_CONTEXT IrpContext, @@ -25,6 +68,9 @@ PVCB Vcb; PFCB Fcb; PCCB Ccb; + LONG Length; + PVOID Buffer; + NTSTATUS Status = STATUS_SUCCESS; /* Get IRP stack location */ IrpSp = IoGetCurrentIrpStackLocation(Irp); @@ -32,19 +78,51 @@ /* Get the file object */ FileObject = IrpSp->FileObject; + /* Copy variables to something with shorter names */ InfoClass = IrpSp->Parameters.QueryFile.FileInformationClass; + Length = IrpSp->Parameters.QueryFile.Length; + Buffer = Irp->AssociatedIrp.SystemBuffer; - DPRINT1("FatCommonQueryInformation\n", 0); + DPRINT1("FatiQueryInformation\n", 0); DPRINT1("\tIrp = %08lx\n", Irp); - DPRINT1("\tLength = %08lx\n", IrpSp->Parameters.QueryFile.Length); + DPRINT1("\tLength = %08lx\n", Length); DPRINT1("\tFileInformationClass = %08lx\n", InfoClass); - DPRINT1("\tBuffer = %08lx\n", Irp->AssociatedIrp.SystemBuffer); + DPRINT1("\tBuffer = %08lx\n", Buffer); FileType = FatDecodeFileObject(FileObject, &Vcb, &Fcb, &Ccb); DPRINT1("Vcb %p, Fcb %p, Ccb %p, open type %d\n", Vcb, Fcb, Ccb, FileType); - return STATUS_SUCCESS; + // TODO: Acquire FCB locks + + switch (InfoClass) + { + case FileStandardInformation: + FatiQueryStandardInformation(IrpContext, Fcb, FileObject, Buffer, &Length); + break; + case FileInternalInformation: + FatiQueryInternalInformation(IrpContext, Fcb, FileObject, Buffer, &Length); + break; + default: + DPRINT1("Unimplemented information class %d requested\n", InfoClass); + Status = STATUS_INVALID_PARAMETER; + } + + /* Check for buffer overflow */ + if (Length < 0) + { + Status = STATUS_BUFFER_OVERFLOW; + Length = 0; + } + + /* Set IoStatus.Information to amount of filled bytes */ + Irp->IoStatus.Information = IrpSp->Parameters.QueryFile.Length - Length; + + // TODO: Release FCB locks + + /* Complete request and return status */ + FatCompleteRequest(IrpContext, Irp, Status); + return Status; } NTSTATUS Modified: trunk/reactos/drivers/filesystems/fastfat_new/fullfat.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/fullfat.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/fullfat.c [iso-8859-1] Fri Oct 2 12:49:57 2009 @@ -41,16 +41,17 @@ FatReadBlocks(FF_T_UINT8 *DestBuffer, FF_T_UINT32 SectorAddress, FF_T_UINT32 Count, void *pParam) { LARGE_INTEGER Offset; - PVOID Buffer; + //PVOID Buffer; PVCB Vcb = (PVCB)pParam; - PBCB Bcb; + //PBCB Bcb; ULONG SectorSize = 512; // FIXME: hardcoding 512 is bad + IO_STATUS_BLOCK IoSb; DPRINT("FatReadBlocks %p %d %d %p\n", DestBuffer, SectorAddress, Count, pParam); /* Calculate the offset */ Offset.QuadPart = Int32x32To64(SectorAddress, SectorSize); - +#if 0 if (!CcMapData(Vcb->StreamFileObject, &Offset, Count * SectorSize, @@ -68,6 +69,9 @@ /* Unpin unneeded data */ CcUnpinData(Bcb); +#else + CcCopyRead(Vcb->StreamFileObject, &Offset, Count * SectorSize, TRUE, DestBuffer, &IoSb); +#endif /* Return amount of read data in sectors */ return Count; Modified: trunk/reactos/drivers/filesystems/fastfat_new/rw.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat_new/rw.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat_new/rw.c [iso-8859-1] Fri Oct 2 12:49:57 2009 @@ -54,6 +54,10 @@ Buffer = FatMapUserBuffer(IrpContext->Irp); DPRINT1("Normal cached read, buffer %p\n"); + /* Set offset */ + FF_Seek(Fcb->FatHandle, ByteOffset.LowPart, FF_SEEK_SET); + + /* Read */ BytesRead = FF_Read(Fcb->FatHandle, NumberOfBytes, 1, Buffer); DPRINT1("Read %d bytes\n", BytesRead);

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs