Ros-diffs October 2014

ros-diffs@reactos.org

19 participants
730 discussions

[pschweitzer] 65042: [FASTFAT] Add sanity checks in VfatSetRenameInformation() to make sure we don't leak any FCB reference

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Mon Oct 27 13:38:14 2014 New Revision: 65042 URL: http://svn.reactos.org/svn/reactos?rev=65042&view=rev Log: [FASTFAT] Add sanity checks in VfatSetRenameInformation() to make sure we don't leak any FCB reference Modified: trunk/reactos/drivers/filesystems/fastfat/finfo.c Modified: trunk/reactos/drivers/filesystems/fastfat/finfo.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/finfo.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/finfo.c [iso-8859-1] Mon Oct 27 13:38:14 2014 @@ -473,6 +473,8 @@ OBJECT_ATTRIBUTES ObjectAttributes; HANDLE TargetHandle; BOOLEAN DeletedTarget; + ULONG OldReferences, NewReferences; + PVFATFCB OldParent; DPRINT("VfatSetRenameInfo(%p, %p, %p, %p, %p)\n", FileObject, FCB, DeviceExt, RenameInfo, TargetFileObject); @@ -481,6 +483,8 @@ { return STATUS_INVALID_PARAMETER; } + + OldReferences = FCB->parentFcb->RefCount; /* If we are performing relative opening for rename, get FO for getting FCB and path name */ if (RenameInfo->RootDirectory != NULL) @@ -686,6 +690,7 @@ if (FsRtlAreNamesEqual(&SourceFile, &NewFile, FALSE, NULL)) { Status = STATUS_SUCCESS; + ASSERT(OldReferences == FCB->parentFcb->RefCount); goto Cleanup; } @@ -729,6 +734,8 @@ &DeletedTarget); if (!NT_SUCCESS(Status)) { + ASSERT(OldReferences == FCB->parentFcb->RefCount - 1); + ASSERT(OldReferences == ParentFCB->RefCount - 1); goto Cleanup; } @@ -773,11 +780,16 @@ } } } + + ASSERT(OldReferences == FCB->parentFcb->RefCount - 1); // extra grab + ASSERT(OldReferences == ParentFCB->RefCount - 1); // extra grab } else { + /* Try to find target */ ParentFCB = NULL; + OldParent = FCB->parentFcb; Status = vfatPrepareTargetForRename(DeviceExt, &ParentFCB, &NewName, @@ -786,8 +798,11 @@ &DeletedTarget); if (!NT_SUCCESS(Status)) { + ASSERT(OldReferences == FCB->parentFcb->RefCount); goto Cleanup; } + + NewReferences = ParentFCB->RefCount; FsRtlNotifyFullReportChange(DeviceExt->NotifySync, &(DeviceExt->NotifyList), @@ -831,6 +846,8 @@ } } + ASSERT(OldReferences == OldParent->RefCount + 1); // removed file + ASSERT(NewReferences == ParentFCB->RefCount - 1); // new file Cleanup: if (ParentFCB != NULL) vfatReleaseFCB(DeviceExt, ParentFCB); if (NewName.Buffer != NULL) ExFreePoolWithTag(NewName.Buffer, TAG_VFAT);

9 years, 10 months

[pschweitzer] 65041: [NTFS] Bugfixing... Part 10/X: - Properly compute entry name length in CompareFileName() - Also, in CompareFileName() properly handle the return of RtlCompareUnicodeString(); t...

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Mon Oct 27 12:35:58 2014 New Revision: 65041 URL: http://svn.reactos.org/svn/reactos?rev=65041&view=rev Log: [NTFS] Bugfixing... Part 10/X: - Properly compute entry name length in CompareFileName() - Also, in CompareFileName() properly handle the return of RtlCompareUnicodeString(); this is not RtlEqualUnicodeString()! - In NtfsLookupFileAt(), don't return an error when we're done walking the path, it's a normal behavior All these fixes allow our NTFS to go one step farther: it can open directory/files (reading files data remains untested so far) in root and in its subdirs. Which was broken previously. The said bugfixes in action (and in image): http://www.heisspiter.net/~Pierre/rostests/NTFS_listing_subdir.png Modified: trunk/reactos/drivers/filesystems/ntfs/mft.c Modified: trunk/reactos/drivers/filesystems/ntfs/mft.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/ntfs/m… ============================================================================== --- trunk/reactos/drivers/filesystems/ntfs/mft.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/ntfs/mft.c [iso-8859-1] Mon Oct 27 12:35:58 2014 @@ -469,7 +469,7 @@ EntryName.Buffer = IndexEntry->FileName.Name; EntryName.Length = - EntryName.MaximumLength = IndexEntry->FileName.NameLength; + EntryName.MaximumLength = IndexEntry->FileName.NameLength * sizeof(WCHAR); if (DirSearch) { @@ -477,7 +477,7 @@ } else { - return (RtlCompareUnicodeString(FileName, &EntryName, (IndexEntry->FileName.NameType != NTFS_FILE_NAME_POSIX)) == TRUE); + return (RtlCompareUnicodeString(FileName, &EntryName, (IndexEntry->FileName.NameType != NTFS_FILE_NAME_POSIX)) == 0); } } @@ -702,7 +702,7 @@ } if (Remaining.Length == 0) - return STATUS_OBJECT_PATH_NOT_FOUND; + break; FsRtlDissectName(Current, &Current, &Remaining); }

9 years, 10 months

[tfaber] 65040: [ADVAPI32] - Hackfix CORE-8717 to stop 2nd stage from failing. Proper fix will be submitted to Wine.

