Ros-diffs September 2017

ros-diffs@reactos.org

17 participants
284 discussions

[pschweitzer] 75840: [KMTESTS:MM] Tests for paging IO MDL for read operations. Equivalent to r75833.
by pschweitzer＠svn.reactos.org 15 Sep '17

15 Sep '17

Author: pschweitzer Date: Fri Sep 15 08:16:14 2017 New Revision: 75840 URL: http://svn.reactos.org/svn/reactos?rev=75840&view=rev Log: [KMTESTS:MM] Tests for paging IO MDL for read operations. Equivalent to r75833. Modified: trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c Modified: trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_mm/NtCreateS… ============================================================================== --- trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c [iso-8859-1] Fri Sep 15 08:16:14 2017 @@ -318,10 +318,18 @@ } else { + PMDL Mdl; + ok(Irp->AssociatedIrp.SystemBuffer == NULL, "A SystemBuffer was allocated!\n"); Buffer = MapAndLockUserBuffer(Irp, Length); ok(Buffer != NULL, "Null pointer!\n"); RtlFillMemory(Buffer, Length, 0xBA); + + Mdl = Irp->MdlAddress; + ok(Mdl != NULL, "Null pointer for MDL!\n"); + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) != 0, "MDL not locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) == 0, "MDL from non paged\n"); + ok((Irp->Flags & IRP_PAGING_IO) != 0, "Non paging IO\n"); Status = STATUS_SUCCESS; }

1 0

[mjansen] 75839: [LIBTIRPC] Do not add makefiles in the CMakeLists.txt
by mjansen＠svn.reactos.org 14 Sep '17

14 Sep '17

Author: mjansen Date: Thu Sep 14 20:31:53 2017 New Revision: 75839 URL: http://svn.reactos.org/svn/reactos?rev=75839&view=rev Log: [LIBTIRPC] Do not add makefiles in the CMakeLists.txt Modified: trunk/reactos/dll/3rdparty/libtirpc/CMakeLists.txt Modified: trunk/reactos/dll/3rdparty/libtirpc/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/3rdparty/libtirpc/CMak… ============================================================================== --- trunk/reactos/dll/3rdparty/libtirpc/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/dll/3rdparty/libtirpc/CMakeLists.txt [iso-8859-1] Thu Sep 14 20:31:53 2017 @@ -40,8 +40,8 @@ src/key_call.c src/key_prot_xdr.c src/libtirpc.def - src/makefile - src/Makefile.am + #src/makefile + #src/Makefile.am src/mt_misc.c src/netname.c src/netnamer.c

1 0

[pschweitzer] 75838: [RAPPS] Update French translation
by pschweitzer＠svn.reactos.org 14 Sep '17

14 Sep '17