by tfaber＠svn.reactos.org

Author: tfaber Date: Mon Oct 27 11:52:44 2014 New Revision: 65040 URL: http://svn.reactos.org/svn/reactos?rev=65040&view=rev Log: [ADVAPI32] - Hackfix CORE-8717 to stop 2nd stage from failing. Proper fix will be submitted to Wine. Modified: trunk/reactos/dll/win32/advapi32/wine/security.c Modified: trunk/reactos/dll/win32/advapi32/wine/security.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/advapi32/wine/se… ============================================================================== --- trunk/reactos/dll/win32/advapi32/wine/security.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/advapi32/wine/security.c [iso-8859-1] Mon Oct 27 11:52:44 2014 @@ -3533,6 +3533,9 @@ while (*StringSid == ' ') StringSid++; + if (!*StringSid) + goto lend; /* ERROR_INVALID_SID */ + *cBytes = ComputeStringSidSize(StringSid); if (!pisid) /* Simply compute the size */ {

9 years, 10 months

[jimtabor] 65039: [NtUser] - Fix default sizes for desktop resources. See CORE-8659.

by jimtabor＠svn.reactos.org

Author: jimtabor Date: Mon Oct 27 02:11:08 2014 New Revision: 65039 URL: http://svn.reactos.org/svn/reactos?rev=65039&view=rev Log: [NtUser] - Fix default sizes for desktop resources. See CORE-8659. Modified: trunk/reactos/win32ss/user/ntuser/desktop.c Modified: trunk/reactos/win32ss/user/ntuser/desktop.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/deskto… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/desktop.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/desktop.c [iso-8859-1] Mon Oct 27 02:11:08 2014 @@ -24,6 +24,10 @@ IntFreeDesktopHeap(IN PDESKTOP pdesk); /* GLOBALS *******************************************************************/ + +/* These can be changed via csrss startup, these are defaults */ +DWORD gdwDesktopSectionSize = 512; +DWORD gdwNOIOSectionSize = 128; // A guess, for one or more of the first three system desktops. /* Currently active desktop */ PDESKTOP gpdeskInputDesktop = NULL; @@ -1194,7 +1198,7 @@ UserInitializeDesktop(PDESKTOP pdesk, PUNICODE_STRING DesktopName, PWINSTATION_OBJECT pwinsta) { PVOID DesktopHeapSystemBase = NULL; - ULONG_PTR HeapSize = 400 * 1024; + ULONG_PTR HeapSize = gdwDesktopSectionSize * 1024; SIZE_T DesktopInfoSize; ULONG i;

9 years, 10 months

[gadamopoulos] 65038: [BEOWSEUI] - Simplify shellbrowser.cpp a bit

by gadamopoulos＠svn.reactos.org

Author: gadamopoulos Date: Mon Oct 27 00:39:53 2014 New Revision: 65038 URL: http://svn.reactos.org/svn/reactos?rev=65038&view=rev Log: [BEOWSEUI] - Simplify shellbrowser.cpp a bit Modified: branches/shell-experiments/dll/win32/browseui/shellbrowser.cpp Modified: branches/shell-experiments/dll/win32/browseui/shellbrowser.cpp URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/bro… ============================================================================== --- branches/shell-experiments/dll/win32/browseui/shellbrowser.cpp [iso-8859-1] (original) +++ branches/shell-experiments/dll/win32/browseui/shellbrowser.cpp [iso-8859-1] Mon Oct 27 00:39:53 2014 @@ -1097,7 +1097,6 @@ HRESULT CShellBrowser::GetMenuBand(REFIID riid, void **shellMenu) { - CComPtr<IServiceProvider> serviceProvider; CComPtr<IBandSite> bandSite; CComPtr<IDeskBand> deskBand; HRESULT hResult; @@ -1121,7 +1120,6 @@ CComPtr<IUnknown> newBaseBar; CComPtr<IDeskBar> deskBar; CComPtr<IUnknown> newBaseBarSite; - CComPtr<IObjectWithSite> objectWithSite; CComPtr<IDeskBarClient> deskBarClient; IUnknown **cache; HRESULT hResult; @@ -1140,10 +1138,7 @@ return hResult; // tell the new base bar about the shell browser - hResult = newBaseBar->QueryInterface(IID_PPV_ARG(IObjectWithSite, &objectWithSite)); - if (FAILED_UNEXPECTEDLY(hResult)) - return hResult; - hResult = objectWithSite->SetSite(static_cast<IDropTarget *>(this)); + hResult = IUnknown_SetSite(newBaseBar, static_cast<IDropTarget *>(this)); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; @@ -1171,7 +1166,6 @@ HRESULT CShellBrowser::ShowBand(const CLSID &classID, bool vertical) { CComPtr<IDockingWindow> dockingWindow; - CComPtr<IOleCommandTarget> oleCommandTarget; CComPtr<IUnknown> baseBarSite; CComPtr<IUnknown> newBand; CComPtr<IUnknown> theBaseBar; @@ -1194,12 +1188,9 @@ hResult = theBaseBar->QueryInterface(IID_PPV_ARG(IDockingWindow, &dockingWindow)); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; - hResult = baseBarSite->QueryInterface(IID_PPV_ARG(IOleCommandTarget, &oleCommandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return hResult; V_VT(&vaIn) = VT_UNKNOWN; V_UNKNOWN(&vaIn) = newBand.p; - hResult = oleCommandTarget->Exec(&CGID_IDeskBand, 1, 1, &vaIn, NULL); + hResult = IUnknown_Exec(baseBarSite, CGID_IDeskBand, 1, 1, &vaIn, NULL); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; hResult = dockingWindow->ShowDW(TRUE); @@ -1210,14 +1201,11 @@ HRESULT CShellBrowser::NavigateToParent() { - LPITEMIDLIST newDirectory; - HRESULT hResult; - - newDirectory = ILClone(fCurrentDirectoryPIDL); + LPITEMIDLIST newDirectory = ILClone(fCurrentDirectoryPIDL); if (newDirectory == NULL) return E_OUTOFMEMORY; ILRemoveLastID(newDirectory); - hResult = BrowseToPIDL(newDirectory, BTP_UPDATE_CUR_HISTORY | BTP_UPDATE_NEXT_HISTORY); + HRESULT hResult = BrowseToPIDL(newDirectory, BTP_UPDATE_CUR_HISTORY | BTP_UPDATE_NEXT_HISTORY); ILFree(newDirectory); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; @@ -1226,9 +1214,7 @@ BOOL CALLBACK AddFolderOptionsPage(HPROPSHEETPAGE thePage, LPARAM lParam) { - PROPSHEETHEADER *sheetInfo; - - sheetInfo = (PROPSHEETHEADER *)lParam; + PROPSHEETHEADER* sheetInfo = reinterpret_cast<PROPSHEETHEADER*>(lParam); if (sheetInfo->nPages >= folderOptionsPageCountMax) return FALSE; sheetInfo->phpage[sheetInfo->nPages] = thePage; @@ -1239,7 +1225,6 @@ HRESULT CShellBrowser::DoFolderOptions() { CComPtr<IShellPropSheetExt> folderOptionsSheet; - CComPtr<IObjectWithSite> objectWithSite; PROPSHEETHEADER m_PropSheet; HPROPSHEETPAGE m_psp[folderOptionsPageCountMax]; // CComPtr<IGlobalFolderSettings> globalSettings; @@ -1256,9 +1241,7 @@ return E_FAIL; // must set site in order for Apply to all Folders on Advanced page to be enabled - hResult = folderOptionsSheet->QueryInterface(IID_PPV_ARG(IObjectWithSite, &objectWithSite)); - if (SUCCEEDED(hResult) && objectWithSite.p != NULL) - hResult = objectWithSite->SetSite(static_cast<IDispatch *>(this)); + hResult = IUnknown_SetSite(folderOptionsSheet, static_cast<IDispatch *>(this)); m_PropSheet.phpage = m_psp; #if 0 @@ -1553,7 +1536,6 @@ void CShellBrowser::UpdateViewMenu(HMENU theMenu) { - CComPtr<IOleCommandTarget> oleCommandTarget; CComPtr<ITravelLog> travelLog; HMENU gotoMenu; OLECMD commandList[5]; @@ -1572,10 +1554,8 @@ commandList[3].cmdID = ITID_TOOLBARLOCKED; commandList[4].cmdID = ITID_CUSTOMIZEENABLED; - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &oleCommandTarget)); - if (SUCCEEDED(hResult)) - hResult = oleCommandTarget->QueryStatus(&CGID_PrivCITCommands, 5, commandList, NULL); + hResult = IUnknown_QueryStatus(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, 5, commandList, NULL); if (FAILED_UNEXPECTEDLY(hResult)) DeleteMenu(theMenu, IDM_VIEW_TOOLBARS, MF_BYCOMMAND); else @@ -2589,10 +2569,8 @@ HRESULT STDMETHODCALLTYPE CShellBrowser::GoBack() { - CComPtr<ITravelLog> travelLog; - HRESULT hResult; - - hResult = GetTravelLog(&travelLog); + CComPtr<ITravelLog> travelLog; + HRESULT hResult = GetTravelLog(&travelLog); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; return travelLog->Travel(static_cast<IDropTarget *>(this), TLOG_BACK); @@ -2600,10 +2578,8 @@ HRESULT STDMETHODCALLTYPE CShellBrowser::GoForward() { - CComPtr<ITravelLog> travelLog; - HRESULT hResult; - - hResult = GetTravelLog(&travelLog); + CComPtr<ITravelLog> travelLog; + HRESULT hResult = GetTravelLog(&travelLog); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; return travelLog->Travel(static_cast<IDropTarget *>(this), TLOG_FORE); @@ -2842,16 +2818,14 @@ HRESULT STDMETHODCALLTYPE CShellBrowser::Navigate2(VARIANT *URL, VARIANT *Flags, VARIANT *TargetFrameName, VARIANT *PostData, VARIANT *Headers) { - LPITEMIDLIST pidl; - HRESULT hResult; - // called from drive combo box to navigate to a directory if (V_VT(URL) != (VT_ARRAY | VT_UI1)) return E_INVALIDARG; if (V_ARRAY(URL)->cDims != 1 || V_ARRAY(URL)->cbElements != 1) return E_INVALIDARG; - pidl = (LPITEMIDLIST)V_ARRAY(URL)->pvData; - hResult = BrowseToPIDL((LPITEMIDLIST)pidl, BTP_UPDATE_CUR_HISTORY | BTP_UPDATE_NEXT_HISTORY); + + LPITEMIDLIST pidl = static_cast<LPITEMIDLIST>(V_ARRAY(URL)->pvData); + HRESULT hResult = BrowseToPIDL(pidl, BTP_UPDATE_CUR_HISTORY | BTP_UPDATE_NEXT_HISTORY); if (FAILED_UNEXPECTEDLY(hResult)) return hResult; return S_OK; @@ -3195,9 +3169,7 @@ LRESULT CShellBrowser::OnFolderOptions(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - hResult = DoFolderOptions(); + HRESULT hResult = DoFolderOptions(); return 0; } @@ -3225,63 +3197,44 @@ LRESULT CShellBrowser::OnGoBack(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - hResult = GoBack(); + HRESULT hResult = GoBack(); return 0; } LRESULT CShellBrowser::OnGoForward(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - hResult = GoForward(); + HRESULT hResult = GoForward(); return 0; } LRESULT CShellBrowser::OnGoUpLevel(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - hResult = NavigateToParent(); + HRESULT hResult = NavigateToParent(); return 0; } LRESULT CShellBrowser::OnGoHome(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - hResult = GoHome(); + HRESULT hResult = GoHome(); return 0; } LRESULT CShellBrowser::OnBackspace(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - // FIXME: This does not appear to be what windows does. - hResult = NavigateToParent(); + HRESULT hResult = NavigateToParent(); return 0; } +HRESULT WINAPI SHOpenNewFrame(LPITEMIDLIST pidl, IUnknown *paramC, long param10, DWORD dwFlags); + LRESULT CShellBrowser::OnIsThisLegal(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - HRESULT hResult; - - typedef HRESULT (WINAPI *PSHOpenNewFrame)(LPITEMIDLIST pidl, IUnknown *b, long c, long d); - PSHOpenNewFrame Func = NULL; - HMODULE Module = GetModuleHandle(TEXT("browseui.dll")); - if (Module != NULL) - Func = reinterpret_cast<PSHOpenNewFrame>(GetProcAddress(Module, (LPCSTR) 103)); - if (Func != NULL) - { - LPITEMIDLIST desktopPIDL; - - hResult = SHGetFolderLocation(NULL, CSIDL_DESKTOP, NULL, 0, &desktopPIDL); - if (SUCCEEDED(hResult)) - { - hResult = Func(desktopPIDL, NULL, -1, 1); - } + LPITEMIDLIST desktopPIDL; + HRESULT hResult = SHGetFolderLocation(NULL, CSIDL_DESKTOP, NULL, 0, &desktopPIDL); + if (SUCCEEDED(hResult)) + { + hResult = SHOpenNewFrame(desktopPIDL, NULL, -1, 1); } return 0; } @@ -3295,79 +3248,49 @@ LRESULT CShellBrowser::OnToggleToolbarLock(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_TOOLBARLOCKED, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_TOOLBARLOCKED, 0, NULL, NULL); return 0; } LRESULT CShellBrowser::OnToggleToolbarBandVisible(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_TOOLBARBANDSHOWN, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_TOOLBARBANDSHOWN, 0, NULL, NULL); return 0; } LRESULT CShellBrowser::OnToggleAddressBandVisible(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_ADDRESSBANDSHOWN, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_ADDRESSBANDSHOWN, 0, NULL, NULL); return 0; } LRESULT CShellBrowser::OnToggleLinksBandVisible(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_LINKSBANDSHOWN, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_LINKSBANDSHOWN, 0, NULL, NULL); return 0; } LRESULT CShellBrowser::OnToggleTextLabels(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_TEXTLABELS, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_TEXTLABELS, 0, NULL, NULL); return 0; } LRESULT CShellBrowser::OnToolbarCustomize(WORD wNotifyCode, WORD wID, HWND hWndCtl, BOOL &bHandled) { - CComPtr<IOleCommandTarget> commandTarget; - HRESULT hResult; - - hResult = fClientBars[BIInternetToolbar].clientBar->QueryInterface( - IID_PPV_ARG(IOleCommandTarget, &commandTarget)); - if (FAILED_UNEXPECTEDLY(hResult)) - return 0; - hResult = commandTarget->Exec(&CGID_PrivCITCommands, ITID_CUSTOMIZEENABLED, 0, NULL, NULL); + HRESULT hResult; + hResult = IUnknown_Exec(fClientBars[BIInternetToolbar].clientBar, + CGID_PrivCITCommands, ITID_CUSTOMIZEENABLED, 0, NULL, NULL); return 0; }