Author: pschweitzer Date: Thu Sep 14 16:00:01 2017 New Revision: 75838 URL: http://svn.reactos.org/svn/reactos?rev=75838&view=rev Log: [RAPPS] Update French translation Modified: trunk/reactos/base/applications/rapps/lang/fr-FR.rc Modified: trunk/reactos/base/applications/rapps/lang/fr-FR.rc URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/la… ============================================================================== --- trunk/reactos/base/applications/rapps/lang/fr-FR.rc [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/lang/fr-FR.rc [iso-8859-1] Thu Sep 14 16:00:01 2017 @@ -152,13 +152,13 @@ STRINGTABLE BEGIN IDS_AINFO_VERSION "\nVersion : " - IDS_AINFO_AVAILABLEVERSION "\nAvailable Version: " + IDS_AINFO_AVAILABLEVERSION "\nVersion disponible : " IDS_AINFO_DESCRIPTION "\nDescription : " IDS_AINFO_SIZE "\nTaille : " IDS_AINFO_URLSITE "\nSite internet : " IDS_AINFO_LICENSE "\nLicence : " IDS_AINFO_URLDOWNLOAD "\nTÃ©lÃ©charger : " - IDS_AINFO_LANGUAGES "\nLanguages: " + IDS_AINFO_LANGUAGES "\nLangues : " END STRINGTABLE @@ -187,7 +187,7 @@ IDS_INSTALL "Installer" IDS_UNINSTALL "DÃ©sinstaller" IDS_MODIFY "Modifier" - IDS_APPS_COUNT "Nombre d'applications : %d; Selected: %d" + IDS_APPS_COUNT "Nombre d'applications : %d ; SÃ©lectionnÃ©es : %d" IDS_WELCOME_TITLE "Bienvenue dans ReactOS Applications Manager !\n\n" IDS_WELCOME_TEXT "Choisissez une catÃ©gorie Ã gauche, ensuite choisissez une application Ã installer ou dÃ©sinstaller.\nSite internet de ReactOS : " IDS_WELCOME_URL "http://www.reactos.org" @@ -200,28 +200,28 @@ IDS_APP_REG_REMOVE "Ãtes-vous sÃ»r de vouloir supprimer du registre les donnÃ©es du programme installÃ© ?" IDS_INFORMATION "Information" IDS_UNABLE_TO_DOWNLOAD "Impossible de tÃ©lÃ©charger le paquet : adresse introuvable !" - IDS_UNABLE_TO_DOWNLOAD2 "Unable to download the package! Check Internet Connection!" + IDS_UNABLE_TO_DOWNLOAD2 "Impossible de tÃ©lÃ©charger le paquet : vÃ©rifiez votre connexion Ã Internet !" IDS_UNABLE_TO_REMOVE "Impossible de supprimer du registre les donnÃ©es du programme !" - IDS_UNABLE_TO_INSTALL "Unable to open installer!" + IDS_UNABLE_TO_INSTALL "Impossible d'ouvrir l'installeur !" IDS_CERT_DOES_NOT_MATCH "La vÃ©rification du certificat SSL a Ã©chouÃ©." IDS_INTEG_CHECK_TITLE "VÃ©rification de l'intÃ©gritÃ© du paquet..." IDS_INTEG_CHECK_FAIL "Le contrÃ´le d'intÃ©gritÃ© du paquet a Ã©chouÃ©, il se peut qu'il ait Ã©tÃ© corrompu ou altÃ©rÃ© au cours du tÃ©lÃ©chargement. L'exÃ©cution du programme n'est pas recommandÃ©e." - IDS_INTERRUPTED_DOWNLOAD "The download was interrupted. Check connection to Internet." - IDS_UNABLE_TO_WRITE "Unable to write to disk. Disk may be at capacity." - IDS_SELECT_ALL "Select/Deselect All" - IDS_INSTALL_SELECTED "Install Selected" -END - -STRINGTABLE -BEGIN - IDS_STATUS_INSTALLED "Installed" - IDS_STATUS_NOTINSTALLED "Not installed" - IDS_STATUS_DOWNLOADED "Downloaded" - IDS_STATUS_UPDATE_AVAILABLE "Update available" - IDS_STATUS_DOWNLOADING "Downloadingâ¦" - IDS_STATUS_INSTALLING "Installingâ¦" - IDS_STATUS_WAITING "Waiting to installâ¦" - IDS_STATUS_FINISHED "Finished" + IDS_INTERRUPTED_DOWNLOAD "Le tÃ©lÃ©chargement a Ã©tÃ© interrompu. VÃ©rifiez votre connexion Ã Internet." + IDS_UNABLE_TO_WRITE "Impossible d'Ã©crire sur le disque. Votre disque pourrait Ãªtre plein." + IDS_SELECT_ALL "SÃ©lectionner/DÃ©sÃ©lectionner tout" + IDS_INSTALL_SELECTED "Installer les sÃ©lectionnÃ©s" +END + +STRINGTABLE +BEGIN + IDS_STATUS_INSTALLED "InstallÃ©" + IDS_STATUS_NOTINSTALLED "Non installÃ©" + IDS_STATUS_DOWNLOADED "TÃ©lÃ©chargÃ©" + IDS_STATUS_UPDATE_AVAILABLE "Mise Ã jour disponible" + IDS_STATUS_DOWNLOADING "TÃ©lÃ©chargementâ¦" + IDS_STATUS_INSTALLING "Installationâ¦" + IDS_STATUS_WAITING "En attente d'installationâ¦" + IDS_STATUS_FINISHED "TerminÃ©" END STRINGTABLE @@ -233,16 +233,16 @@ STRINGTABLE BEGIN - IDS_LANGUAGE_AVAILABLE_TRANSLATION "Supports your language" - IDS_LANGUAGE_NO_TRANSLATION "Supports other languages" - IDS_LANGUAGE_ENGLISH_TRANSLATION "Supports English" - IDS_LANGUAGE_SINGLE "Single language" + IDS_LANGUAGE_AVAILABLE_TRANSLATION "Supporte votre langue" + IDS_LANGUAGE_NO_TRANSLATION "Supporte d'autres langues" + IDS_LANGUAGE_ENGLISH_TRANSLATION "Supporte l'Anglais" + IDS_LANGUAGE_SINGLE "Langue unique" IDS_LANGUAGE_MORE_PLACEHOLDER " (+%d more)" IDS_LANGUAGE_AVAILABLE_PLACEHOLDER " (+%d available)" END STRINGTABLE BEGIN - IDS_DL_DIALOG_DB_DISP "Applications Database" - IDS_DL_DIALOG_DB_DOWNLOAD_DISP "Updating Database..." -END + IDS_DL_DIALOG_DB_DISP "Base de donnÃ©es d'applications" + IDS_DL_DIALOG_DB_DOWNLOAD_DISP "Mise Ã jour de la base de donnÃ©es..." +END