9 years, 10 months

[aandrejevic] 65037: [FAST486] Don't leave the prefetch cache in a possibly invalid state if Fast486ReadLinearMemory returns FALSE.

by aandrejevic＠svn.reactos.org

Author: aandrejevic Date: Mon Oct 27 00:21:06 2014 New Revision: 65037 URL: http://svn.reactos.org/svn/reactos?rev=65037&view=rev Log: [FAST486] Don't leave the prefetch cache in a possibly invalid state if Fast486ReadLinearMemory returns FALSE. Modified: trunk/reactos/lib/fast486/common.c Modified: trunk/reactos/lib/fast486/common.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.c?rev=6… ============================================================================== --- trunk/reactos/lib/fast486/common.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.c [iso-8859-1] Mon Oct 27 00:21:06 2014 @@ -121,7 +121,11 @@ Size); return TRUE; } - else return FALSE; + else + { + State->PrefetchValid = FALSE; + return FALSE; + } } else #endif

9 years, 10 months

[gadamopoulos] 65036: [BROWSUI] - Fix CAddressBand::Invoke to correctly detect if an item exists in the list and select the correct item when it does. - Improve CAddressEditBox::Execute to check if...

by gadamopoulos＠svn.reactos.org

Author: gadamopoulos Date: Sun Oct 26 23:46:35 2014 New Revision: 65036 URL: http://svn.reactos.org/svn/reactos?rev=65036&view=rev Log: [BROWSUI] - Fix CAddressBand::Invoke to correctly detect if an item exists in the list and select the correct item when it does. - Improve CAddressEditBox::Execute to check if the passed pidl is the one that is being displayed. - Also fix it to parse the path if needed. - Directly call CAddressEditBox::Execute when enter is pressed or the Go button is pressed - Should fix most issues with the address bar Modified: branches/shell-experiments/dll/win32/browseui/addressband.cpp branches/shell-experiments/dll/win32/browseui/addresseditbox.cpp Modified: branches/shell-experiments/dll/win32/browseui/addressband.cpp URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/bro… ============================================================================== --- branches/shell-experiments/dll/win32/browseui/addressband.cpp [iso-8859-1] (original) +++ branches/shell-experiments/dll/win32/browseui/addressband.cpp [iso-8859-1] Sun Oct 26 23:46:35 2014 @@ -489,12 +489,17 @@ oldIndex = SendMessage(m_hWnd, CB_GETCURSEL, 0, 0); + itemExists = FALSE; + pidlPrevious = NULL; + + ZeroMemory(&item, sizeof(item)); item.mask = CBEIF_LPARAM; item.iItem = 0; - itemExists = SendMessage(m_hWnd, CBEM_GETITEM, 0, reinterpret_cast<LPARAM>(&item)); - if (itemExists) + if (SendMessage(m_hWnd, CBEM_GETITEM, 0, reinterpret_cast<LPARAM>(&item))) { pidlPrevious = reinterpret_cast<LPITEMIDLIST>(item.lParam); + if (pidlPrevious) + itemExists = TRUE; } hr = IUnknown_QueryService(fSite, SID_STopLevelBrowser, IID_PPV_ARG(IBrowserService, &isb)); @@ -520,6 +525,7 @@ if (itemExists) { result = SendMessage(m_hWnd, CBEM_SETITEM, 0, reinterpret_cast<LPARAM>(&item)); + oldIndex = 0; if (result) { Modified: branches/shell-experiments/dll/win32/browseui/addresseditbox.cpp URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/bro… ============================================================================== --- branches/shell-experiments/dll/win32/browseui/addresseditbox.cpp [iso-8859-1] (original) +++ branches/shell-experiments/dll/win32/browseui/addresseditbox.cpp [iso-8859-1] Sun Oct 26 23:46:35 2014 @@ -149,30 +149,72 @@ { HRESULT hr; + /* + * Parse the path is it wasn't parsed + */ + if (!pidlLastParsed) + ParseNow(0); + if (!pidlLastParsed) return E_FAIL; + /* + * Get the IShellBrowser and IBrowserService interfaces of the shell browser + */ CComPtr<IShellBrowser> pisb; hr = IUnknown_QueryService(fSite, SID_SShellBrowser, IID_PPV_ARG(IShellBrowser, &pisb)); + if (FAILED(hr)) + return hr; + + CComPtr<IBrowserService> pbs; + pisb->QueryInterface(IID_PPV_ARG(IBrowserService, &pbs)); + if (FAILED(hr)) + return hr; + + /* + * Get the current pidl of the shellbrowser and check if it is the same with the parsed one + */ + PIDLIST_ABSOLUTE pidl; + hr = pbs->GetPidl(&pidl); + if (FAILED(hr)) + return hr; + + CComPtr<IShellFolder> psf; + hr = SHGetDesktopFolder(&psf); + if (FAILED(hr)) + return hr; + + hr = psf->CompareIDs(0, pidl, pidlLastParsed); + + SHFree(pidl); + if (hr == 0) + return S_OK; + + /* + * Attempt to browse to the parsed pidl + */ + hr = pisb->BrowseObject(pidlLastParsed, 0); if (SUCCEEDED(hr)) - { - hr = pisb->BrowseObject(pidlLastParsed, 0); - if (FAILED_UNEXPECTEDLY(hr)) - { - HWND topLevelWindow; - LPCITEMIDLIST pidlChild; - CComPtr<IShellFolder> sf; - CComPtr<IShellBrowser> pisb; - - hr = IUnknown_QueryService(fSite, SID_SShellBrowser, IID_PPV_ARG(IShellBrowser, &pisb)); - - IUnknown_GetWindow(pisb, &topLevelWindow); - - hr = SHBindToParent(pidlLastParsed, IID_PPV_ARG(IShellFolder, &sf), &pidlChild); - - SHInvokeDefaultCommand(topLevelWindow, sf, pidlChild); - } - } + return hr; + + /* + * Browsing to the pidl failed so it's not a folder. So invoke its defaule command. + */ + HWND topLevelWindow; + hr = IUnknown_GetWindow(pisb, &topLevelWindow); + if (FAILED(hr)) + return hr; + + LPCITEMIDLIST pidlChild; + CComPtr<IShellFolder> sf; + hr = SHBindToParent(pidlLastParsed, IID_PPV_ARG(IShellFolder, &sf), &pidlChild); + if (FAILED(hr)) + return hr; + + hr = SHInvokeDefaultCommand(topLevelWindow, sf, pidlChild); + if (FAILED(hr)) + return hr; + return hr; } @@ -194,7 +236,15 @@ hdr = (LPNMHDR) lParam; if (hdr->code == CBEN_ENDEDIT) { - ParseNow(0); + NMCBEENDEDITW *endEdit = (NMCBEENDEDITW*) lParam; + if (endEdit->iWhy == CBENF_RETURN) + { + Execute(0); + } + else if (endEdit->iWhy == CBENF_ESCAPE) + { + /* Reset the contents of the combo box */ + } } break; }