1 0

[ashaposhnikov] 75837: [RAPPS] Toggle toolbar captions on resize - Implemented hiding toolbar captions when buttons are about to intersect with the searchbar - Minor style changes
by ashaposhnikov＠svn.reactos.org 14 Sep '17

14 Sep '17

Author: ashaposhnikov Date: Thu Sep 14 15:56:19 2017 New Revision: 75837 URL: http://svn.reactos.org/svn/reactos?rev=75837&view=rev Log: [RAPPS] Toggle toolbar captions on resize - Implemented hiding toolbar captions when buttons are about to intersect with the searchbar - Minor style changes Modified: trunk/reactos/base/applications/rapps/gui.cpp Modified: trunk/reactos/base/applications/rapps/gui.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/rapps/gu… ============================================================================== --- trunk/reactos/base/applications/rapps/gui.cpp [iso-8859-1] (original) +++ trunk/reactos/base/applications/rapps/gui.cpp [iso-8859-1] Thu Sep 14 15:56:19 2017 @@ -209,7 +209,8 @@ class CMainToolbar : public CUiWindow< CToolbar<> > { -#define TOOLBAR_HEIGHT 24 + const INT m_iToolbarHeight; + DWORD m_dButtonsWidthMax; WCHAR szInstallBtn[MAX_STR_LEN]; WCHAR szUninstallBtn[MAX_STR_LEN]; @@ -223,8 +224,8 @@ if (!(hImage = (HICON) LoadImageW(hInst, MAKEINTRESOURCE(ImageIndex), IMAGE_ICON, - TOOLBAR_HEIGHT, - TOOLBAR_HEIGHT, + m_iToolbarHeight, + m_iToolbarHeight, 0))) { /* TODO: Error message */ @@ -239,8 +240,8 @@ HIMAGELIST hImageList; /* Create the toolbar icon image list */ - hImageList = ImageList_Create(TOOLBAR_HEIGHT,//GetSystemMetrics(SM_CXSMICON), - TOOLBAR_HEIGHT,//GetSystemMetrics(SM_CYSMICON), + hImageList = ImageList_Create(m_iToolbarHeight,//GetSystemMetrics(SM_CXSMICON), + m_iToolbarHeight,//GetSystemMetrics(SM_CYSMICON), ILC_MASK | GetSystemColorDepth(), 1, 1); if (!hImageList) @@ -262,6 +263,10 @@ } public: + CMainToolbar() : m_iToolbarHeight(24) + { + } + VOID OnGetDispInfo(LPTOOLTIPTEXT lpttt) { UINT idButton = (UINT) lpttt->hdr.idFrom; @@ -300,21 +305,19 @@ HWND Create(HWND hwndParent) { - HIMAGELIST hImageList; - - // buttons - static TBBUTTON Buttons[] = + /* Create buttons */ + TBBUTTON Buttons[] = { /* iBitmap, idCommand, fsState, fsStyle, bReserved[2], dwData, iString */ - { 0, ID_INSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szInstallBtn }, - { 1, ID_UNINSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szUninstallBtn }, - { 2, ID_MODIFY, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szModifyBtn }, - { 3, ID_CHECK_ALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szSelectAll}, - {-1, 0, TBSTATE_ENABLED, BTNS_SEP, { 0 }, 0, 0 }, - { 4, ID_REFRESH, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, - { 5, ID_RESETDB, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, - {-1, 0, TBSTATE_ENABLED, BTNS_SEP, { 0 }, 0, 0 }, - { 6, ID_SETTINGS, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, - { 7, ID_EXIT, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, + { 0, ID_INSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szInstallBtn }, + { 1, ID_UNINSTALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szUninstallBtn }, + { 2, ID_MODIFY, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szModifyBtn }, + { 3, ID_CHECK_ALL, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, (INT_PTR) szSelectAll }, + { -1, 0, TBSTATE_ENABLED, BTNS_SEP, { 0 }, 0, 0 }, + { 4, ID_REFRESH, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, + { 5, ID_RESETDB, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, + { -1, 0, TBSTATE_ENABLED, BTNS_SEP, { 0 }, 0, 0 }, + { 6, ID_SETTINGS, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, + { 7, ID_EXIT, TBSTATE_ENABLED, BTNS_BUTTON | BTNS_AUTOSIZE, { 0 }, 0, 0 }, }; LoadStringW(hInst, IDS_INSTALL, szInstallBtn, _countof(szInstallBtn)); @@ -337,7 +340,8 @@ SendMessageW(TB_SETEXTENDEDSTYLE, 0, TBSTYLE_EX_HIDECLIPPEDBUTTONS); SetButtonStructSize(); - hImageList = InitImageList(); + /* Set image list */ + HIMAGELIST hImageList = InitImageList(); if (!hImageList) { @@ -349,7 +353,29 @@ AddButtons(_countof(Buttons), Buttons); + /* Remember ideal width to use as a max width of buttons */ + SIZE size; + GetIdealSize(FALSE, &size); + m_dButtonsWidthMax = size.cx; + return m_hWnd; + } + + VOID HideButtonCaption() + { + DWORD dCurrentExStyle = (DWORD) SendMessageW(TB_GETEXTENDEDSTYLE, 0, 0); + SendMessageW(TB_SETEXTENDEDSTYLE, 0, dCurrentExStyle | TBSTYLE_EX_MIXEDBUTTONS); + } + + VOID ShowButtonCaption() + { + DWORD dCurrentExStyle = (DWORD) SendMessageW(TB_GETEXTENDEDSTYLE, 0, 0); + SendMessageW(TB_SETEXTENDEDSTYLE, 0, dCurrentExStyle & ~TBSTYLE_EX_MIXEDBUTTONS); + } + + DWORD GetMaxButtonsWidth() const + { + return m_dButtonsWidthMax; } }; @@ -616,6 +642,13 @@ public CWindow { public: + const INT m_Width; + const INT m_Height; + + CSearchBar() : m_Width(200), m_Height(22) + { + } + VOID SetText(LPCWSTR lpszText) { SendMessageW(SB_SETTEXT, SBT_NOBORDERS, (LPARAM) lpszText); @@ -626,7 +659,7 @@ ATL::CStringW szBuf; m_hWnd = CreateWindowExW(WS_EX_CLIENTEDGE, L"Edit", NULL, WS_CHILD | WS_VISIBLE | ES_LEFT | ES_AUTOHSCROLL, - 0, 0, 200, 22, + 0, 0, m_Width, m_Height, hwndParent, (HMENU) NULL, hInst, 0); @@ -661,6 +694,7 @@ BOOL bSearchEnabled; BOOL bUpdating; + public: CMainWindow() : m_ClientPanel(NULL), @@ -676,13 +710,13 @@ /* Add columns to ListView */ szText.LoadStringW(IDS_APP_NAME); - m_ListView->AddColumn(0, szText, 200, LVCFMT_LEFT); + m_ListView->AddColumn(0, szText, 250, LVCFMT_LEFT); szText.LoadStringW(IDS_APP_INST_VERSION); m_ListView->AddColumn(1, szText, 90, LVCFMT_RIGHT); szText.LoadStringW(IDS_APP_DESCRIPTION); - m_ListView->AddColumn(3, szText, 250, LVCFMT_LEFT); + m_ListView->AddColumn(3, szText, 300, LVCFMT_LEFT); // Unnesesary since the list updates on every TreeView selection // UpdateApplicationsList(ENUM_ALL_COMPONENTS); @@ -797,7 +831,7 @@ m_HSplitter->m_HorizontalAlignment = UiAlign_Stretch; m_HSplitter->m_DynamicFirst = TRUE; m_HSplitter->m_Horizontal = TRUE; - m_HSplitter->m_Pos = INT_MAX; //set INT_MAX to use lowest possible position (m_MinSecond) + m_HSplitter->m_Pos = INT_MAX; //set INT_MAX to use lowest possible position (m_MinSecond) m_HSplitter->m_MinFirst = 10; m_HSplitter->m_MinSecond = 140; m_VSplitter->Second().Append(m_HSplitter); @@ -810,7 +844,7 @@ m_SearchBar = new CUiWindow<CSearchBar>(); m_SearchBar->m_VerticalAlignment = UiAlign_LeftTop; m_SearchBar->m_HorizontalAlignment = UiAlign_RightBtm; - m_SearchBar->m_Margin.top = 6; + m_SearchBar->m_Margin.top = 4; m_SearchBar->m_Margin.right = 6; return m_SearchBar->Create(m_Toolbar->m_hWnd) != NULL; @@ -880,11 +914,25 @@ VOID OnSize(HWND hwnd, WPARAM wParam, LPARAM lParam) { + /* Size status bar */ m_StatusBar->SendMessage(WM_SIZE, 0, 0); /* Size tool bar */ m_Toolbar->AutoSize(); + + /* Automatically hide captions */ + DWORD dToolbarTreshold = m_Toolbar->GetMaxButtonsWidth(); + DWORD dSearchbarMargin = (LOWORD(lParam) - m_SearchBar->m_Width); + + if (dSearchbarMargin > dToolbarTreshold) + { + m_Toolbar->ShowButtonCaption(); + } + else if (dSearchbarMargin < dToolbarTreshold) + { + m_Toolbar->HideButtonCaption(); + } RECT r = {0, 0, LOWORD(lParam), HIWORD(lParam)}; HDWP hdwp = NULL; @@ -894,10 +942,11 @@ if (hdwp) { hdwp = m_ClientPanel->OnParentSize(r, hdwp); - } - if (hdwp) - { - EndDeferWindowPos(hdwp); + if (hdwp) + { + EndDeferWindowPos(hdwp); + } + } // TODO: Sub-layouts for children of children @@ -906,11 +955,12 @@ if (hdwp) { hdwp = m_SearchBar->OnParentSize(r, hdwp); - } - if (hdwp) - { - EndDeferWindowPos(hdwp); - } + if (hdwp) + { + EndDeferWindowPos(hdwp); + } + } + } BOOL ProcessWindowMessage(HWND hwnd, UINT Msg, WPARAM wParam, LPARAM lParam, LRESULT& theResult, DWORD dwMapId) @@ -920,7 +970,7 @@ { case WM_CREATE: if (!InitControls()) - ::PostMessage(hwnd, WM_CLOSE, 0, 0); + ::PostMessageW(hwnd, WM_CLOSE, 0, 0); break; case WM_DESTROY:

1 0

[pschweitzer] 75836: [BTRFS] Disable the MountMgr thread; given the current storage stack state, it's useless and can even cause troubles CORE-13769
by pschweitzer＠svn.reactos.org 14 Sep '17

14 Sep '17

Author: pschweitzer Date: Thu Sep 14 09:04:32 2017 New Revision: 75836 URL: http://svn.reactos.org/svn/reactos?rev=75836&view=rev Log: [BTRFS] Disable the MountMgr thread; given the current storage stack state, it's useless and can even cause troubles CORE-13769 Modified: trunk/reactos/drivers/filesystems/btrfs/btrfs.c Modified: trunk/reactos/drivers/filesystems/btrfs/btrfs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/btrfs/… ============================================================================== --- trunk/reactos/drivers/filesystems/btrfs/btrfs.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/btrfs/btrfs.c [iso-8859-1] Thu Sep 14 09:04:32 2017 @@ -5588,9 +5588,11 @@ KeInitializeEvent(&mountmgr_thread_event, NotificationEvent, FALSE); +#ifndef __REACTOS__ Status = PsCreateSystemThread(&mountmgr_thread_handle, 0, NULL, NULL, NULL, mountmgr_thread, NULL); if (!NT_SUCCESS(Status)) WARN("PsCreateSystemThread returned %08x\n", Status); +#endif IoRegisterFileSystem(DeviceObject);