9 years, 10 months

[aandrejevic] 65035: [FAST486] Implement an (optional) instruction prefetch cache. Implement the INVLPG instruction.

by aandrejevic＠svn.reactos.org

Author: aandrejevic Date: Sun Oct 26 23:37:54 2014 New Revision: 65035 URL: http://svn.reactos.org/svn/reactos?rev=65035&view=rev Log: [FAST486] Implement an (optional) instruction prefetch cache. Implement the INVLPG instruction. Modified: trunk/reactos/include/reactos/libs/fast486/fast486.h trunk/reactos/lib/fast486/common.c trunk/reactos/lib/fast486/common.h trunk/reactos/lib/fast486/common.inl trunk/reactos/lib/fast486/extraops.c trunk/reactos/lib/fast486/opcodes.c trunk/reactos/lib/fast486/opgroups.c Modified: trunk/reactos/include/reactos/libs/fast486/fast486.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/libs/fast4… ============================================================================== --- trunk/reactos/include/reactos/libs/fast486/fast486.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/libs/fast486/fast486.h [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -92,6 +92,18 @@ #define FAST486_PREFIX_REP (1 << 5) #define FAST486_FPU_DEFAULT_CONTROL 0x037F + +#define FAST486_PAGE_SIZE 4096 +#define FAST486_CACHE_SIZE 32 + +/* + * These are condiciones sine quibus non that should be respected, because + * otherwise when fetching DWORDs you would read extra garbage bytes + * (by reading outside of the prefetch buffer). The prefetch cache must + * also not cross a page boundary. + */ +C_ASSERT((FAST486_CACHE_SIZE >= sizeof(DWORD)) + && (FAST486_CACHE_SIZE <= FAST486_PAGE_SIZE)); struct _FAST486_STATE; typedef struct _FAST486_STATE FAST486_STATE, *PFAST486_STATE; @@ -486,6 +498,11 @@ FAST486_INT_STATUS IntStatus; UCHAR PendingIntNum; PULONG Tlb; +#ifndef FAST486_NO_PREFETCH + BOOLEAN PrefetchValid; + ULONG PrefetchAddress; + UCHAR PrefetchCache[FAST486_CACHE_SIZE]; +#endif #ifndef FAST486_NO_FPU FAST486_FPU_DATA_REG FpuRegisters[FAST486_NUM_FPU_REGS]; FAST486_FPU_STATUS_REG FpuStatus; Modified: trunk/reactos/lib/fast486/common.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.c?rev=6… ============================================================================== --- trunk/reactos/lib/fast486/common.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.c [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -95,8 +95,40 @@ /* Find the linear address */ LinearAddress = CachedDescriptor->Base + Offset; - /* Read from the linear address */ - return Fast486ReadLinearMemory(State, LinearAddress, Buffer, Size); +#ifndef FAST486_NO_PREFETCH + if (InstFetch && ((Offset + FAST486_CACHE_SIZE - 1) <= CachedDescriptor->Limit)) + { + State->PrefetchAddress = LinearAddress; + + if ((State->ControlRegisters[FAST486_REG_CR0] & FAST486_CR0_PG) + && (PAGE_OFFSET(State->PrefetchAddress) > (FAST486_PAGE_SIZE - FAST486_CACHE_SIZE))) + { + /* We mustn't prefetch across a page boundary */ + State->PrefetchAddress = PAGE_ALIGN(State->PrefetchAddress) + | (FAST486_PAGE_SIZE - FAST486_CACHE_SIZE); + } + + /* Prefetch */ + if (Fast486ReadLinearMemory(State, + State->PrefetchAddress, + State->PrefetchCache, + FAST486_CACHE_SIZE)) + { + State->PrefetchValid = TRUE; + + RtlMoveMemory(Buffer, + &State->PrefetchCache[LinearAddress - State->PrefetchAddress], + Size); + return TRUE; + } + else return FALSE; + } + else +#endif + { + /* Read from the linear address */ + return Fast486ReadLinearMemory(State, LinearAddress, Buffer, Size); + } } BOOLEAN @@ -155,6 +187,18 @@ /* Find the linear address */ LinearAddress = CachedDescriptor->Base + Offset; + +#ifndef FAST486_NO_PREFETCH + if (State->PrefetchValid + && (LinearAddress >= State->PrefetchAddress) + && ((LinearAddress + Size) <= (State->PrefetchAddress + FAST486_CACHE_SIZE))) + { + /* Update the prefetch */ + RtlMoveMemory(&State->PrefetchCache[LinearAddress - State->PrefetchAddress], + Buffer, + min(Size, FAST486_CACHE_SIZE + State->PrefetchAddress - LinearAddress)); + } +#endif /* Write to the linear address */ return Fast486WriteLinearMemory(State, LinearAddress, Buffer, Size); Modified: trunk/reactos/lib/fast486/common.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.h?rev=6… ============================================================================== --- trunk/reactos/lib/fast486/common.h [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.h [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -69,10 +69,6 @@ #define GET_ADDR_PDE(x) ((x) >> 22) #define GET_ADDR_PTE(x) (((x) >> 12) & 0x3FF) #define INVALID_TLB_FIELD 0xFFFFFFFF - -#ifndef PAGE_SIZE -#define PAGE_SIZE 4096 -#endif typedef struct _FAST486_MOD_REG_RM { Modified: trunk/reactos/lib/fast486/common.inl URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/common.inl?rev… ============================================================================== --- trunk/reactos/lib/fast486/common.inl [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/common.inl [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -163,9 +163,9 @@ for (Page = PAGE_ALIGN(LinearAddress); Page <= PAGE_ALIGN(LinearAddress + Size - 1); - Page += PAGE_SIZE) - { - ULONG PageOffset = 0, PageLength = PAGE_SIZE; + Page += FAST486_PAGE_SIZE) + { + ULONG PageOffset = 0, PageLength = FAST486_PAGE_SIZE; /* Get the table entry */ TableEntry.Value = Fast486GetPageTableEntry(State, Page, FALSE); @@ -230,9 +230,9 @@ for (Page = PAGE_ALIGN(LinearAddress); Page <= PAGE_ALIGN(LinearAddress + Size - 1); - Page += PAGE_SIZE) - { - ULONG PageOffset = 0, PageLength = PAGE_SIZE; + Page += FAST486_PAGE_SIZE) + { + ULONG PageOffset = 0, PageLength = FAST486_PAGE_SIZE; /* Get the table entry */ TableEntry.Value = Fast486GetPageTableEntry(State, Page, TRUE); @@ -521,6 +521,11 @@ else if (Segment == FAST486_REG_CS) { /* Loading the code segment */ + +#ifndef FAST486_NO_PREFETCH + /* Invalidate the prefetch */ + State->PrefetchValid = FALSE; +#endif if (GET_SEGMENT_INDEX(Selector) == 0) { @@ -641,21 +646,39 @@ PUCHAR Data) { PFAST486_SEG_REG CachedDescriptor; + ULONG Offset; +#ifndef FAST486_NO_PREFETCH + ULONG LinearAddress; +#endif /* Get the cached descriptor of CS */ CachedDescriptor = &State->SegmentRegs[FAST486_REG_CS]; - /* Read from memory */ - if (!Fast486ReadMemory(State, - FAST486_REG_CS, - (CachedDescriptor->Size) ? State->InstPtr.Long - : State->InstPtr.LowWord, - TRUE, - Data, - sizeof(UCHAR))) - { - /* Exception occurred during instruction fetch */ - return FALSE; + Offset = (CachedDescriptor->Size) ? State->InstPtr.Long + : State->InstPtr.LowWord; +#ifndef FAST486_NO_PREFETCH + LinearAddress = CachedDescriptor->Base + Offset; + + if (State->PrefetchValid + && (LinearAddress >= State->PrefetchAddress) + && ((LinearAddress + sizeof(UCHAR)) <= (State->PrefetchAddress + FAST486_CACHE_SIZE))) + { + *Data = *(PUCHAR)&State->PrefetchCache[LinearAddress - State->PrefetchAddress]; + } + else +#endif + { + /* Read from memory */ + if (!Fast486ReadMemory(State, + FAST486_REG_CS, + Offset, + TRUE, + Data, + sizeof(UCHAR))) + { + /* Exception occurred during instruction fetch */ + return FALSE; + } } /* Advance the instruction pointer */ @@ -672,22 +695,41 @@ PUSHORT Data) { PFAST486_SEG_REG CachedDescriptor; + ULONG Offset; +#ifndef FAST486_NO_PREFETCH + ULONG LinearAddress; +#endif /* Get the cached descriptor of CS */ CachedDescriptor = &State->SegmentRegs[FAST486_REG_CS]; - /* Read from memory */ - // FIXME: Fix byte order on big-endian machines - if (!Fast486ReadMemory(State, - FAST486_REG_CS, - (CachedDescriptor->Size) ? State->InstPtr.Long - : State->InstPtr.LowWord, - TRUE, - Data, - sizeof(USHORT))) - { - /* Exception occurred during instruction fetch */ - return FALSE; + Offset = (CachedDescriptor->Size) ? State->InstPtr.Long + : State->InstPtr.LowWord; + +#ifndef FAST486_NO_PREFETCH + LinearAddress = CachedDescriptor->Base + Offset; + + if (State->PrefetchValid + && (LinearAddress >= State->PrefetchAddress) + && ((LinearAddress + sizeof(USHORT)) <= (State->PrefetchAddress + FAST486_CACHE_SIZE))) + { + *Data = *(PUSHORT)&State->PrefetchCache[LinearAddress - State->PrefetchAddress]; + } + else +#endif + { + /* Read from memory */ + // FIXME: Fix byte order on big-endian machines + if (!Fast486ReadMemory(State, + FAST486_REG_CS, + Offset, + TRUE, + Data, + sizeof(USHORT))) + { + /* Exception occurred during instruction fetch */ + return FALSE; + } } /* Advance the instruction pointer */ @@ -704,22 +746,41 @@ PULONG Data) { PFAST486_SEG_REG CachedDescriptor; + ULONG Offset; +#ifndef FAST486_NO_PREFETCH + ULONG LinearAddress; +#endif /* Get the cached descriptor of CS */ CachedDescriptor = &State->SegmentRegs[FAST486_REG_CS]; - /* Read from memory */ - // FIXME: Fix byte order on big-endian machines - if (!Fast486ReadMemory(State, - FAST486_REG_CS, - (CachedDescriptor->Size) ? State->InstPtr.Long - : State->InstPtr.LowWord, - TRUE, - Data, - sizeof(ULONG))) - { - /* Exception occurred during instruction fetch */ - return FALSE; + Offset = (CachedDescriptor->Size) ? State->InstPtr.Long + : State->InstPtr.LowWord; + +#ifndef FAST486_NO_PREFETCH + LinearAddress = CachedDescriptor->Base + Offset; + + if (State->PrefetchValid + && (LinearAddress >= State->PrefetchAddress) + && ((LinearAddress + sizeof(ULONG)) <= (State->PrefetchAddress + FAST486_CACHE_SIZE))) + { + *Data = *(PULONG)&State->PrefetchCache[LinearAddress - State->PrefetchAddress]; + } + else +#endif + { + /* Read from memory */ + // FIXME: Fix byte order on big-endian machines + if (!Fast486ReadMemory(State, + FAST486_REG_CS, + Offset, + TRUE, + Data, + sizeof(ULONG))) + { + /* Exception occurred during instruction fetch */ + return FALSE; + } } /* Advance the instruction pointer */ Modified: trunk/reactos/lib/fast486/extraops.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/extraops.c?rev… ============================================================================== --- trunk/reactos/lib/fast486/extraops.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/extraops.c [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -686,6 +686,11 @@ } } +#ifndef FAST486_NO_PREFETCH + /* Changing CR0 or CR3 can interfere with prefetching (because of paging) */ + State->PrefetchValid = FALSE; +#endif + /* Load a value to the control register */ State->ControlRegisters[ModRegRm.Register] = Value; } Modified: trunk/reactos/lib/fast486/opcodes.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/opcodes.c?rev=… ============================================================================== --- trunk/reactos/lib/fast486/opcodes.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/opcodes.c [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -4240,6 +4240,11 @@ { UCHAR BopCode; +#ifndef FAST486_NO_PREFETCH + /* Invalidate the prefetch since BOP handlers can alter the memory */ + State->PrefetchValid = FALSE; +#endif + /* Fetch the BOP code */ if (!Fast486FetchByte(State, &BopCode)) { Modified: trunk/reactos/lib/fast486/opgroups.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/fast486/opgroups.c?rev… ============================================================================== --- trunk/reactos/lib/fast486/opgroups.c [iso-8859-1] (original) +++ trunk/reactos/lib/fast486/opgroups.c [iso-8859-1] Sun Oct 26 23:37:54 2014 @@ -2177,7 +2177,31 @@ /* INVLPG */ case 7: { - UNIMPLEMENTED; +#ifndef FAST486_NO_PREFETCH + /* Invalidate the prefetch */ + State->PrefetchValid = FALSE; +#endif + + /* This is a privileged instruction */ + if (Fast486GetCurrentPrivLevel(State) != 0) + { + Fast486Exception(State, FAST486_EXCEPTION_GP); + return; + } + + if (!ModRegRm.Memory) + { + /* The second operand must be a memory location */ + Fast486Exception(State, FAST486_EXCEPTION_UD); + return; + } + + if (State->Tlb != NULL) + { + /* Clear the TLB entry */ + State->Tlb[ModRegRm.MemoryAddress >> 12] = INVALID_TLB_FIELD; + } + break; }