1 0

[pschweitzer] 75835: [KMTESTS:MM] Few tests for MmBuildMdlForNonPagedPool() I wrote earlier. They all pass fine on ROS.
by pschweitzer＠svn.reactos.org 13 Sep '17

13 Sep '17

Author: pschweitzer Date: Wed Sep 13 16:38:32 2017 New Revision: 75835 URL: http://svn.reactos.org/svn/reactos?rev=75835&view=rev Log: [KMTESTS:MM] Few tests for MmBuildMdlForNonPagedPool() I wrote earlier. They all pass fine on ROS. Modified: trunk/rostests/kmtests/ntos_mm/MmMdl.c Modified: trunk/rostests/kmtests/ntos_mm/MmMdl.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_mm/MmMdl.c?r… ============================================================================== --- trunk/rostests/kmtests/ntos_mm/MmMdl.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/ntos_mm/MmMdl.c [iso-8859-1] Wed Sep 13 16:38:32 2017 @@ -188,7 +188,56 @@ } } +static +VOID +TestMmBuildMdlForNonPagedPool(VOID) +{ + PVOID Page; + PMDL Mdl; + + Page = ExAllocatePoolWithTag(PagedPool, PAGE_SIZE, 'Test'); + ok(Page != NULL, "ExAllocatePoolWithTag failed\n"); + if (skip(Page != NULL, "No buffer\n")) + return; + + Mdl = IoAllocateMdl(Page, PAGE_SIZE, FALSE, FALSE, NULL); + ok(Mdl != NULL, "IoAllocateMdl failed\n"); + if (skip(Mdl != NULL, "No MDL\n")) + return; + + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) == 0, "MDL locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) == 0, "MDL from non paged\n"); + + MmBuildMdlForNonPagedPool(Mdl); + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) == 0, "MDL locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) != 0, "MDL from paged\n"); + + IoFreeMdl(Mdl); + ExFreePoolWithTag(Page, 'Test'); + + Page = ExAllocatePoolWithTag(NonPagedPool, PAGE_SIZE, 'Test'); + ok(Page != NULL, "ExAllocatePoolWithTag failed\n"); + if (skip(Page != NULL, "No buffer\n")) + return; + + Mdl = IoAllocateMdl(Page, PAGE_SIZE, FALSE, FALSE, NULL); + ok(Mdl != NULL, "IoAllocateMdl failed\n"); + if (skip(Mdl != NULL, "No MDL\n")) + return; + + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) == 0, "MDL locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) == 0, "MDL from non paged\n"); + + MmBuildMdlForNonPagedPool(Mdl); + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) == 0, "MDL locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) != 0, "MDL from paged\n"); + + IoFreeMdl(Mdl); + ExFreePoolWithTag(Page, 'Test'); +} + START_TEST(MmMdl) { TestMmAllocatePagesForMdl(); + TestMmBuildMdlForNonPagedPool(); }

1 0

[pschweitzer] 75834: [NTOSKRNL] Don't call MmBuildMdlForNonPagedPool in CcWriteVirtualAddress(): - BaseAddress comes from PagedPood - Windows returns a locked MDL from paged pool - See tests from r...
by pschweitzer＠svn.reactos.org 13 Sep '17

13 Sep '17

Author: pschweitzer Date: Wed Sep 13 16:13:21 2017 New Revision: 75834 URL: http://svn.reactos.org/svn/reactos?rev=75834&view=rev Log: [NTOSKRNL] Don't call MmBuildMdlForNonPagedPool in CcWriteVirtualAddress(): - BaseAddress comes from PagedPood - Windows returns a locked MDL from paged pool - See tests from r75833 that are getting fixed with that revision This fixes assertions failures on BtrFS write. CORE-13763 CORE-13769 Modified: trunk/reactos/ntoskrnl/cc/copy.c Modified: trunk/reactos/ntoskrnl/cc/copy.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/cc/copy.c?rev=758… ============================================================================== --- trunk/reactos/ntoskrnl/cc/copy.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/cc/copy.c [iso-8859-1] Wed Sep 13 16:13:21 2017 @@ -147,14 +147,30 @@ { return STATUS_INSUFFICIENT_RESOURCES; } - MmBuildMdlForNonPagedPool(Mdl); - Mdl->MdlFlags |= MDL_IO_PAGE_READ; - KeInitializeEvent(&Event, NotificationEvent, FALSE); - Status = IoSynchronousPageWrite(Vacb->SharedCacheMap->FileObject, Mdl, &Vacb->FileOffset, &Event, &IoStatus); - if (Status == STATUS_PENDING) - { - KeWaitForSingleObject(&Event, Executive, KernelMode, FALSE, NULL); - Status = IoStatus.Status; + + Status = STATUS_SUCCESS; + _SEH2_TRY + { + MmProbeAndLockPages(Mdl, KernelMode, IoReadAccess); + } + _SEH2_EXCEPT (EXCEPTION_EXECUTE_HANDLER) + { + Status = _SEH2_GetExceptionCode(); + KeBugCheck(CACHE_MANAGER); + } _SEH2_END; + + if (NT_SUCCESS(Status)) + { + Mdl->MdlFlags |= MDL_IO_PAGE_READ; + KeInitializeEvent(&Event, NotificationEvent, FALSE); + Status = IoSynchronousPageWrite(Vacb->SharedCacheMap->FileObject, Mdl, &Vacb->FileOffset, &Event, &IoStatus); + if (Status == STATUS_PENDING) + { + KeWaitForSingleObject(&Event, Executive, KernelMode, FALSE, NULL); + Status = IoStatus.Status; + } + + MmUnlockPages(Mdl); } IoFreeMdl(Mdl); if (!NT_SUCCESS(Status) && (Status != STATUS_END_OF_FILE))