9 years, 10 months

[pschweitzer] 65034: [KERNEL32] Prevent a use-after-free issue in GetVolumeNameForVolumeMountPointW()

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sun Oct 26 21:09:07 2014 New Revision: 65034 URL: http://svn.reactos.org/svn/reactos?rev=65034&view=rev Log: [KERNEL32] Prevent a use-after-free issue in GetVolumeNameForVolumeMountPointW() Modified: trunk/reactos/dll/win32/kernel32/client/file/mntpoint.c Modified: trunk/reactos/dll/win32/kernel32/client/file/mntpoint.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/kernel32/client/… ============================================================================== --- trunk/reactos/dll/win32/kernel32/client/file/mntpoint.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/kernel32/client/file/mntpoint.c [iso-8859-1] Sun Oct 26 21:09:07 2014 @@ -118,14 +118,15 @@ NULL, 0, MountDevName, BufferLength); if (!NT_SUCCESS(Status)) { - RtlFreeHeap(GetProcessHeap(), 0, MountDevName); if (Status == STATUS_BUFFER_OVERFLOW) { BufferLength = sizeof(MOUNTDEV_NAME) + MountDevName->NameLength; + RtlFreeHeap(GetProcessHeap(), 0, MountDevName); continue; } else { + RtlFreeHeap(GetProcessHeap(), 0, MountDevName); NtClose(FileHandle); BaseSetLastNTError(Status); return FALSE;