1 0

[pschweitzer] 75833: [KMTESTS:MM] Add a test that shows that MDLs issued by Cc for paging IOs are not in the correct state (nor correctly allocated) CORE-13769
by pschweitzer＠svn.reactos.org 13 Sep '17

13 Sep '17

Author: pschweitzer Date: Wed Sep 13 11:53:56 2017 New Revision: 75833 URL: http://svn.reactos.org/svn/reactos?rev=75833&view=rev Log: [KMTESTS:MM] Add a test that shows that MDLs issued by Cc for paging IOs are not in the correct state (nor correctly allocated) CORE-13769 Modified: trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c Modified: trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/ntos_mm/NtCreateS… ============================================================================== --- trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/ntos_mm/NtCreateSection_drv.c [iso-8859-1] Wed Sep 13 11:53:56 2017 @@ -381,6 +381,14 @@ } else { + PMDL Mdl; + + Mdl = Irp->MdlAddress; + ok(Mdl != NULL, "Null pointer for MDL!\n"); + ok((Mdl->MdlFlags & MDL_PAGES_LOCKED) != 0, "MDL not locked\n"); + ok((Mdl->MdlFlags & MDL_SOURCE_IS_NONPAGED_POOL) == 0, "MDL from non paged\n"); + ok((Irp->Flags & IRP_PAGING_IO) != 0, "Non paging IO\n"); + Status = STATUS_SUCCESS; }

1 0

[spetreolle] 75832: Stub CoGetInterceptorFromTypeInfo for Firefox. (lets xul.dll to load) CORE-12828
by spetreolle＠svn.reactos.org 12 Sep '17

12 Sep '17

Author: spetreolle Date: Tue Sep 12 20:25:59 2017 New Revision: 75832 URL: http://svn.reactos.org/svn/reactos?rev=75832&view=rev Log: Stub CoGetInterceptorFromTypeInfo for Firefox. (lets xul.dll to load) CORE-12828 Modified: trunk/reactos/dll/win32/ole32/ole32.spec Modified: trunk/reactos/dll/win32/ole32/ole32.spec URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/ole32/ole32.spec… ============================================================================== --- trunk/reactos/dll/win32/ole32/ole32.spec [iso-8859-1] (original) +++ trunk/reactos/dll/win32/ole32/ole32.spec [iso-8859-1] Tue Sep 12 20:25:59 2017 @@ -46,7 +46,7 @@ @ stdcall CoGetInstanceFromFile(ptr ptr ptr long long wstr long ptr) @ stdcall CoGetInstanceFromIStorage(ptr ptr ptr long ptr long ptr) @ stdcall -stub CoGetInterceptor(ptr ptr ptr ptr) -# CoGetInterceptorFromTypeInfo +@ stdcall -stub CoGetInterceptorFromTypeInfo(ptr ptr ptr ptr ptr) @ stdcall CoGetInterfaceAndReleaseStream(ptr ptr ptr) @ stdcall CoGetMalloc(long ptr) @ stdcall CoGetMarshalSizeMax(ptr ptr ptr long ptr long)

1 0

[dreimer] 75831: [RAPPS] ENOUGH! CORE-13666 #resolve #comment Fixed forever!
by dreimer＠svn.reactos.org 11 Sep '17

11 Sep '17

Author: dreimer Date: Mon Sep 11 19:20:32 2017 New Revision: 75831 URL: http://svn.reactos.org/svn/reactos?rev=75831&view=rev Log: [RAPPS] ENOUGH! CORE-13666 #resolve #comment Fixed forever! Removed: trunk/reactos/media/rapps/utorrent.txt Removed: trunk/reactos/media/rapps/utorrent.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/rapps/utorrent.txt?r… ============================================================================== --- trunk/reactos/media/rapps/utorrent.txt [iso-8859-1] (original) +++ trunk/reactos/media/rapps/utorrent.txt (removed) @@ -1,65 +0,0 @@ -[Section] -Name = ÂµTorrent -Version = CURRENT -License = Freeware for non-commercial uses -Description = Small and fast BitTorrent Client. -Size = 1.7 MiB -Category = 5 -URLSite = http://www.utorrent.com/ -URLDownload = http://download-new.utorrent.com/endpoint/utorrent/os/windows/track/stable/… -SHA1 = 8bbf8a007365253a75c1262720750b3379f4121f -CDPath = none - -[Section.0407] -License = Freeware fÃ¼r nichtkommerzielle Nutzung -Description = Kleiner und schneller BitTorrent Client. -Size = 1,7 MiB - -[Section.0a] -License = Gratuito para uso no comercial -Description = Un cliente BitTorrent pequeÃ±o y rÃ¡pido. - -[Section.040c] -License = Gratuit pour une utilisation non-commerciale -Description = Client BitTorrent petit et rapide. -Size = 1,7 Mio - -[Section.0410] -License = Gratuita per un uso non-commerciale -Description = Piccolo e rapido Client BitTorrent. - -[Section.0413] -License = Freeware voor niet-commerciÃ«el gebruik -Description = Kleine en snelle BitTorrent afnemer. - -[Section.0415] -License = Freeware (do uÅ¼ytku domowego) -Description = MaÅy i szybki klient BitTorrent. - -[Section.0418] -License = GratuitÄ pentru uz necomercial -Description = Client BitTorrent, mic Èi rapid. -Size = 1,7 Mio - -[Section.0419] -License = ÐÐµÑÐ¿Ð»Ð°ÑÐ½Ð°Ñ Ð´Ð»Ñ Ð½ÐµÐºÐ¾Ð¼Ð¼ÐµÑÑÐµÑÐºÐ¾Ð³Ð¾ Ð¸ÑÐ¿Ð¾Ð»ÑÐ·Ð¾Ð²Ð°Ð½Ð¸Ñ -Description = ÐÐ°Ð»ÐµÐ½ÑÐºÐ¸Ð¹ Ð¸ Ð±ÑÑÑÑÑÐ¹ ÐºÐ»Ð¸ÐµÐ½Ñ BitTorrent. -Size = 1,7 ÐÐ¸Ð - -[Section.041f] -License = Tecimlik olmayan kullanÄ±mlar iÃ§in Ã¼cretsiz. -Description = KÃ¼Ã§Ã¼k ve hÄ±zlÄ± BitTorrent istemcisi. -Size = 1,7 MiB - -[Section.0422] -License = ÐÐµÐ·Ð¿Ð»Ð°ÑÐ½Ð° Ð´Ð»Ñ Ð½ÐµÐºÐ¾Ð¼ÐµÑÑÑÐ¹Ð½Ð¾Ð³Ð¾ Ð²Ð¸ÐºÐ¾ÑÐ¸ÑÑÐ°Ð½Ð½Ñ -Description = ÐÐ°Ð»ÐµÐ½ÑÐºÐ¸Ð¹ Ñ ÑÐ²Ð¸Ð´ÐºÐ¸Ð¹ ÐºÐ»ÑÑÐ½Ñ BitTorrent. - -[Section.0804] -License = éåä¸ç¨éçåè´¹è½¯ä»¶ -Description = å°èå¿«ç BitTorrent å®¢æ·ç«¯ã - -[Section.0813] -License = Freeware voor niet-commerciÃ«el gebruik -Description = Kleine en snelle BitTorrent afnemer. -

1 0

← Newer
1
...
15
16
17
18
19
20
21
...
29
Older →

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 September 2017