9 years, 10 months

[pschweitzer] 65033: [FASTFAT] Make sure the appropriate resource is exclusively acquired when playing with FCB references

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sun Oct 26 20:29:00 2014 New Revision: 65033 URL: http://svn.reactos.org/svn/reactos?rev=65033&view=rev Log: [FASTFAT] Make sure the appropriate resource is exclusively acquired when playing with FCB references Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c Modified: trunk/reactos/drivers/filesystems/fastfat/fcb.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/fastfa… ============================================================================== --- trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] (original) +++ trunk/reactos/drivers/filesystems/fastfat/fcb.c [iso-8859-1] Sun Oct 26 20:29:00 2014 @@ -292,6 +292,8 @@ PDEVICE_EXTENSION pVCB, PVFATFCB pFCB) { + ASSERT(ExIsResourceAcquiredExclusive(&pVCB->DirResource)); + ++pFCB->RefCount; } @@ -304,6 +306,8 @@ DPRINT("releasing FCB at %p: %wZ, refCount:%d\n", pFCB, &pFCB->PathNameU, pFCB->RefCount); + + ASSERT(ExIsResourceAcquiredExclusive(&pVCB->DirResource)); while (pFCB) {

9 years, 10 months

← Newer
1
...
10
11
12
13
14
15
16
...
73
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs October 2014