Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 01/01: [WIN32SS][WINSRV] Optimize console background drawing (#2132)
by Katayama Hirofumi MZ 05 Dec '19

05 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=1dcdb4c2d0298ee71822c… commit 1dcdb4c2d0298ee71822c1d432a30604a82bbd21 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Thu Dec 5 20:19:00 2019 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Dec 5 20:19:00 2019 +0900 [WIN32SS][WINSRV] Optimize console background drawing (#2132) Console needs speed. This PR will optimize drawing of console. - Set NULL to the background brush. - Skip WM_ERASEBKGND. --- win32ss/user/winsrv/consrv/frontends/gui/conwnd.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c b/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c index 0d81e1d58b5..7676a99c22b 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c +++ b/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c @@ -149,7 +149,7 @@ RegisterConWndClass(IN HINSTANCE hInstance) WndClass.hIcon = ghDefaultIcon; WndClass.hIconSm = ghDefaultIconSm; WndClass.hCursor = ghDefaultCursor; - WndClass.hbrBackground = (HBRUSH)GetStockObject(BLACK_BRUSH); // The color of a terminal when it is switched off. + WndClass.hbrBackground = NULL; WndClass.lpszMenuName = NULL; WndClass.cbClsExtra = 0; WndClass.cbWndExtra = GWLP_CONWND_ALLOC; @@ -2214,6 +2214,9 @@ ConWndProc(HWND hWnd, UINT msg, WPARAM wParam, LPARAM lParam) if (OnClose(GuiData)) goto Default; break; + case WM_ERASEBKGND: + return TRUE; + case WM_PAINT: OnPaint(GuiData); break;

1 0

[reactos] 01/01: [SHELL32] Add 'm_' prefix to CFSFolder members (#2129)
by Katayama Hirofumi MZ 04 Dec '19

04 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=2fad488a85477c08313ae… commit 2fad488a85477c08313aea66db2025d3f0195ef7 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Thu Dec 5 05:35:25 2019 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Thu Dec 5 05:35:25 2019 +0900 [SHELL32] Add 'm_' prefix to CFSFolder members (#2129) CORE-7585 --- dll/win32/shell32/folders/CFSFolder.cpp | 118 ++++++++++++++++---------------- dll/win32/shell32/folders/CFSFolder.h | 6 +- 2 files changed, 62 insertions(+), 62 deletions(-) diff --git a/dll/win32/shell32/folders/CFSFolder.cpp b/dll/win32/shell32/folders/CFSFolder.cpp index d0aee2faccf..cfba27e0008 100644 --- a/dll/win32/shell32/folders/CFSFolder.cpp +++ b/dll/win32/shell32/folders/CFSFolder.cpp @@ -437,9 +437,9 @@ HRESULT WINAPI CFileSysEnum::Initialize(LPWSTR lpszPath, DWORD dwFlags) CFSFolder::CFSFolder() { - pclsid = (CLSID *)&CLSID_ShellFSFolder; - sPathTarget = NULL; - pidlRoot = NULL; + m_pclsid = &CLSID_ShellFSFolder; + m_sPathTarget = NULL; + m_pidlRoot = NULL; m_bGroupPolicyActive = 0; } @@ -447,8 +447,8 @@ CFSFolder::~CFSFolder() { TRACE("-- destroying IShellFolder(%p)\n", this); - SHFree(pidlRoot); - SHFree(sPathTarget); + SHFree(m_pidlRoot); + SHFree(m_sPathTarget); } @@ -654,7 +654,7 @@ HRESULT WINAPI CFSFolder::ParseDisplayName(HWND hwndOwner, else { /* build the full pathname to the element */ - lstrcpynW(szPath, sPathTarget, MAX_PATH - 1); + lstrcpynW(szPath, m_sPathTarget, MAX_PATH - 1); PathAddBackslashW(szPath); len = wcslen(szPath); lstrcpynW(szPath + len, szElement, MAX_PATH - len); @@ -702,7 +702,7 @@ HRESULT WINAPI CFSFolder::EnumObjects( DWORD dwFlags, LPENUMIDLIST *ppEnumIDList) { - return ShellObjectCreatorInit<CFileSysEnum>(sPathTarget, dwFlags, IID_PPV_ARG(IEnumIDList, ppEnumIDList)); + return ShellObjectCreatorInit<CFileSysEnum>(m_sPathTarget, dwFlags, IID_PPV_ARG(IEnumIDList, ppEnumIDList)); } /************************************************************************** @@ -725,7 +725,7 @@ HRESULT WINAPI CFSFolder::BindToObject( CComPtr<IShellFolder> pSF; HRESULT hr; - if (!pidlRoot || !ppvOut || !pidl || !pidl->mkid.cb) + if (!m_pidlRoot || !ppvOut || !pidl || !pidl->mkid.cb) { ERR("CFSFolder::BindToObject: Invalid parameters\n"); return E_INVALIDARG; @@ -747,7 +747,7 @@ HRESULT WINAPI CFSFolder::BindToObject( PERSIST_FOLDER_TARGET_INFO pfti = {0}; pfti.dwAttributes = -1; pfti.csidl = -1; - PathCombineW(pfti.szTargetParsingName, sPathTarget, pDataW->wszName); + PathCombineW(pfti.szTargetParsingName, m_sPathTarget, pDataW->wszName); /* Get the CLSID to bind to */ CLSID clsidFolder; @@ -767,7 +767,7 @@ HRESULT WINAPI CFSFolder::BindToObject( return hr; } - hr = SHELL32_BindToSF(pidlRoot, &pfti, pidl, &clsidFolder, riid, ppvOut); + hr = SHELL32_BindToSF(m_pidlRoot, &pfti, pidl, &clsidFolder, riid, ppvOut); if (FAILED_UNEXPECTEDLY(hr)) return hr; @@ -876,19 +876,19 @@ HRESULT WINAPI CFSFolder::CreateViewObject(HWND hwndOwner, if (bIsDropTarget || bIsShellView) { - DWORD dwDirAttributes = _ILGetFileAttributes(ILFindLastID(pidlRoot), NULL, 0); + DWORD dwDirAttributes = _ILGetFileAttributes(ILFindLastID(m_pidlRoot), NULL, 0); if ((dwDirAttributes & (FILE_ATTRIBUTE_SYSTEM | FILE_ATTRIBUTE_READONLY)) != 0) { CLSID clsidFolder; - hr = SHELL32_GetCLSIDForDirectory(sPathTarget, L"UICLSID", &clsidFolder); + hr = SHELL32_GetCLSIDForDirectory(m_sPathTarget, L"UICLSID", &clsidFolder); if (SUCCEEDED(hr)) { CComPtr<IPersistFolder> spFolder; hr = SHCoCreateInstance(NULL, &clsidFolder, NULL, IID_PPV_ARG(IPersistFolder, &spFolder)); if (!FAILED_UNEXPECTEDLY(hr)) { - hr = spFolder->Initialize(pidlRoot); + hr = spFolder->Initialize(m_pidlRoot); if (!FAILED_UNEXPECTEDLY(hr)) { @@ -909,7 +909,7 @@ HRESULT WINAPI CFSFolder::CreateViewObject(HWND hwndOwner, // No UICLSID handler found, continue to the default handlers if (bIsDropTarget) { - hr = CFSDropTarget_CreateInstance(sPathTarget, riid, ppvOut); + hr = CFSDropTarget_CreateInstance(m_sPathTarget, riid, ppvOut); } else if (IsEqualIID (riid, IID_IContextMenu)) { @@ -920,7 +920,7 @@ HRESULT WINAPI CFSFolder::CreateViewObject(HWND hwndOwner, DEFCONTEXTMENU dcm; dcm.hwnd = hwndOwner; dcm.pcmcb = this; - dcm.pidlFolder = pidlRoot; + dcm.pidlFolder = m_pidlRoot; dcm.psf = this; dcm.cidl = 0; dcm.apidl = NULL; @@ -968,7 +968,7 @@ HRESULT WINAPI CFSFolder::GetAttributesOf(UINT cidl, if(cidl == 0) { - LPCITEMIDLIST rpidl = ILFindLastID(pidlRoot); + LPCITEMIDLIST rpidl = ILFindLastID(m_pidlRoot); if (_ILIsFolder(rpidl) || _ILIsValue(rpidl)) { @@ -977,7 +977,7 @@ HRESULT WINAPI CFSFolder::GetAttributesOf(UINT cidl, else if (_ILIsDrive(rpidl)) { IShellFolder *psfParent = NULL; - hr = SHBindToParent(pidlRoot, IID_PPV_ARG(IShellFolder, &psfParent), NULL); + hr = SHBindToParent(m_pidlRoot, IID_PPV_ARG(IShellFolder, &psfParent), NULL); if(SUCCEEDED(hr)) { hr = psfParent->GetAttributesOf(1, &rpidl, (SFGAOF*)rgfInOut); @@ -1064,7 +1064,7 @@ HRESULT WINAPI CFSFolder::GetUIObjectOf(HWND hwndOwner, DEFCONTEXTMENU dcm; dcm.hwnd = hwndOwner; dcm.pcmcb = this; - dcm.pidlFolder = pidlRoot; + dcm.pidlFolder = m_pidlRoot; dcm.psf = this; dcm.cidl = cidl; dcm.apidl = apidl; @@ -1077,7 +1077,7 @@ HRESULT WINAPI CFSFolder::GetUIObjectOf(HWND hwndOwner, { if (cidl >= 1) { - hr = IDataObject_Constructor (hwndOwner, pidlRoot, apidl, cidl, TRUE, (IDataObject **)&pObj); + hr = IDataObject_Constructor (hwndOwner, m_pidlRoot, apidl, cidl, TRUE, (IDataObject **)&pObj); } else { @@ -1096,7 +1096,7 @@ HRESULT WINAPI CFSFolder::GetUIObjectOf(HWND hwndOwner, /* only interested in attempting to bind to shell folders, not files (except exe), so if we fail, rebind to root */ if (cidl != 1 || FAILED(hr = this->_GetDropTarget(apidl[0], (LPVOID*) &pObj))) { - hr = CFSDropTarget_CreateInstance(sPathTarget, riid, (LPVOID*) &pObj); + hr = CFSDropTarget_CreateInstance(m_sPathTarget, riid, (LPVOID*) &pObj); } } else @@ -1198,9 +1198,9 @@ HRESULT WINAPI CFSFolder::GetDisplayNameOf(PCUITEMID_CHILD pidl, /* If it is an empty pidl return only the path of the folder */ if ((GET_SHGDN_FOR(dwFlags) & SHGDN_FORPARSING) && (GET_SHGDN_RELATION(dwFlags) != SHGDN_INFOLDER) && - sPathTarget) + m_sPathTarget) { - return SHSetStrRet(strRet, sPathTarget); + return SHSetStrRet(strRet, m_sPathTarget); } return E_INVALIDARG; } @@ -1212,9 +1212,9 @@ HRESULT WINAPI CFSFolder::GetDisplayNameOf(PCUITEMID_CHILD pidl, if ((GET_SHGDN_FOR(dwFlags) & SHGDN_FORPARSING) && (GET_SHGDN_RELATION(dwFlags) != SHGDN_INFOLDER) && - sPathTarget) + m_sPathTarget) { - lstrcpynW(pszPath, sPathTarget, MAX_PATH); + lstrcpynW(pszPath, m_sPathTarget, MAX_PATH); PathAddBackslashW(pszPath); len = wcslen(pszPath); } @@ -1261,11 +1261,11 @@ HRESULT WINAPI CFSFolder::SetNameOf( } /* build source path */ - PathCombineW(szSrc, sPathTarget, pDataW->wszName); + PathCombineW(szSrc, m_sPathTarget, pDataW->wszName); /* build destination path */ if (dwFlags == SHGDN_NORMAL || dwFlags & SHGDN_INFOLDER) - PathCombineW(szDest, sPathTarget, lpName); + PathCombineW(szDest, m_sPathTarget, lpName); else lstrcpynW(szDest, lpName, MAX_PATH); @@ -1415,7 +1415,7 @@ HRESULT WINAPI CFSFolder::GetClassID(CLSID * lpClassId) if (!lpClassId) return E_POINTER; - *lpClassId = *pclsid; + *lpClassId = *m_pclsid; return S_OK; } @@ -1424,7 +1424,7 @@ HRESULT WINAPI CFSFolder::GetClassID(CLSID * lpClassId) * CFSFolder::Initialize * * NOTES - * sPathTarget is not set. Don't know how to handle in a non rooted environment. + * m_sPathTarget is not set. Don't know how to handle in a non rooted environment. */ HRESULT WINAPI CFSFolder::Initialize(PCIDLIST_ABSOLUTE pidl) { @@ -1432,23 +1432,23 @@ HRESULT WINAPI CFSFolder::Initialize(PCIDLIST_ABSOLUTE pidl) TRACE ("(%p)->(%p)\n", this, pidl); - SHFree (pidlRoot); /* free the old pidl */ - pidlRoot = ILClone (pidl); /* set my pidl */ + SHFree(m_pidlRoot); /* free the old pidl */ + m_pidlRoot = ILClone (pidl); /* set my pidl */ - SHFree (sPathTarget); - sPathTarget = NULL; + SHFree (m_sPathTarget); + m_sPathTarget = NULL; /* set my path */ if (SHGetPathFromIDListW (pidl, wszTemp)) { int len = wcslen(wszTemp); - sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); - if (!sPathTarget) + m_sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); + if (!m_sPathTarget) return E_OUTOFMEMORY; - memcpy(sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); + memcpy(m_sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); } - TRACE ("--(%p)->(%s)\n", this, debugstr_w(sPathTarget)); + TRACE ("--(%p)->(%s)\n", this, debugstr_w(m_sPathTarget)); return S_OK; } @@ -1462,7 +1462,7 @@ HRESULT WINAPI CFSFolder::GetCurFolder(PIDLIST_ABSOLUTE * pidl) if (!pidl) return E_POINTER; - *pidl = ILClone(pidlRoot); + *pidl = ILClone(m_pidlRoot); return S_OK; } @@ -1487,15 +1487,15 @@ HRESULT WINAPI CFSFolder::InitializeEx(IBindCtx * pbc, LPCITEMIDLIST pidlRootx, if (ppfti && ppfti->pidlTargetFolder) pdump(ppfti->pidlTargetFolder); - if (pidlRoot) - __SHFreeAndNil(&pidlRoot); /* free the old */ - if (sPathTarget) - __SHFreeAndNil(&sPathTarget); + if (m_pidlRoot) + __SHFreeAndNil(&m_pidlRoot); /* free the old */ + if (m_sPathTarget) + __SHFreeAndNil(&m_sPathTarget); /* * Root path and pidl */ - pidlRoot = ILClone(pidlRootx); + m_pidlRoot = ILClone(pidlRootx); /* * the target folder is spezified in csidl OR pidlTargetFolder OR @@ -1508,19 +1508,19 @@ HRESULT WINAPI CFSFolder::InitializeEx(IBindCtx * pbc, LPCITEMIDLIST pidlRootx, if (SHGetSpecialFolderPathW(0, wszTemp, ppfti->csidl, ppfti->csidl & CSIDL_FLAG_CREATE)) { int len = wcslen(wszTemp); - sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); - if (!sPathTarget) + m_sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); + if (!m_sPathTarget) return E_OUTOFMEMORY; - memcpy(sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); + memcpy(m_sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); } } else if (ppfti->szTargetParsingName[0]) { int len = wcslen(ppfti->szTargetParsingName); - sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); - if (!sPathTarget) + m_sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); + if (!m_sPathTarget) return E_OUTOFMEMORY; - memcpy(sPathTarget, ppfti->szTargetParsingName, + memcpy(m_sPathTarget, ppfti->szTargetParsingName, (len + 1) * sizeof(WCHAR)); } else if (ppfti->pidlTargetFolder) @@ -1528,17 +1528,17 @@ HRESULT WINAPI CFSFolder::InitializeEx(IBindCtx * pbc, LPCITEMIDLIST pidlRootx, if (SHGetPathFromIDListW(ppfti->pidlTargetFolder, wszTemp)) { int len = wcslen(wszTemp); - sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); - if (!sPathTarget) + m_sPathTarget = (WCHAR *)SHAlloc((len + 1) * sizeof(WCHAR)); + if (!m_sPathTarget) return E_OUTOFMEMORY; - memcpy(sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); + memcpy(m_sPathTarget, wszTemp, (len + 1) * sizeof(WCHAR)); } } } - TRACE("--(%p)->(target=%s)\n", this, debugstr_w(sPathTarget)); - pdump(pidlRoot); - return (sPathTarget) ? S_OK : E_FAIL; + TRACE("--(%p)->(target=%s)\n", this, debugstr_w(m_sPathTarget)); + pdump(m_pidlRoot); + return (m_sPathTarget) ? S_OK : E_FAIL; } HRESULT WINAPI CFSFolder::GetFolderTargetInfo(PERSIST_FOLDER_TARGET_INFO * ppfti) @@ -1630,7 +1630,7 @@ HRESULT CFSFolder::_CreateShellExtInstance(const CLSID *pclsid, LPCITEMIDLIST pi return E_INVALIDARG; } - PathCombineW(wszPath, sPathTarget, pDataW->wszName); + PathCombineW(wszPath, m_sPathTarget, pDataW->wszName); CComPtr<IPersistFile> pp; hr = SHCoCreateInstance(NULL, pclsid, NULL, IID_PPV_ARG(IPersistFile, &pp)); @@ -1658,10 +1658,10 @@ HRESULT WINAPI CFSFolder::CallBack(IShellFolder *psf, HWND hwndOwner, IDataObjec { if (uMsg == DFM_INVOKECOMMAND && wParam == 0) { - PUITEMID_CHILD pidlChild = ILClone(ILFindLastID(pidlRoot)); - LPITEMIDLIST pidlParent = ILClone(pidlRoot); + PUITEMID_CHILD pidlChild = ILClone(ILFindLastID(m_pidlRoot)); + LPITEMIDLIST pidlParent = ILClone(m_pidlRoot); ILRemoveLastID(pidlParent); - HRESULT hr = SH_ShowPropertiesDialog(sPathTarget, pidlParent, &pidlChild); + HRESULT hr = SH_ShowPropertiesDialog(m_sPathTarget, pidlParent, &pidlChild); if (FAILED(hr)) ERR("SH_ShowPropertiesDialog failed\n"); ILFree(pidlChild); @@ -1731,7 +1731,7 @@ HRESULT WINAPI CFSFolder::GetCustomViewInfo(ULONG unknown, SFVM_CUSTOMVIEWINFO_D WCHAR szPath[MAX_PATH], szIniFile[MAX_PATH]; // does the folder exists? - if (!SHGetPathFromIDListW(pidlRoot, szPath) || !PathIsDirectoryW(szPath)) + if (!SHGetPathFromIDListW(m_pidlRoot, szPath) || !PathIsDirectoryW(szPath)) { return E_INVALIDARG; } diff --git a/dll/win32/shell32/folders/CFSFolder.h b/dll/win32/shell32/folders/CFSFolder.h index 39b5c09ca2b..6c6f055a3f8 100644 --- a/dll/win32/shell32/folders/CFSFolder.h +++ b/dll/win32/shell32/folders/CFSFolder.h @@ -32,12 +32,12 @@ class CFSFolder : public IShellFolderViewCB { private: - CLSID *pclsid; + const CLSID *m_pclsid; /* both paths are parsible from the desktop */ - LPWSTR sPathTarget; /* complete path to target used for enumeration and ChangeNotify */ + LPWSTR m_sPathTarget; /* complete path to target used for enumeration and ChangeNotify */ - LPITEMIDLIST pidlRoot; /* absolute pidl */ + LPITEMIDLIST m_pidlRoot; /* absolute pidl */ DWORD m_bGroupPolicyActive; HRESULT _CreateShellExtInstance(const CLSID *pclsid, LPCITEMIDLIST pidl, REFIID riid, LPVOID *ppvOut);

1 0

[reactos] 01/01: [ROSTESTS] Skip some flaky test / do not count successes
by Timo Kreuzer 04 Dec '19

04 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e26c8bc66f9bf2bb053d0… commit e26c8bc66f9bf2bb053d0aa56a771c99c8accb52 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun Sep 15 15:07:32 2019 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Wed Dec 4 01:24:49 2019 +0100 [ROSTESTS] Skip some flaky test / do not count successes --- modules/rostests/apitests/kernel32/TunnelCache.c | 1 + .../apitests/ntdll/NtQueryInformationProcess.c | 4 ++ modules/rostests/apitests/user32/RedrawWindow.c | 1 + modules/rostests/apitests/user32/SetProp.c | 1 + modules/rostests/winetests/gdi32/bitmap.c | 8 +-- modules/rostests/winetests/gdi32/path.c | 3 ++ modules/rostests/winetests/kernel32/mailslot.c | 1 + modules/rostests/winetests/kernel32/pipe.c | 3 ++ modules/rostests/winetests/kernel32/process.c | 4 +- modules/rostests/winetests/kernel32/sync.c | 4 ++ modules/rostests/winetests/mshtml/xmlhttprequest.c | 3 ++ modules/rostests/winetests/msi/msi.c | 12 ++++- modules/rostests/winetests/msxml3/domdoc.c | 2 +- modules/rostests/winetests/ntdll/exception.c | 1 + modules/rostests/winetests/ntdll/info.c | 12 +++-- modules/rostests/winetests/ntdll/process.c | 2 + modules/rostests/winetests/ole32/compobj.c | 1 + modules/rostests/winetests/qmgr/job.c | 3 ++ modules/rostests/winetests/quartz/filtermapper.c | 1 + modules/rostests/winetests/quartz/referenceclock.c | 1 + modules/rostests/winetests/riched20/editor.c | 1 + modules/rostests/winetests/rpcrt4/server.c | 5 ++ modules/rostests/winetests/user32/listbox.c | 1 + modules/rostests/winetests/user32/msg.c | 2 + modules/rostests/winetests/user32/win.c | 8 +++ modules/rostests/winetests/usp10/usp10.c | 2 + modules/rostests/winetests/wininet/ftp.c | 6 +++ modules/rostests/winetests/wininet/http.c | 10 ++++ sdk/include/reactos/wine/test.h | 62 +++++++++++++++++++++- 29 files changed, 152 insertions(+), 13 deletions(-) diff --git a/modules/rostests/apitests/kernel32/TunnelCache.c b/modules/rostests/apitests/kernel32/TunnelCache.c index 9e67e74404d..8081da4a496 100644 --- a/modules/rostests/apitests/kernel32/TunnelCache.c +++ b/modules/rostests/apitests/kernel32/TunnelCache.c @@ -176,6 +176,7 @@ Test_LongTests(void) ok(GetFileTime(hFile, &File1Time, NULL, NULL) != FALSE, "GetFileTime() failed\n"); CloseHandle(hFile); + ros_skip_flaky ok(RtlCompareMemory(&FileTime, &File1Time, sizeof(FILETIME)) == sizeof(FILETIME), "Tunnel cache failed\n"); DeleteFile("file2"); diff --git a/modules/rostests/apitests/ntdll/NtQueryInformationProcess.c b/modules/rostests/apitests/ntdll/NtQueryInformationProcess.c index ec6c56d170a..bdbfc3f726a 100644 --- a/modules/rostests/apitests/ntdll/NtQueryInformationProcess.c +++ b/modules/rostests/apitests/ntdll/NtQueryInformationProcess.c @@ -119,6 +119,7 @@ Test_ProcessTimes(void) sizeof(KERNEL_USER_TIMES), NULL); ok_hex(Status, STATUS_SUCCESS); + ros_skip_flaky ok(Times1.CreateTime.QuadPart < TestStartTime.QuadPart, "CreateTime is %I64u, expected < %I64u\n", Times1.CreateTime.QuadPart, TestStartTime.QuadPart); ok(Times1.CreateTime.QuadPart > TestStartTime.QuadPart - 100000000LL, @@ -126,6 +127,7 @@ Test_ProcessTimes(void) ok(Times1.ExitTime.QuadPart == 0, "ExitTime is %I64u, expected 0\n", Times1.ExitTime.QuadPart); ok(Times1.KernelTime.QuadPart != 0, "KernelTime is 0\n"); + ros_skip_flaky ok(Times1.UserTime.QuadPart != 0, "UserTime is 0\n"); /* Do some busy waiting to increase UserTime */ @@ -162,9 +164,11 @@ Test_ProcessTimes(void) /* Time values must have increased */ ok(Times2.KernelTime.QuadPart > Times1.KernelTime.QuadPart, "KernelTime values inconsistent. Expected %I64u > %I64u\n", Times2.KernelTime.QuadPart, Times1.KernelTime.QuadPart); + ros_skip_flaky ok(Times2.UserTime.QuadPart > Times1.UserTime.QuadPart, "UserTime values inconsistent. Expected %I64u > %I64u\n", Times2.UserTime.QuadPart, Times1.UserTime.QuadPart); /* They can't have increased by more than wall clock time difference (we only have one thread) */ + ros_skip_flaky ok(Times2.KernelTime.QuadPart - Times1.KernelTime.QuadPart < Time2.QuadPart - Time1.QuadPart, "KernelTime values inconsistent. Expected %I64u - %I64u < %I64u\n", Times2.KernelTime.QuadPart, Times1.KernelTime.QuadPart, Time2.QuadPart - Time1.QuadPart); diff --git a/modules/rostests/apitests/user32/RedrawWindow.c b/modules/rostests/apitests/user32/RedrawWindow.c index 8cd2b38e6fb..50ed0ea19e8 100644 --- a/modules/rostests/apitests/user32/RedrawWindow.c +++ b/modules/rostests/apitests/user32/RedrawWindow.c @@ -19,6 +19,7 @@ WndProc( _In_ WPARAM wParam, _In_ LPARAM lParam) { + disable_success_count ok(GetCurrentThreadId() == dwThreadId, "Thread 0x%lx instead of 0x%lx\n", GetCurrentThreadId(), dwThreadId); if (message == WM_PAINT) { diff --git a/modules/rostests/apitests/user32/SetProp.c b/modules/rostests/apitests/user32/SetProp.c index 3d656fd9936..838dc5e78cf 100644 --- a/modules/rostests/apitests/user32/SetProp.c +++ b/modules/rostests/apitests/user32/SetProp.c @@ -158,6 +158,7 @@ START_TEST(SetProp) /* In particular we shouldn't see these from WM_SETICON */ SysICAtom = RegisterWindowMessageW(L"SysIC"); Prop = GetPropW(hWnd, (PCWSTR)MAKEINTATOM(SysICAtom)); + ros_skip_flaky ok(Prop == NULL, "SysIC prop (0x%04x) is %p\n", SysICAtom, Prop); SysICSAtom = RegisterWindowMessageW(L"SysICS"); diff --git a/modules/rostests/winetests/gdi32/bitmap.c b/modules/rostests/winetests/gdi32/bitmap.c index 2409bd3f898..88c827c5657 100644 --- a/modules/rostests/winetests/gdi32/bitmap.c +++ b/modules/rostests/winetests/gdi32/bitmap.c @@ -5586,8 +5586,8 @@ static void test_SetDIBitsToDevice_RLE8(void) for (i = 0; i < 24; i++) ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); for (i = 24; i < 64; i++) if (i == 52) ok( dib_bits[i] == 0x00808080, "%d: got %08x\n", i, dib_bits[i] ); - else if (i & 4) ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); - else ok( dib_bits[i] == bottom_up[i - 20], "%d: got %08x\n", i, dib_bits[i] ); + else if (i & 4) ros_skip_flaky ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); + else ros_skip_flaky ok(dib_bits[i] == bottom_up[i - 20], "%d: got %08x\n", i, dib_bits[i]); memset( dib_bits, 0xaa, 64 * 4 ); /* top-down compressed dibs are invalid */ @@ -5653,8 +5653,8 @@ static void test_SetDIBitsToDevice_RLE8(void) ok( ret == 37, "got %d\n", ret ); for (i = 0; i < 40; i++) if (i == 12) ok( dib_bits[i] == 0x00808080, "%d: got %08x\n", i, dib_bits[i] ); - else if (i & 4) ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); - else ok( dib_bits[i] == top_down[i + 28], "%d: got %08x\n", i, dib_bits[i] ); + else if (i & 4) ros_skip_flaky ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); + else ros_skip_flaky ok( dib_bits[i] == top_down[i + 28], "%d: got %08x\n", i, dib_bits[i] ); for (i = 40; i < 64; i++) ok( dib_bits[i] == 0xaaaaaaaa, "%d: got %08x\n", i, dib_bits[i] ); memset( dib_bits, 0xaa, 64 * 4 ); diff --git a/modules/rostests/winetests/gdi32/path.c b/modules/rostests/winetests/gdi32/path.c index 48ef4f016a1..abceadd0c1c 100644 --- a/modules/rostests/winetests/gdi32/path.c +++ b/modules/rostests/winetests/gdi32/path.c @@ -420,6 +420,7 @@ static void ok_path(HDC hdc, const char *path_name, const path_test_t *expected, size = GetPath(hdc, pnt, types, size); assert(size > 0); + ros_skip_flaky ok( size == expected_size, "%s: Path size %d does not match expected size %d\n", path_name, size, expected_size); @@ -429,11 +430,13 @@ static void ok_path(HDC hdc, const char *path_name, const path_test_t *expected, * floating point to integer conversion */ static const int fudge = 2; + ros_skip_flaky ok( types[idx] == expected[idx].type, "%s: Expected #%d: %s (%d,%d) but got %s (%d,%d)\n", path_name, idx, type_string[expected[idx].type], expected[idx].x, expected[idx].y, type_string[types[idx]], pnt[idx].x, pnt[idx].y); if (types[idx] == expected[idx].type) + ros_skip_flaky ok( (pnt[idx].x >= expected[idx].x - fudge && pnt[idx].x <= expected[idx].x + fudge) && (pnt[idx].y >= expected[idx].y - fudge && pnt[idx].y <= expected[idx].y + fudge), "%s: Expected #%d: %s position (%d,%d) but got (%d,%d)\n", path_name, idx, diff --git a/modules/rostests/winetests/kernel32/mailslot.c b/modules/rostests/winetests/kernel32/mailslot.c index da595e97143..27258247287 100644 --- a/modules/rostests/winetests/kernel32/mailslot.c +++ b/modules/rostests/winetests/kernel32/mailslot.c @@ -324,6 +324,7 @@ todo_wine ok( !ReadFile( hSlot, buffer, sizeof buffer, &count, NULL), "slot read\n"); ok( GetLastError() == ERROR_SEM_TIMEOUT, "wrong error %u\n", GetLastError() ); dwTimeout = GetTickCount() - dwTimeout; + ros_skip_flaky ok( dwTimeout >= 990, "timeout too short %u\n", dwTimeout ); ok( CloseHandle( hSlot ), "closing the mailslot\n"); diff --git a/modules/rostests/winetests/kernel32/pipe.c b/modules/rostests/winetests/kernel32/pipe.c index 8e5c07ff56e..b2f673400a5 100644 --- a/modules/rostests/winetests/kernel32/pipe.c +++ b/modules/rostests/winetests/kernel32/pipe.c @@ -117,6 +117,7 @@ static BOOL RpcReadFile(HANDLE hFile, LPVOID buffer, DWORD bytesToRead, LPDWORD static void _test_not_signaled(unsigned line, HANDLE handle) { DWORD res = WaitForSingleObject(handle, 0); + disable_success_count ok_(__FILE__,line)(res == WAIT_TIMEOUT, "WaitForSingleObject returned %u (%u)\n", res, GetLastError()); } @@ -2784,7 +2785,9 @@ static void _overlapped_write_async(unsigned line, HANDLE writer, void *buf, DWO memset(overlapped, 0, sizeof(*overlapped)); overlapped->hEvent = CreateEventW(NULL, TRUE, FALSE, NULL); res = WriteFile(writer, buf, size, &written_bytes, overlapped); + disable_success_count ok_(__FILE__,line)(!res && GetLastError() == ERROR_IO_PENDING, "WriteFile returned %x(%u)\n", res, GetLastError()); + disable_success_count ok_(__FILE__,line)(!written_bytes, "written_bytes = %u\n", written_bytes); _test_not_signaled(line, overlapped->hEvent); diff --git a/modules/rostests/winetests/kernel32/process.c b/modules/rostests/winetests/kernel32/process.c index 0c973ad366d..b0bf22a0951 100644 --- a/modules/rostests/winetests/kernel32/process.c +++ b/modules/rostests/winetests/kernel32/process.c @@ -2572,7 +2572,7 @@ static void test_TerminateJobObject(void) ret = GetExitCodeProcess(pi.hProcess, &dwret); ok(ret, "GetExitCodeProcess error %u\n", GetLastError()); - ok(dwret == 123 || broken(dwret == 0) /* randomly fails on Win 2000 / XP */, + ok(dwret == 123 || broken(dwret == 0) || broken(dwret == 259) /* randomly fails on Win 2000 / XP */, "wrong exitcode %u\n", dwret); CloseHandle(pi.hProcess); @@ -3419,7 +3419,9 @@ static void test_SuspendProcessState(void) #endif ret = ReadProcessMemory( pi.hProcess, peb_ptr, &child_peb, sizeof(child_peb), NULL ); + ros_skip_flaky ok( ret, "Failed to read PEB (%u)\n", GetLastError() ); + ros_skip_flaky ok( child_peb.ImageBaseAddress == exe_base, "wrong base %p/%p\n", child_peb.ImageBaseAddress, exe_base ); ok( entry_ptr == (char *)exe_base + nt_header.OptionalHeader.AddressOfEntryPoint, diff --git a/modules/rostests/winetests/kernel32/sync.c b/modules/rostests/winetests/kernel32/sync.c index 0de706bd26d..fced98bf217 100644 --- a/modules/rostests/winetests/kernel32/sync.c +++ b/modules/rostests/winetests/kernel32/sync.c @@ -778,6 +778,7 @@ static void test_iocp_callback(void) static void CALLBACK timer_queue_cb1(PVOID p, BOOLEAN timedOut) { int *pn = p; + disable_success_count ok(timedOut, "Timer callbacks should always time out\n"); ++*pn; } @@ -2571,12 +2572,14 @@ static DWORD WINAPI apc_deadlock_thread(void *param) size = 0x1000; status = pNtAllocateVirtualMemory(pi->hProcess, &base, 0, &size, MEM_RESERVE | MEM_COMMIT, PAGE_READWRITE); + disable_success_count ok(!status, "expected STATUS_SUCCESS, got %08x\n", status); ok(base != NULL, "expected base != NULL, got %p\n", base); SetEvent(info->event); size = 0; status = pNtFreeVirtualMemory(pi->hProcess, &base, &size, MEM_RELEASE); + disable_success_count ok(!status, "expected STATUS_SUCCESS, got %08x\n", status); SetEvent(info->event); } @@ -2613,6 +2616,7 @@ static void test_apc_deadlock(void) result = WaitForSingleObject(event, 1000); ok(result == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", result); + disable_success_count for (i = 0; i < 1000; i++) { result = SuspendThread(pi.hThread); diff --git a/modules/rostests/winetests/mshtml/xmlhttprequest.c b/modules/rostests/winetests/mshtml/xmlhttprequest.c index 8a4bd1793be..8706e2b1f52 100644 --- a/modules/rostests/winetests/mshtml/xmlhttprequest.c +++ b/modules/rostests/winetests/mshtml/xmlhttprequest.c @@ -231,9 +231,11 @@ static HRESULT WINAPI xmlhttprequest_onreadystatechange(IDispatchEx *iface, DISP LONG val; HRESULT hres; + if (!expect_xmlhttprequest_onreadystatechange_loading) test_event_args(&DIID_DispHTMLXMLHttpRequest, id, wFlags, pdp, pvarRes, pei, pspCaller); hres = IHTMLXMLHttpRequest_get_readyState(xhr, &val); + disable_success_count ok(hres == S_OK, "get_readyState failed: %08x\n", hres); readystatechange_cnt++; @@ -246,6 +248,7 @@ static HRESULT WINAPI xmlhttprequest_onreadystatechange(IDispatchEx *iface, DISP break; case 3: loading_cnt++; + disable_success_count CHECK_EXPECT2(xmlhttprequest_onreadystatechange_loading); break; case 4: diff --git a/modules/rostests/winetests/msi/msi.c b/modules/rostests/winetests/msi/msi.c index 4b545f2d5a9..fcb7d66eaae 100644 --- a/modules/rostests/winetests/msi/msi.c +++ b/modules/rostests/winetests/msi/msi.c @@ -13505,6 +13505,7 @@ static void test_MsiEnumProducts(void) ok(r == ERROR_NO_MORE_ITEMS, "Expected ERROR_NO_MORE_ITEMS, got %u\n", r); ok(found1, "product1 not found\n"); ok(found2, "product2 not found\n"); + ros_skip_flaky ok(found3, "product3 not found\n"); delete_key(key1, "", access & KEY_WOW64_64KEY); @@ -13693,18 +13694,25 @@ static void test_MsiEnumProductsEx(void) ok( !sid[0], "got \"%s\"\n", sid ); ok( !len, "unexpected length %u\n", len ); } - if (!strcmp( product2, guid )) + else if (!strcmp( product2, guid )) { ok( context == MSIINSTALLCONTEXT_USERMANAGED, "got %u\n", context ); ok( sid[0], "empty sid\n" ); ok( len == strlen(sid), "unexpected length %u\n", len ); } - if (!strcmp( product3, guid )) + else if (!strcmp( product3, guid )) { ok( context == MSIINSTALLCONTEXT_USERUNMANAGED, "got %u\n", context ); ok( sid[0], "empty sid\n" ); ok( len == strlen(sid), "unexpected length %u\n", len ); } + else + { + trace("Unexpected guid: %s (have %s | %s | %s)\n", guid, product1, product2, product3); + ok(context != MSIINSTALLCONTEXT_NONE, "got %u\n", context); + ok(sid[0], "empty sid\n"); + ok(len == strlen(sid), "unexpected length %u\n", len); + } index++; guid[0] = 0; context = 0xdeadbeef; diff --git a/modules/rostests/winetests/msxml3/domdoc.c b/modules/rostests/winetests/msxml3/domdoc.c index 9fd83dda7dd..fbd0f8edef6 100644 --- a/modules/rostests/winetests/msxml3/domdoc.c +++ b/modules/rostests/winetests/msxml3/domdoc.c @@ -8027,7 +8027,7 @@ static void test_get_ownerDocument(void) IXMLDOMDocument_Release(doc2); IXMLDOMDocument_Release(doc3); IXMLDOMDocument2_Release(doc); - IXMLDOMDocument2_Release(doc_owner); + //IXMLDOMDocument2_Release(doc_owner); FIXME: double-free! free_bstrs(); } diff --git a/modules/rostests/winetests/ntdll/exception.c b/modules/rostests/winetests/ntdll/exception.c index 7d3c89acafb..d5a197b4a31 100644 --- a/modules/rostests/winetests/ntdll/exception.c +++ b/modules/rostests/winetests/ntdll/exception.c @@ -782,6 +782,7 @@ static DWORD bpx_handler( EXCEPTION_RECORD *rec, EXCEPTION_REGISTRATION_RECORD * ok( context->Eip == (DWORD)code_mem, "eip is wrong: %x instead of %x\n", context->Eip, (DWORD)code_mem); ok( (context->Dr6 & 0xf) == 1, "B0 flag is not set in Dr6\n"); + ros_skip_flaky ok( !(context->Dr6 & 0x4000), "BS flag is set in Dr6\n"); context->Dr0 = context->Dr0 + 1; /* set hw bp again on next instruction */ context->EFlags |= 0x100; /* enable single stepping */ diff --git a/modules/rostests/winetests/ntdll/info.c b/modules/rostests/winetests/ntdll/info.c index e641a8d0ff2..a1a131e41c6 100644 --- a/modules/rostests/winetests/ntdll/info.c +++ b/modules/rostests/winetests/ntdll/info.c @@ -353,6 +353,7 @@ static void test_query_process(void) last_pid = (DWORD_PTR)spi->UniqueProcessId; + disable_success_count ok( spi->dwThreadCount > 0, "Expected some threads for this process, got 0\n"); /* Loop through the threads, skip NT4 for now */ @@ -363,6 +364,7 @@ static void test_query_process(void) for ( j = 0; j < spi->dwThreadCount; j++) { k++; + disable_success_count ok ( spi->ti[j].ClientId.UniqueProcess == spi->UniqueProcessId, "The owning pid of the thread (%p) doesn't equal the pid (%p) of the process\n", spi->ti[j].ClientId.UniqueProcess, spi->UniqueProcessId); @@ -1691,12 +1693,14 @@ static void test_query_process_debug_flags(int argc, char **argv) for (;;) { ret = WaitForDebugEvent(&ev, 1000); + disable_success_count ok(ret, "WaitForDebugEvent failed, last error %#x.\n", GetLastError()); if (!ret) break; if (ev.dwDebugEventCode == LOAD_DLL_DEBUG_EVENT) break; ret = ContinueDebugEvent(ev.dwProcessId, ev.dwThreadId, DBG_CONTINUE); + disable_success_count ok(ret, "ContinueDebugEvent failed, last error %#x.\n", GetLastError()); if (!ret) break; } @@ -1733,6 +1737,7 @@ static void test_query_process_debug_flags(int argc, char **argv) for (j = 0; j < 100; j++) { ret = WaitForDebugEvent(&ev, 1000); + disable_success_count ok(ret || broken(GetLastError() == ERROR_SEM_TIMEOUT), "WaitForDebugEvent failed, last error %#x.\n", GetLastError()); if (!ret) break; @@ -1740,6 +1745,7 @@ static void test_query_process_debug_flags(int argc, char **argv) if (ev.dwDebugEventCode == EXIT_PROCESS_DEBUG_EVENT) break; ret = ContinueDebugEvent(ev.dwProcessId, ev.dwThreadId, DBG_CONTINUE); + disable_success_count ok(ret, "ContinueDebugEvent failed, last error %#x.\n", GetLastError()); if (!ret) break; } @@ -2248,12 +2254,12 @@ START_TEST(info) char **argv; int argc; - if(!InitFunctionPtrs()) - return; - argc = winetest_get_mainargs(&argv); if (argc >= 3) return; /* Child */ + if (!InitFunctionPtrs()) + return; + /* NtQuerySystemInformation */ /* 0x0 SystemBasicInformation */ diff --git a/modules/rostests/winetests/ntdll/process.c b/modules/rostests/winetests/ntdll/process.c index 41303b7e9ae..f10ed527834 100644 --- a/modules/rostests/winetests/ntdll/process.c +++ b/modules/rostests/winetests/ntdll/process.c @@ -123,6 +123,7 @@ static void test_NtSuspendProcess(char *process_name) ret = WaitForSingleObject(event, 200); ok(ret == WAIT_TIMEOUT, "Expected timeout, got: %d\n", ret); + disable_success_count for (;;) { ret = WaitForDebugEvent(&ev, INFINITE); @@ -158,6 +159,7 @@ static void test_NtSuspendProcess(char *process_name) ok(ret, "ContinueDebugEvent failed, last error %#x.\n", GetLastError()); ret = WaitForSingleObject(event, 200); + ros_skip_flaky ok(ret == WAIT_OBJECT_0, "Event was not signaled: %d\n", ret); TerminateProcess(info.hProcess, 0); diff --git a/modules/rostests/winetests/ole32/compobj.c b/modules/rostests/winetests/ole32/compobj.c index 49faabdece3..435ef803041 100644 --- a/modules/rostests/winetests/ole32/compobj.c +++ b/modules/rostests/winetests/ole32/compobj.c @@ -841,6 +841,7 @@ static DWORD WINAPI MessageFilter_MessagePending( DWORD dwPendingType) { trace("MessagePending\n"); + ros_skip_flaky todo_wine ok(0, "unexpected call\n"); return PENDINGMSG_WAITNOPROCESS; } diff --git a/modules/rostests/winetests/qmgr/job.c b/modules/rostests/winetests/qmgr/job.c index 6d8185a078d..2bee0a37ef6 100644 --- a/modules/rostests/winetests/qmgr/job.c +++ b/modules/rostests/winetests/qmgr/job.c @@ -359,6 +359,7 @@ static void test_CompleteLocal(void) hres = IBackgroundCopyJob_Resume(test_job); ok(hres == S_OK, "IBackgroundCopyJob_Resume\n"); + disable_success_count for (i = 0; i < timeout_sec; ++i) { hres = IBackgroundCopyJob_GetState(test_job, &state); @@ -428,6 +429,7 @@ static void test_CompleteLocalURL(void) hres = IBackgroundCopyJob_Resume(test_job); ok(hres == S_OK, "IBackgroundCopyJob_Resume\n"); + disable_success_count for (i = 0; i < timeout_sec; ++i) { hres = IBackgroundCopyJob_GetState(test_job, &state); @@ -570,6 +572,7 @@ static void test_HttpOptions(void) hr = IBackgroundCopyJob_Resume(test_job); ok(hr == S_OK, "got 0x%08x\n", hr); + disable_success_count for (i = 0; i < timeout; i++) { hr = IBackgroundCopyJob_GetState(test_job, &state); diff --git a/modules/rostests/winetests/quartz/filtermapper.c b/modules/rostests/winetests/quartz/filtermapper.c index 613d98d60f6..f5fe6464b86 100644 --- a/modules/rostests/winetests/quartz/filtermapper.c +++ b/modules/rostests/winetests/quartz/filtermapper.c @@ -39,6 +39,7 @@ static BOOL enum_find_filter(const WCHAR *wszFilterName, IEnumMoniker *pEnum) HRESULT hr; static const WCHAR wszFriendlyName[] = {'F','r','i','e','n','d','l','y','N','a','m','e',0}; + disable_success_count while(!found && IEnumMoniker_Next(pEnum, 1, &pMoniker, &nb) == S_OK) { IPropertyBag * pPropBagCat = NULL; diff --git a/modules/rostests/winetests/quartz/referenceclock.c b/modules/rostests/winetests/quartz/referenceclock.c index 4721675c055..f72cd905415 100644 --- a/modules/rostests/winetests/quartz/referenceclock.c +++ b/modules/rostests/winetests/quartz/referenceclock.c @@ -86,6 +86,7 @@ static void test_IReferenceClock_methods(const char * clockdesc, IReferenceClock /* FIXME: How much deviation should be allowed after a sleep? */ /* 0.3% is common, and 0.4% is sometimes observed. */ diff = time2 - time1; + ros_skip_flaky ok (9940000 <= diff && diff <= 10240000, "%s - Expected difference around 10000000, got %u\n", clockdesc, diff); } diff --git a/modules/rostests/winetests/riched20/editor.c b/modules/rostests/winetests/riched20/editor.c index 28f9849cba0..c2b30c06794 100644 --- a/modules/rostests/winetests/riched20/editor.c +++ b/modules/rostests/winetests/riched20/editor.c @@ -657,6 +657,7 @@ static void test_EM_POSFROMCHAR(void) } else { + ros_skip_flaky ok(HIWORD(result) == i * height, "EM_POSFROMCHAR reports y=%d, expected %d\n", HIWORD(result), i * height); ok(LOWORD(result) == xpos, "EM_POSFROMCHAR reports x=%d, expected 1\n", LOWORD(result)); } diff --git a/modules/rostests/winetests/rpcrt4/server.c b/modules/rostests/winetests/rpcrt4/server.c index 592ee9d9a90..05252877d4e 100644 --- a/modules/rostests/winetests/rpcrt4/server.c +++ b/modules/rostests/winetests/rpcrt4/server.c @@ -1553,7 +1553,9 @@ pointer_tests(void) names = NULL; get_names(&n, &names); ok(n == 2, "expected 2, got %d\n", n); + ros_skip_flaky ok(!strcmp(names[0], "Hello"), "expected Hello, got %s\n", names[0]); + ros_skip_flaky ok(!strcmp(names[1], "World!"), "expected World!, got %s\n", names[1]); MIDL_user_free(names[0]); MIDL_user_free(names[1]); @@ -1563,7 +1565,9 @@ pointer_tests(void) namesw = NULL; get_namesw(&n, &namesw); ok(n == 2, "expected 2, got %d\n", n); + ros_skip_flaky ok(!lstrcmpW(namesw[0], helloW), "expected Hello, got %s\n", wine_dbgstr_w(namesw[0])); + ros_skip_flaky ok(!lstrcmpW(namesw[1], worldW), "expected World!, got %s\n", wine_dbgstr_w(namesw[1])); MIDL_user_free(namesw[0]); MIDL_user_free(namesw[1]); @@ -1573,6 +1577,7 @@ pointer_tests(void) if (!is_interp) { /* broken in widl */ pa2 = a; + ros_skip_flaky ok(sum_pcarr2(4, &pa2) == 10, "RPC sum_pcarr2\n"); } diff --git a/modules/rostests/winetests/user32/listbox.c b/modules/rostests/winetests/user32/listbox.c index 8d8115131ef..3f83b5ac3ef 100644 --- a/modules/rostests/winetests/user32/listbox.c +++ b/modules/rostests/winetests/user32/listbox.c @@ -1720,6 +1720,7 @@ static void test_listbox_dlgdir(void) memset(tempBuffer, 0, MAX_PATH); driveletter = '\0'; SendMessageA(g_listBox, LB_GETTEXT, i, (LPARAM)itemBuffer); + if (!strstr(itemBuffer, ".exe")) continue; // skip downloaded/generated files from other tests res = SendMessageA(g_listBox, LB_SETCURSEL, i, 0); ok (res == i, "SendMessageA(LB_SETCURSEL, %d) failed\n", i); if (sscanf(itemBuffer, "[-%c-]", &driveletter) == 1) { diff --git a/modules/rostests/winetests/user32/msg.c b/modules/rostests/winetests/user32/msg.c index e66d0dcdcf1..7df4ba021db 100644 --- a/modules/rostests/winetests/user32/msg.c +++ b/modules/rostests/winetests/user32/msg.c @@ -10289,6 +10289,7 @@ static void test_timers(void) start = GetTickCount(); while (GetTickCount()-start < 1001 && GetMessageA(&msg, info.hWnd, 0, 0)) DispatchMessageA(&msg); +ros_skip_flaky todo_wine ok(abs(count-TIMER_COUNT_EXPECTED) < TIMER_COUNT_TOLERANCE /* xp */ || broken(abs(count-64) < TIMER_COUNT_TOLERANCE) /* most common */ @@ -16902,6 +16903,7 @@ static void test_hotkey(void) keybd_event(VK_LWIN, 0, KEYEVENTF_KEYUP, 0); while (PeekMessageA(&msg, NULL, 0, 0, PM_REMOVE)) { + ros_skip_flaky ok(msg.hwnd != NULL, "unexpected thread message %x\n", msg.message); DispatchMessageA(&msg); } diff --git a/modules/rostests/winetests/user32/win.c b/modules/rostests/winetests/user32/win.c index 74fb11251d0..4e98d2a7abf 100644 --- a/modules/rostests/winetests/user32/win.c +++ b/modules/rostests/winetests/user32/win.c @@ -149,6 +149,7 @@ static void check_wnd_state_(const char *file, int line, /* foreground can be moved to a different app pretty much at any time */ if (foreground && GetForegroundWindow() && GetWindowThreadProcessId(GetForegroundWindow(), NULL) == GetCurrentThreadId()) + disable_success_count ok_(file, line)(foreground == GetForegroundWindow(), "GetForegroundWindow() = %p\n", GetForegroundWindow()); ok_(file, line)(focus == GetFocus(), "GetFocus() = %p\n", GetFocus()); ok_(file, line)(capture == GetCapture(), "GetCapture() = %p\n", GetCapture()); @@ -164,6 +165,7 @@ static void check_active_state_(const char *file, int line, /* foreground can be moved to a different app pretty much at any time */ if (foreground && GetForegroundWindow() && GetWindowThreadProcessId(GetForegroundWindow(), NULL) == GetCurrentThreadId()) + disable_success_count ok_(file, line)(foreground == GetForegroundWindow(), "GetForegroundWindow() = %p\n", GetForegroundWindow()); ok_(file, line)(focus == GetFocus(), "GetFocus() = %p\n", GetFocus()); } @@ -774,6 +776,7 @@ static LRESULT WINAPI main_window_procA(HWND hwnd, UINT msg, WPARAM wparam, LPAR break; } case WM_WINDOWPOSCHANGED: + disable_success_count { RECT rc1, rc2; WINDOWPOS *winpos = (WINDOWPOS *)lparam; @@ -966,6 +969,8 @@ static void verify_window_info(const char *hook, HWND hwnd, const WINDOWINFO *in if (GetForegroundWindow()) ok(info->dwWindowStatus == status, "wrong dwWindowStatus: %04x != %04x active %p fg %p in hook %s\n", info->dwWindowStatus, status, GetActiveWindow(), GetForegroundWindow(), hook); + else + ok(1, "Just counting"); /* win2k and XP return broken border info in GetWindowInfo most of * the time, so there is no point in testing it. @@ -981,6 +986,7 @@ if (0) } ok(info->atomWindowType == GetClassLongA(hwnd, GCW_ATOM), "wrong atomWindowType for %p in hook %s\n", hwnd, hook); + todo_ros ok(info->wCreatorVersion == 0x0400 /* NT4, Win2000, XP, Win2003 */ || info->wCreatorVersion == 0x0500 /* Vista */, "wrong wCreatorVersion %04x for %p in hook %s\n", info->wCreatorVersion, hwnd, hook); @@ -3091,8 +3097,10 @@ static void test_SetActiveWindow(HWND hwnd) ok(hwnd2 == hwnd, "SetActiveWindow returned %p instead of %p\n", hwnd2, hwnd); if (!GetActiveWindow()) /* doesn't always work on vista */ { + ros_skip_flaky check_wnd_state(0, 0, 0, 0); hwnd2 = SetActiveWindow(hwnd); + ros_skip_flaky ok(hwnd2 == 0, "SetActiveWindow returned %p instead of 0\n", hwnd2); } check_wnd_state(hwnd, hwnd, hwnd, 0); diff --git a/modules/rostests/winetests/usp10/usp10.c b/modules/rostests/winetests/usp10/usp10.c index 9e870ee3a5f..3a74ca32556 100644 --- a/modules/rostests/winetests/usp10/usp10.c +++ b/modules/rostests/winetests/usp10/usp10.c @@ -1713,6 +1713,8 @@ static void test_ScriptShapeOpenType(HDC hdc) DeleteObject(hfont); } + hfont = NULL; + ros_skip_flaky test_valid = find_font_for_range(hdc, "Estrangelo Edessa", 71, test_syriac[0], &hfont, &hfont_orig, &fingerprint_estrangelo); if (hfont != NULL) { diff --git a/modules/rostests/winetests/wininet/ftp.c b/modules/rostests/winetests/wininet/ftp.c index d2118fadedd..baf20ffa118 100644 --- a/modules/rostests/winetests/wininet/ftp.c +++ b/modules/rostests/winetests/wininet/ftp.c @@ -74,10 +74,12 @@ static void test_connect(HINTERNET hInternet) hFtp = InternetConnectA(hInternet, "ftp.winehq.org", INTERNET_DEFAULT_FTP_PORT, "anonymous", NULL, INTERNET_SERVICE_FTP, INTERNET_FLAG_PASSIVE, 0); if (hFtp) /* some servers accept an empty password */ { + ros_skip_flaky ok ( GetLastError() == ERROR_SUCCESS, "ERROR_SUCCESS, got %d\n", GetLastError()); InternetCloseHandle(hFtp); } else + ros_skip_flaky ok ( GetLastError() == ERROR_INTERNET_LOGIN_FAILURE, "Expected ERROR_INTERNET_LOGIN_FAILURE, got %d\n", GetLastError()); @@ -109,7 +111,9 @@ static void test_connect(HINTERNET hInternet) SetLastError(0xdeadbeef); hFtp = InternetConnectA(hInternet, "ftp.winehq.org", INTERNET_DEFAULT_FTP_PORT, "anonymous", "IEUser@", INTERNET_SERVICE_FTP, INTERNET_FLAG_PASSIVE, 0); } + ros_skip_flaky ok ( hFtp != NULL, "InternetConnect failed : %d\n", GetLastError()); + ros_skip_flaky ok ( GetLastError() == ERROR_SUCCESS, "ERROR_SUCCESS, got %d\n", GetLastError()); InternetCloseHandle(hFtp); @@ -119,11 +123,13 @@ static void test_connect(HINTERNET hInternet) INTERNET_SERVICE_FTP, INTERNET_FLAG_PASSIVE, 0); if (!hFtp) { + ros_skip_flaky ok(GetLastError() == ERROR_INTERNET_LOGIN_FAILURE, "Expected ERROR_INTERNET_LOGIN_FAILURE, got %d\n", GetLastError()); } else { + ros_skip_flaky ok(GetLastError() == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", GetLastError()); InternetCloseHandle(hFtp); diff --git a/modules/rostests/winetests/wininet/http.c b/modules/rostests/winetests/wininet/http.c index 1f61b9218a1..1c5cbfe834b 100644 --- a/modules/rostests/winetests/wininet/http.c +++ b/modules/rostests/winetests/wininet/http.c @@ -351,6 +351,7 @@ static VOID WINAPI callback( DWORD dwStatusInformationLength ) { + ros_skip_flaky CHECK_EXPECT(dwInternetStatus); switch (dwInternetStatus) { @@ -4800,9 +4801,11 @@ static void test_async_read(int port) ret = InternetReadFileExA( req, &ib, 0, 0xdeadbeef ); if (!count) /* the first part should arrive immediately */ ok( ret, "InternetReadFileExA failed %u\n", GetLastError() ); + ros_skip_flaky if (!ret) { ok( GetLastError() == ERROR_IO_PENDING, "expected ERROR_IO_PENDING, got %u\n", GetLastError() ); + ros_skip_flaky CHECK_NOTIFIED( INTERNET_STATUS_RECEIVING_RESPONSE ); SET_EXPECT( INTERNET_STATUS_REQUEST_COMPLETE ); if (!pending_reads++) @@ -4891,6 +4894,7 @@ static void test_async_read(int port) { ok( GetLastError() == ERROR_IO_PENDING, "expected ERROR_IO_PENDING, got %u\n", GetLastError() ); ok( bytes == 0, "expected 0, got %u\n", bytes ); + ros_skip_flaky CHECK_NOTIFIED( INTERNET_STATUS_RECEIVING_RESPONSE ); SET_EXPECT( INTERNET_STATUS_REQUEST_COMPLETE ); if (!pending_reads++) @@ -4902,9 +4906,13 @@ static void test_async_read(int port) res = WaitForSingleObject( complete_event, INFINITE ); ok( res == WAIT_OBJECT_0, "expected WAIT_OBJECT_0, got %u\n", res ); ok( req_error == ERROR_SUCCESS, "expected ERROR_SUCCESS, got %u\n", req_error ); + ros_skip_flaky { todo_wine_if( pending_reads > 1 ) ok( bytes != 0, "expected bytes != 0\n" ); + } + ros_skip_flaky CHECK_NOTIFIED( INTERNET_STATUS_RESPONSE_RECEIVED ); + ros_skip_flaky CHECK_NOTIFIED( INTERNET_STATUS_REQUEST_COMPLETE ); } @@ -4913,6 +4921,7 @@ static void test_async_read(int port) if (!bytes) break; } + ros_skip_flaky ok( pending_reads == 1, "expected 1 pending read, got %u\n", pending_reads ); ok( !strcmp(buffer, page1), "unexpected buffer content\n" ); close_async_handle( ses, 2 ); @@ -6160,6 +6169,7 @@ static void test_security_flags(void) } HeapFree(GetProcessHeap(), 0, cert); + ros_skip_flaky CHECK_NOTIFIED2(INTERNET_STATUS_CONNECTING_TO_SERVER, 2); CHECK_NOTIFIED2(INTERNET_STATUS_CONNECTED_TO_SERVER, 2); CHECK_NOTIFIED2(INTERNET_STATUS_CLOSING_CONNECTION, 2); diff --git a/sdk/include/reactos/wine/test.h b/sdk/include/reactos/wine/test.h index 0b4c54019c1..52bad3a8f84 100644 --- a/sdk/include/reactos/wine/test.h +++ b/sdk/include/reactos/wine/test.h @@ -53,6 +53,8 @@ extern "C" { /* debug level */ extern int winetest_debug; +extern int report_success; + /* running in interactive mode? */ extern int winetest_interactive; @@ -65,8 +67,13 @@ extern int winetest_loop_todo(void); extern void winetest_end_todo(void); extern int winetest_get_mainargs( char*** pargv ); extern LONG winetest_get_failures(void); +extern LONG winetest_get_successes(void); extern void winetest_add_failures( LONG new_failures ); extern void winetest_wait_child_process( HANDLE process ); +extern void winetest_disable_success_count_start(); +extern int winetest_disable_success_count_end(); +extern int winetest_disable_success_count_get(); +extern int winetest_disable_success_count_set(); extern const char *wine_dbgstr_wn( const WCHAR *str, intptr_t n ); extern const char *wine_dbgstr_guid( const GUID *guid ); @@ -121,6 +128,7 @@ extern void __winetest_cdecl winetest_ok( int condition, const char *msg, ... ) extern void __winetest_cdecl winetest_skip( const char *msg, ... ) __attribute__((format (printf,1,2))); extern void __winetest_cdecl winetest_win_skip( const char *msg, ... ) __attribute__((format (printf,1,2))); extern void __winetest_cdecl winetest_trace( const char *msg, ... ) __attribute__((format (printf,1,2))); +extern void __winetest_cdecl winetest_print(const char* msg, ...) __attribute__((format(printf, 1, 2))); #else /* __GNUC__ */ # define WINETEST_PRINTF_ATTR(fmt,args) @@ -128,6 +136,7 @@ extern void __winetest_cdecl winetest_ok( int condition, const char *msg, ... ); extern void __winetest_cdecl winetest_skip( const char *msg, ... ); extern void __winetest_cdecl winetest_win_skip( const char *msg, ... ); extern void __winetest_cdecl winetest_trace( const char *msg, ... ); +extern void __winetest_cdecl winetest_print(const char* msg, ...); #endif /* __GNUC__ */ @@ -155,6 +164,14 @@ extern void __winetest_cdecl winetest_trace( const char *msg, ... ); #define todo_wine_if(is_todo) todo_if((is_todo) && !strcmp(winetest_platform, "wine")) #endif +#define ros_skip_flaky for (winetest_print("Skipping flaky test\n");0;) + +#define disable_success_count for (winetest_disable_success_count_start(); \ + (winetest_disable_success_count_get() ? \ + winetest_disable_success_count_end() : \ + winetest_disable_success_count_set()); \ + ) + #define ARRAY_SIZE(x) (sizeof(x) / sizeof((x)[0])) #ifdef NONAMELESSUNION @@ -241,7 +258,7 @@ int winetest_interactive = 0; const char *winetest_platform = "windows"; /* report successful tests (BOOL) */ -static int report_success = 0; +int report_success = 0; /* passing arguments around */ static int winetest_argc; @@ -261,6 +278,7 @@ typedef struct const char* current_file; /* file of current check */ int current_line; /* line of current check */ unsigned int todo_level; /* current todo nesting level */ + unsigned int nocount_level; int todo_do_loop; char *str_pos; /* position in debug buffer */ char strings[2000]; /* buffer for debug strings */ @@ -368,6 +386,7 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args ) data->current_file, data->current_line ); vfprintf(stdout, msg, args); } + if (data->nocount_level == 0) InterlockedIncrement(&todo_successes); return 1; } @@ -384,9 +403,10 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args ) } else { - if (report_success) + if (report_success && data->nocount_level == 0) fprintf( stdout, __winetest_file_line_prefix ": Test succeeded\n", data->current_file, data->current_line); + if (data->nocount_level == 0) InterlockedIncrement(&successes); return 1; } @@ -416,6 +436,17 @@ void __winetest_cdecl winetest_trace( const char *msg, ... ) } } +void __winetest_cdecl winetest_print(const char* msg, ...) +{ + __winetest_va_list valist; + tls_data* data = get_tls_data(); + + fprintf(stdout, __winetest_file_line_prefix ": ", data->current_file, data->current_line); + __winetest_va_start(valist, msg); + vfprintf(stdout, msg, valist); + __winetest_va_end(valist); +} + void winetest_vskip( const char *msg, __winetest_va_list args ) { tls_data* data=get_tls_data(); @@ -480,6 +511,11 @@ LONG winetest_get_failures(void) return failures; } +LONG winetest_get_successes(void) +{ + return successes; +} + void winetest_add_failures( LONG new_failures ) { while (new_failures-- > 0) @@ -512,6 +548,28 @@ void winetest_wait_child_process( HANDLE process ) } } +void winetest_disable_success_count_start() +{ + get_tls_data()->nocount_level <<= 1; +} + +int winetest_disable_success_count_end() +{ + get_tls_data()->nocount_level >>= 1; + return 0; +} + +int winetest_disable_success_count_get() +{ + return get_tls_data()->nocount_level & 1; +} + +int winetest_disable_success_count_set() +{ + get_tls_data()->nocount_level |= 1; + return 1; +} + const char *wine_dbgstr_wn( const WCHAR *str, intptr_t n ) { char *dst, *res;

1 0

[reactos] 01/01: [SHELL32] Delete unused CNewMenu::m_wszPath
by Katayama Hirofumi MZ 03 Dec '19

03 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c082d08cf34cf09e00300… commit c082d08cf34cf09e003009dc5862fc7e0a6c38c1 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Tue Dec 3 13:46:22 2019 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Tue Dec 3 13:46:35 2019 +0900 [SHELL32] Delete unused CNewMenu::m_wszPath --- dll/win32/shell32/CNewMenu.cpp | 1 - dll/win32/shell32/CNewMenu.h | 1 - 2 files changed, 2 deletions(-) diff --git a/dll/win32/shell32/CNewMenu.cpp b/dll/win32/shell32/CNewMenu.cpp index b0e8052070c..87559e017f4 100644 --- a/dll/win32/shell32/CNewMenu.cpp +++ b/dll/win32/shell32/CNewMenu.cpp @@ -27,7 +27,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(shell); CNewMenu::CNewMenu() : m_pidlFolder(NULL), - m_wszPath(NULL), m_pItems(NULL), m_pLinkItem(NULL), m_pSite(NULL), diff --git a/dll/win32/shell32/CNewMenu.h b/dll/win32/shell32/CNewMenu.h index 45aaa624f26..6d0bbe6b927 100644 --- a/dll/win32/shell32/CNewMenu.h +++ b/dll/win32/shell32/CNewMenu.h @@ -54,7 +54,6 @@ private: }; LPITEMIDLIST m_pidlFolder; - LPWSTR m_wszPath; SHELLNEW_ITEM *m_pItems; SHELLNEW_ITEM *m_pLinkItem; // Points to the link handler item in the m_pItems list. CComPtr<IUnknown> m_pSite;

1 0

[reactos] 01/01: [VIDEOPRT] Protect access to HwResetAdaptersList with spinlock.
by Hermès Bélusca-Maïto 03 Dec '19

03 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=44314189991059109e60f… commit 44314189991059109e60f4b101171aaf1c735f8a Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Tue Dec 3 02:08:34 2019 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Dec 3 02:46:38 2019 +0100 [VIDEOPRT] Protect access to HwResetAdaptersList with spinlock. --- win32ss/drivers/videoprt/dispatch.c | 45 ++++++++++++++++++++++++++++--------- win32ss/drivers/videoprt/videoprt.c | 13 ++++++----- win32ss/drivers/videoprt/videoprt.h | 1 + 3 files changed, 44 insertions(+), 15 deletions(-) diff --git a/win32ss/drivers/videoprt/dispatch.c b/win32ss/drivers/videoprt/dispatch.c index 42f7023b0b1..b116cf1100b 100644 --- a/win32ss/drivers/videoprt/dispatch.c +++ b/win32ss/drivers/videoprt/dispatch.c @@ -74,22 +74,35 @@ IntVideoPortResetDisplayParametersEx( _In_ ULONG Rows, _In_ BOOLEAN CalledByInbv) { - BOOLEAN Success = TRUE; + BOOLEAN Success = TRUE; // Suppose we don't need to perform a full reset. + KIRQL OldIrql; PLIST_ENTRY PrevEntry, Entry; PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; PVIDEO_PORT_DRIVER_EXTENSION DriverExtension; + /* Check if we are at dispatch level or lower, and acquire the lock */ + OldIrql = KeGetCurrentIrql(); + if (OldIrql <= DISPATCH_LEVEL) + { + /* Loop until the lock is free, then raise IRQL to dispatch level */ + while (!KeTestSpinLock(&HwResetAdaptersLock)); + KeRaiseIrql(DISPATCH_LEVEL, &OldIrql); + } + KeAcquireSpinLockAtDpcLevel(&HwResetAdaptersLock); + + /* Bail out early if we don't have any resettable adapter */ if (IsListEmpty(&HwResetAdaptersList)) - return FALSE; + { + Success = FALSE; // No adapter found: request HAL to perform a full reset. + goto Quit; + } + /* + * If we have been unexpectedly called via a callback from + * InbvAcquireDisplayOwnership(), start monitoring INBV. + */ if (CalledByInbv) - { - /* - * We have been unexpectedly called via a callback from - * InbvAcquireDisplayOwnership(): start monitoring INBV. - */ InbvMonitoring = TRUE; - } for (PrevEntry = &HwResetAdaptersList, Entry = PrevEntry->Flink; Entry != &HwResetAdaptersList; @@ -102,7 +115,10 @@ IntVideoPortResetDisplayParametersEx( */ // #define IS_ALIGNED(addr, align) (((ULONG64)(addr) & (align - 1)) == 0) if (((ULONG_PTR)Entry & (sizeof(ULONG_PTR) - 1)) != 0) - return FALSE; + { + Success = FALSE; // We failed: request HAL to perform a full reset. + goto Quit; + } DeviceExtension = CONTAINING_RECORD(Entry, VIDEO_PORT_DEVICE_EXTENSION, @@ -118,7 +134,10 @@ IntVideoPortResetDisplayParametersEx( ASSERT(DriverExtension); if (DeviceExtension->HwResetListEntry.Blink != PrevEntry) - _SEH2_YIELD(return FALSE); + { + Success = FALSE; // We failed: request HAL to perform a full reset. + _SEH2_YIELD(goto Quit); + } if ((DeviceExtension->DeviceOpened >= 1) && (DriverExtension->InitializationData.HwResetHw != NULL)) @@ -134,6 +153,12 @@ IntVideoPortResetDisplayParametersEx( _SEH2_END; } +Quit: + /* Release the lock and restore the old IRQL if we were at dispatch level or lower */ + KeReleaseSpinLockFromDpcLevel(&HwResetAdaptersLock); + if (OldIrql <= DISPATCH_LEVEL) + KeLowerIrql(OldIrql); + return Success; } diff --git a/win32ss/drivers/videoprt/videoprt.c b/win32ss/drivers/videoprt/videoprt.c index 6e75d563ea9..b4207309e93 100644 --- a/win32ss/drivers/videoprt/videoprt.c +++ b/win32ss/drivers/videoprt/videoprt.c @@ -33,6 +33,7 @@ PKPROCESS CsrProcess = NULL; ULONG VideoPortDeviceNumber = 0; KMUTEX VideoPortInt10Mutex; +KSPIN_LOCK HwResetAdaptersLock; RTL_STATIC_LIST_HEAD(HwResetAdaptersList); /* PRIVATE FUNCTIONS **********************************************************/ @@ -410,8 +411,9 @@ IntVideoPortFindAdapter( InitializeListHead(&DeviceExtension->HwResetListEntry); if (DriverExtension->InitializationData.HwResetHw != NULL) { - InsertTailList(&HwResetAdaptersList, - &DeviceExtension->HwResetListEntry); + ExInterlockedInsertTailList(&HwResetAdaptersList, + &DeviceExtension->HwResetListEntry, + &HwResetAdaptersLock); } /* Query children of the device. */ @@ -471,14 +473,15 @@ VideoPortInitialize( NTSTATUS Status; PVIDEO_PORT_DRIVER_EXTENSION DriverExtension; BOOLEAN PnpDriver = FALSE, LegacyDetection = FALSE; - static BOOLEAN Int10MutexInitialized; + static BOOLEAN FirstInitialization; TRACE_(VIDEOPRT, "VideoPortInitialize\n"); - if (!Int10MutexInitialized) + if (!FirstInitialization) { KeInitializeMutex(&VideoPortInt10Mutex, 0); - Int10MutexInitialized = TRUE; + KeInitializeSpinLock(&HwResetAdaptersLock); + FirstInitialization = TRUE; } /* As a first thing do parameter checks. */ diff --git a/win32ss/drivers/videoprt/videoprt.h b/win32ss/drivers/videoprt/videoprt.h index 94289dc031a..8035278e349 100644 --- a/win32ss/drivers/videoprt/videoprt.h +++ b/win32ss/drivers/videoprt/videoprt.h @@ -246,6 +246,7 @@ IntVideoPortMapPhysicalMemory( extern PKPROCESS CsrProcess; extern ULONG VideoPortDeviceNumber; extern KMUTEX VideoPortInt10Mutex; +extern KSPIN_LOCK HwResetAdaptersLock; extern LIST_ENTRY HwResetAdaptersList; VOID FASTCALL

1 0

[reactos] 01/01: [NTOS:PO] PopShutdownHandler(): Fully screen-center images. Addendum to 55aed4f8. CORE-16342
by Serge Gautherie 03 Dec '19

03 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=ad21227ac8babb9b5f45c… commit ad21227ac8babb9b5f45c83e0d26a3c766b38657 Author: Serge Gautherie <reactos-git_serge_171003(a)gautherie.fr> AuthorDate: Mon Dec 2 19:21:39 2019 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Dec 3 02:10:37 2019 +0100 [NTOS:PO] PopShutdownHandler(): Fully screen-center images. Addendum to 55aed4f8. CORE-16342 --- ntoskrnl/po/poshtdwn.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ntoskrnl/po/poshtdwn.c b/ntoskrnl/po/poshtdwn.c index aca63dae7ee..23071faad28 100644 --- a/ntoskrnl/po/poshtdwn.c +++ b/ntoskrnl/po/poshtdwn.c @@ -168,8 +168,9 @@ PopShutdownHandler(VOID) Logo2 = InbvGetResourceAddress(IDB_LOGO_DEFAULT); if ((Logo1) && (Logo2)) { - InbvBitBlt(Logo1, 211, 352); - InbvBitBlt(Logo2, 222, 111); + /* 16px space between logo and message */ + InbvBitBlt(Logo1, 213, 354); + InbvBitBlt(Logo2, 225, 114); } } else

1 0

[reactos] 01/01: [CPL][INPUT] Fix bullet (#2124)
by Katayama Hirofumi MZ 02 Dec '19

02 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4420f4e0cad506c8b34e6… commit 4420f4e0cad506c8b34e6d35bf36743546255df9 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Mon Dec 2 18:04:41 2019 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Mon Dec 2 18:04:41 2019 +0900 [CPL][INPUT] Fix bullet (#2124) CORE-13807 --- dll/cpl/input/settings_page.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dll/cpl/input/settings_page.c b/dll/cpl/input/settings_page.c index 8c093986475..28b9f9969db 100644 --- a/dll/cpl/input/settings_page.c +++ b/dll/cpl/input/settings_page.c @@ -64,7 +64,7 @@ CreateLayoutIcon(LPWSTR szLayout, BOOL bIsDefault) ExtTextOutW(hdc, rect.left, rect.top, ETO_OPAQUE, &rect, L"", 0, NULL); SelectObject(hdc, hFont); - DrawTextW(hdc, L"\x2022", 1, &rect, DT_SINGLELINE | DT_CENTER | DT_VCENTER); + DrawFrameControl(hdc, &rect, DFC_MENU, DFCS_MENUBULLET); } else {

1 0

[reactos] 01/01: [NTOSKRNL][INBV] Fix IDB_SHUTDOWN_MSG position
by Katayama Hirofumi MZ 02 Dec '19

02 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=55aed4f8fa919ce3b210e… commit 55aed4f8fa919ce3b210e14bff7aec1919202848 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Mon Dec 2 16:55:02 2019 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Mon Dec 2 16:55:02 2019 +0900 [NTOSKRNL][INBV] Fix IDB_SHUTDOWN_MSG position Make "shutdown.bmp" (IDB_SHUTDOWN_MSG: 213x11px) horizontally centered. --- ntoskrnl/po/poshtdwn.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ntoskrnl/po/poshtdwn.c b/ntoskrnl/po/poshtdwn.c index 17adbc2a415..aca63dae7ee 100644 --- a/ntoskrnl/po/poshtdwn.c +++ b/ntoskrnl/po/poshtdwn.c @@ -168,7 +168,7 @@ PopShutdownHandler(VOID) Logo2 = InbvGetResourceAddress(IDB_LOGO_DEFAULT); if ((Logo1) && (Logo2)) { - InbvBitBlt(Logo1, 220, 352); + InbvBitBlt(Logo1, 211, 352); InbvBitBlt(Logo2, 222, 111); } }

1 0

[reactos] 01/01: [WIN32K][VIDEOPRT] Improve initialization and interfacing with INBV. CORE-12149
by Hermès Bélusca-Maïto 02 Dec '19

02 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0ad65796bbb57be739840… commit 0ad65796bbb57be739840c6f666cd9f8f8a1efda Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Tue Nov 26 02:49:35 2019 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Mon Dec 2 02:33:20 2019 +0100 [WIN32K][VIDEOPRT] Improve initialization and interfacing with INBV. CORE-12149 VIDEOPRT: ========= Improve interfacing with INBV, so as to detect when an external module acquired INBV display ownership, and whether ownership is being released later on. (This does NOT rely on hooking!) For this purpose we improve the IntVideoPortResetDisplayParameters(Ex) callback that gets registered with an InbvNotifyDisplayOwnershipLost() call during initialization, and we add a monitoring thread. The callback is called whenever an external module calls InbvAcquireDisplayOwnership(), for example the bugcheck code or the KDBG debugger in SCREEN mode. When this happens, a flag that tells the monitoring thread to start monitoring INBV is set (ReactOS-specific), and the display adapters get reset with HwResetHw() (as done on Windows). Due to the fact that this INBV callback can be called at *ANY* IRQL, we cannot use dispatcher synchronization mechanisms such as events to tell the INBV monitoring thread to start its operations, so we need to rely instead on a flag to be set. And, since INBV doesn't provide with any proper callback/notification system either, we need to actively monitor its state by pooling. To reduce the load on the system the monitoring thread performs 1-second waits between each check for the flag set by the INBV callback, and during checking the INBV ownership status. When the INBV ownership is detected to be released by an external module, the INBV callback is re-registered (this is *MANDATORY* since the external module has called InbvNotifyDisplayOwnershipLost() with a different callback parameter!), and then we callout to Win32k for re-enabling the display. This has the virtue of correctly resetting the display once the KDBG debugger in SCREEN mode is being exited, and fixes CORE-12149 . The following additional fixes were needed: VIDEOPRT & WIN32K: ================== Remove the registration with INBV that was previously done in a ReactOS- specific hacked IRP_MJ_WRITE call; it is now done correctly during the video device opening done by EngpRegisterGraphicsDevice() in the VIDEOPRT's IRP_MJ_CREATE handler, as done on Windows. WIN32K: ======= - Stub the VideoPortCallout() support, for VIDEOPRT -> WIN32 callbacks. This function gets registered with VIDEOPRT through an IOCTL_VIDEO_INIT_WIN32K_CALLBACKS call in EngpRegisterGraphicsDevice(). - Only partially implement the 'VideoFindAdapterCallout' case, that just re-enables the primary display by refreshing it (using the new function UserRefreshDisplay()). VIDEOPRT: ========= - PVIDEO_WIN32K_CALLOUT is an NTAPI (stdcall) callback. - In the IntVideoPortResetDisplayParameters(Ex) callback, reset all the "resettable" adapters registered in the HwResetAdaptersList list. We thus get rid of the global ResetDisplayParametersDeviceExtension. - Make the IntVideoPortResetDisplayParameters(Ex) callback slightly more robust (using SEH) against potential HwResetListEntry list corruption or invalid DriverExtension->InitializationData.HwResetHw() that would otherwise trigger a BSOD, and this would be disastrous since that callback is precisely called when INBV is acquired, typically when the BSOD code initializes the display for displaying its information... Extras: - Validate the IrpStack->MajorFunction in IntVideoPortDispatchDeviceControl() and implement IRP_MJ_SHUTDOWN handling. Stub out the other IOCTLs that are handled by VIDEOPRT only (and not by the miniports). - VIDEOPRT doesn't require IRP_MJ_INTERNAL_DEVICE_CONTROL (unused). - Implement IOCTL_VIDEO_PREPARE_FOR_EARECOVERY that just resets the display to standard VGA 80x25 text mode. --- sdk/include/psdk/ntddvdeo.h | 4 +- win32ss/drivers/videoprt/CMakeLists.txt | 1 + win32ss/drivers/videoprt/dispatch.c | 438 +++++++++++++++++++++++++------- win32ss/drivers/videoprt/int10.c | 2 +- win32ss/drivers/videoprt/videoprt.c | 28 +- win32ss/drivers/videoprt/videoprt.h | 21 +- win32ss/gdi/eng/device.c | 99 +++++++- win32ss/user/ntuser/display.c | 39 +++ 8 files changed, 509 insertions(+), 123 deletions(-) diff --git a/sdk/include/psdk/ntddvdeo.h b/sdk/include/psdk/ntddvdeo.h index 8fcd361f502..5e2b63b7214 100644 --- a/sdk/include/psdk/ntddvdeo.h +++ b/sdk/include/psdk/ntddvdeo.h @@ -75,7 +75,7 @@ extern "C" { CTL_CODE(FILE_DEVICE_VIDEO, 0x0b, METHOD_BUFFERED, FILE_ANY_ACCESS) #define IOCTL_VIDEO_DISABLE_CURSOR \ - CTL_CODE (FILE_DEVICE_VIDEO, 0x109, METHOD_BUFFERED, FILE_ANY_ACCESS) + CTL_CODE(FILE_DEVICE_VIDEO, 0x109, METHOD_BUFFERED, FILE_ANY_ACCESS) #define IOCTL_VIDEO_DISABLE_POINTER \ CTL_CODE(FILE_DEVICE_VIDEO, 0x10f, METHOD_BUFFERED, FILE_ANY_ACCESS) @@ -254,7 +254,7 @@ typedef struct _VIDEO_WIN32K_CALLBACKS_PARAMS { typedef VOID -(*PVIDEO_WIN32K_CALLOUT)( +(NTAPI *PVIDEO_WIN32K_CALLOUT)( _In_ PVOID Params); typedef struct _VIDEO_WIN32K_CALLBACKS { diff --git a/win32ss/drivers/videoprt/CMakeLists.txt b/win32ss/drivers/videoprt/CMakeLists.txt index 664f8cb7c62..be116547d59 100644 --- a/win32ss/drivers/videoprt/CMakeLists.txt +++ b/win32ss/drivers/videoprt/CMakeLists.txt @@ -29,6 +29,7 @@ add_library(videoprt MODULE ${CMAKE_CURRENT_BINARY_DIR}/videoprt.def) set_module_type(videoprt kernelmodedriver) +target_link_libraries(videoprt ${PSEH_LIB}) add_importlibs(videoprt ntoskrnl hal) add_pch(videoprt videoprt.h SOURCE) add_cd_file(TARGET videoprt DESTINATION reactos/system32/drivers FOR all) diff --git a/win32ss/drivers/videoprt/dispatch.c b/win32ss/drivers/videoprt/dispatch.c index d1682549185..42f7023b0b1 100644 --- a/win32ss/drivers/videoprt/dispatch.c +++ b/win32ss/drivers/videoprt/dispatch.c @@ -22,6 +22,7 @@ #include "videoprt.h" #include <ndk/inbvfuncs.h> +#include <ndk/obfuncs.h> #include <ndk/psfuncs.h> #define NDEBUG @@ -29,40 +30,280 @@ /* GLOBAL VARIABLES ***********************************************************/ -PVIDEO_PORT_DEVICE_EXTENSION ResetDisplayParametersDeviceExtension = NULL; -PVIDEO_WIN32K_CALLOUT Win32kCallout; +static PVIDEO_WIN32K_CALLOUT Win32kCallout = NULL; +static HANDLE InbvThreadHandle = NULL; +static BOOLEAN InbvMonitoring = FALSE; /* PRIVATE FUNCTIONS **********************************************************/ +static VOID +VideoPortWin32kCallout( + _In_ PVIDEO_WIN32K_CALLBACKS_PARAMS CallbackParams) +{ + if (!Win32kCallout) + return; + + /* Perform the call in the context of CSRSS */ + if (!CsrProcess) + return; + + KeAttachProcess(CsrProcess); + Win32kCallout(CallbackParams); + KeDetachProcess(); +} + /* - * Reset display to blue screen + * Reinitialize the display to base VGA mode. + * + * Returns TRUE if it completely resets the adapter to the given character mode. + * Returns FALSE otherwise, indicating that the HAL should perform the VGA mode + * reset itself after HwVidResetHw() returns control. + * + * This callback has been registered with InbvNotifyDisplayOwnershipLost() + * and is called by InbvAcquireDisplayOwnership(), typically when the bugcheck + * code regains display access. Therefore this routine can be called at any + * IRQL, and in particular at IRQL = HIGH_LEVEL. This routine must also reside + * completely in non-paged pool, and cannot perform the following actions: + * Allocate memory, access pageable memory, use any synchronization mechanisms + * or call any routine that must execute at IRQL = DISPATCH_LEVEL or below. */ -BOOLEAN +static BOOLEAN NTAPI -IntVideoPortResetDisplayParameters(ULONG Columns, ULONG Rows) +IntVideoPortResetDisplayParametersEx( + _In_ ULONG Columns, + _In_ ULONG Rows, + _In_ BOOLEAN CalledByInbv) { + BOOLEAN Success = TRUE; + PLIST_ENTRY PrevEntry, Entry; + PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; PVIDEO_PORT_DRIVER_EXTENSION DriverExtension; - if (ResetDisplayParametersDeviceExtension == NULL) + if (IsListEmpty(&HwResetAdaptersList)) return FALSE; - DriverExtension = ResetDisplayParametersDeviceExtension->DriverExtension; + if (CalledByInbv) + { + /* + * We have been unexpectedly called via a callback from + * InbvAcquireDisplayOwnership(): start monitoring INBV. + */ + InbvMonitoring = TRUE; + } - if (DriverExtension->InitializationData.HwResetHw != NULL) + for (PrevEntry = &HwResetAdaptersList, Entry = PrevEntry->Flink; + Entry != &HwResetAdaptersList; + PrevEntry = Entry, Entry = Entry->Flink) { - if (DriverExtension->InitializationData.HwResetHw( - &ResetDisplayParametersDeviceExtension->MiniPortDeviceExtension, - Columns, Rows)) + /* + * Check whether the entry address is properly aligned, + * the device and driver extensions must be readable and + * the device extension properly back-linked to the last entry. + */ +// #define IS_ALIGNED(addr, align) (((ULONG64)(addr) & (align - 1)) == 0) + if (((ULONG_PTR)Entry & (sizeof(ULONG_PTR) - 1)) != 0) + return FALSE; + + DeviceExtension = CONTAINING_RECORD(Entry, + VIDEO_PORT_DEVICE_EXTENSION, + HwResetListEntry); + /* + * As this function can be called as part of the INBV initialization + * by the bugcheck code, avoid any problems and protect all accesses + * within SEH. + */ + _SEH2_TRY + { + DriverExtension = DeviceExtension->DriverExtension; + ASSERT(DriverExtension); + + if (DeviceExtension->HwResetListEntry.Blink != PrevEntry) + _SEH2_YIELD(return FALSE); + + if ((DeviceExtension->DeviceOpened >= 1) && + (DriverExtension->InitializationData.HwResetHw != NULL)) + { + Success &= DriverExtension->InitializationData.HwResetHw( + &DeviceExtension->MiniPortDeviceExtension, + Columns, Rows); + } + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { - ResetDisplayParametersDeviceExtension = NULL; - return TRUE; } + _SEH2_END; } - ResetDisplayParametersDeviceExtension = NULL; - return FALSE; + return Success; } +/* This callback is registered with InbvNotifyDisplayOwnershipLost() */ +static BOOLEAN +NTAPI +IntVideoPortResetDisplayParameters(ULONG Columns, ULONG Rows) +{ + /* Call the extended function, specifying we were called by INBV */ + return IntVideoPortResetDisplayParametersEx(Columns, Rows, TRUE); +} + +/* + * (Adapted for ReactOS/Win2k3 from an original comment + * by Gé van Geldorp, June 2003, r4937) + * + * DISPLAY OWNERSHIP + * + * So, who owns the physical display and is allowed to write to it? + * + * In NT 5.x (Win2k/Win2k3), upon boot INBV/BootVid owns the display, unless + * /NOGUIBOOT has been specified in the boot command line. Later in the boot + * sequence, WIN32K.SYS opens the DISPLAY device. This open call ends up in + * VIDEOPRT.SYS. This component takes ownership of the display by calling + * InbvNotifyDisplayOwnershipLost() -- effectively telling INBV to release + * ownership of the display it previously had. From that moment on, the display + * is owned by that component and can be switched to graphics mode. The display + * is not supposed to return to text mode, except in case of a bugcheck. + * The bugcheck code calls InbvAcquireDisplayOwnership() so as to make INBV + * re-take display ownership, and calls back the function previously registered + * by VIDEOPRT.SYS with InbvNotifyDisplayOwnershipLost(). After the bugcheck, + * execution is halted. So, under NT, the only possible sequence of display + * modes is text mode -> graphics mode -> text mode (the latter hopefully + * happening very infrequently). + * + * In ReactOS things are a little bit different. We want to have a functional + * interactive text mode. We should be able to switch back and forth from + * text mode to graphics mode when a GUI app is started and then finished. + * Also, when the system bugchecks in graphics mode we want to switch back to + * text mode and show the bugcheck information. Last but not least, when using + * KDBG in /DEBUGPORT=SCREEN mode, breaking into the debugger would trigger a + * switch to text mode, and the user would expect that by continuing execution + * a switch back to graphics mode is done. + */ +static VOID +NTAPI +InbvMonitorThread( + _In_ PVOID Context) +{ + VIDEO_WIN32K_CALLBACKS_PARAMS CallbackParams; + LARGE_INTEGER Delay; + USHORT i; + + KeSetPriorityThread(KeGetCurrentThread(), LOW_REALTIME_PRIORITY); + + while (TRUE) + { + /* + * During one second, check the INBV status each 100 milliseconds, + * then revert to 1 second delay. + */ + i = 10; + Delay.QuadPart = (LONGLONG)-100*1000*10; // 100 millisecond delay + while (!InbvMonitoring) + { + KeDelayExecutionThread(KernelMode, FALSE, &Delay); + + if ((i > 0) && (--i == 0)) + Delay.QuadPart = (LONGLONG)-1*1000*1000*10; // 1 second delay + } + + /* + * Loop while the display is owned by INBV. We cannot do anything else + * than polling since INBV does not offer a proper notification system. + * + * During one second, check the INBV status each 100 milliseconds, + * then revert to 1 second delay. + */ + i = 10; + Delay.QuadPart = (LONGLONG)-100*1000*10; // 100 millisecond delay + while (InbvCheckDisplayOwnership()) + { + KeDelayExecutionThread(KernelMode, FALSE, &Delay); + + if ((i > 0) && (--i == 0)) + Delay.QuadPart = (LONGLONG)-1*1000*1000*10; // 1 second delay + } + + /* Reset the monitoring */ + InbvMonitoring = FALSE; + + /* + * Somebody released INBV display ownership, usually by invoking + * InbvNotifyDisplayOwnershipLost(). However the caller of this + * function certainly specified a different callback than ours. + * As we are going to be the only owner of the active display, + * we need to re-register our own display reset callback. + */ + InbvNotifyDisplayOwnershipLost(IntVideoPortResetDisplayParameters); + + /* Tell Win32k to reset the display */ + CallbackParams.CalloutType = VideoFindAdapterCallout; + // CallbackParams.PhysDisp = NULL; + CallbackParams.Param = (ULONG_PTR)TRUE; // TRUE: Re-enable display; FALSE: Disable display. + VideoPortWin32kCallout(&CallbackParams); + } + + // FIXME: See IntVideoPortInbvCleanup(). + // PsTerminateSystemThread(STATUS_SUCCESS); +} + +static NTSTATUS +IntVideoPortInbvInitialize(VOID) +{ + /* Create the INBV monitoring thread if needed */ + if (!InbvThreadHandle) + { + NTSTATUS Status; + OBJECT_ATTRIBUTES ObjectAttributes = RTL_CONSTANT_OBJECT_ATTRIBUTES(NULL, OBJ_KERNEL_HANDLE); + + Status = PsCreateSystemThread(&InbvThreadHandle, + 0, + &ObjectAttributes, + NULL, + NULL, + InbvMonitorThread, + NULL); + if (!NT_SUCCESS(Status)) + InbvThreadHandle = NULL; + } + + /* Re-register the display reset callback with INBV */ + InbvNotifyDisplayOwnershipLost(IntVideoPortResetDisplayParameters); + + return STATUS_SUCCESS; +} + +static NTSTATUS +IntVideoPortInbvCleanup( + IN PDEVICE_OBJECT DeviceObject) +{ + PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; + // HANDLE ThreadHandle; + + DeviceExtension = (PVIDEO_PORT_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + if ((DeviceExtension->DeviceOpened >= 1) && + (InterlockedDecrement((PLONG)&DeviceExtension->DeviceOpened) == 0)) + { + // RemoveEntryList(&DeviceExtension->HwResetListEntry); + InbvNotifyDisplayOwnershipLost(NULL); + IntVideoPortResetDisplayParametersEx(80, 50, FALSE); + // or InbvAcquireDisplayOwnership(); ? + } + +#if 0 + // TODO: Find the best way to communicate the request. + /* Signal the INBV monitoring thread and wait for it to terminate */ + ThreadHandle = InterlockedExchangePointer((PVOID*)&InbvThreadHandle, NULL); + if (ThreadHandle) + { + KeWaitForSingleObject(&ThreadHandle, Executive, KernelMode, FALSE, NULL); + /* Close its handle */ + ObCloseHandle(ThreadHandle, KernelMode); + } +#endif + + return STATUS_SUCCESS; +} + + NTSTATUS NTAPI IntVideoPortAddDevice( @@ -101,36 +342,39 @@ IntVideoPortDispatchOpen( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) { + NTSTATUS Status; PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; PVIDEO_PORT_DRIVER_EXTENSION DriverExtension; - NTSTATUS Status; TRACE_(VIDEOPRT, "IntVideoPortDispatchOpen\n"); - if (CsrssInitialized == FALSE) + if (!CsrProcess) { /* * We know the first open call will be from the CSRSS process * to let us know its handle. */ - INFO_(VIDEOPRT, "Referencing CSRSS\n"); - Csrss = (PKPROCESS)PsGetCurrentProcess(); - INFO_(VIDEOPRT, "Csrss %p\n", Csrss); + CsrProcess = (PKPROCESS)PsGetCurrentProcess(); + ObReferenceObject(CsrProcess); + INFO_(VIDEOPRT, "CsrProcess 0x%p\n", CsrProcess); Status = IntInitializeVideoAddressSpace(); if (!NT_SUCCESS(Status)) { ERR_(VIDEOPRT, "IntInitializeVideoAddressSpace() failed: 0x%lx\n", Status); + ObDereferenceObject(CsrProcess); + CsrProcess = NULL; return Status; } - - CsrssInitialized = TRUE; } DeviceExtension = (PVIDEO_PORT_DEVICE_EXTENSION)DeviceObject->DeviceExtension; DriverExtension = DeviceExtension->DriverExtension; + // FIXME: (Re-)initialize INBV only if DeviceObject doesn't belong to a mirror driver. + IntVideoPortInbvInitialize(); + if (DriverExtension->InitializationData.HwInitialize(&DeviceExtension->MiniPortDeviceExtension)) { Irp->IoStatus.Status = STATUS_SUCCESS; @@ -161,30 +405,19 @@ IntVideoPortDispatchClose( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp) { - PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; - TRACE_(VIDEOPRT, "IntVideoPortDispatchClose\n"); - DeviceExtension = (PVIDEO_PORT_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - if ((DeviceExtension->DeviceOpened >= 1) && - (InterlockedDecrement((PLONG)&DeviceExtension->DeviceOpened) == 0)) - { - ResetDisplayParametersDeviceExtension = NULL; - InbvNotifyDisplayOwnershipLost(NULL); - ResetDisplayParametersDeviceExtension = DeviceExtension; - IntVideoPortResetDisplayParameters(80, 50); - } + IntVideoPortInbvCleanup(DeviceObject); Irp->IoStatus.Status = STATUS_SUCCESS; IoCompleteRequest(Irp, IO_NO_INCREMENT); - return STATUS_SUCCESS; } PSTR IoctlName(ULONG Ioctl) { - switch(Ioctl) + switch (Ioctl) { case IOCTL_VIDEO_ENABLE_VDM: return "IOCTL_VIDEO_ENABLE_VDM"; // CTL_CODE(FILE_DEVICE_VIDEO, 0x00, METHOD_BUFFERED, FILE_ANY_ACCESS) @@ -296,7 +529,7 @@ IoctlName(ULONG Ioctl) return "IOCTL_VIDEO_SET_DISPLAY_BRIGHTNESS"; // CTL_CODE(FILE_DEVICE_VIDEO, 0x127, METHOD_BUFFERED, FILE_ANY_ACCESS) } - return "<unknown ioctl code"; + return "<unknown ioctl code>"; } static @@ -376,7 +609,7 @@ VideoPortInitWin32kCallbacks( /* Save the callout function globally */ Win32kCallout = Win32kCallbacks->Callout; - /* Return reasonable values to win32k */ + /* Return reasonable values to Win32k */ Win32kCallbacks->bACPI = FALSE; Win32kCallbacks->pPhysDeviceObject = DeviceObject; Win32kCallbacks->DualviewFlags = 0; @@ -465,12 +698,64 @@ IntVideoPortDispatchDeviceControl( IrpStack = IoGetCurrentIrpStackLocation(Irp); + switch (IrpStack->MajorFunction) + { + case IRP_MJ_DEVICE_CONTROL: + /* This is the main part of this function and is handled below */ + break; + + case IRP_MJ_SHUTDOWN: + { + /* Dereference CSRSS */ + PKPROCESS OldCsrProcess; + OldCsrProcess = InterlockedExchangePointer((PVOID*)&CsrProcess, NULL); + if (OldCsrProcess) + ObDereferenceObject(OldCsrProcess); + + Irp->IoStatus.Status = STATUS_SUCCESS; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + return STATUS_SUCCESS; + } + + default: + ERR_(VIDEOPRT, "- Unknown MajorFunction 0x%x\n", IrpStack->MajorFunction); + Irp->IoStatus.Status = STATUS_SUCCESS; + IoCompleteRequest(Irp, IO_NO_INCREMENT); + return STATUS_SUCCESS; + } + IoControlCode = IrpStack->Parameters.DeviceIoControl.IoControlCode; - INFO_(VIDEOPRT, "- IoControlCode: %x: %s\n", IoControlCode, IoctlName(IoControlCode)); + INFO_(VIDEOPRT, "- IoControlCode: 0x%x: %s\n", IoControlCode, IoctlName(IoControlCode)); - switch(IoControlCode) + switch (IoControlCode) { + case IOCTL_VIDEO_ENABLE_VDM: + case IOCTL_VIDEO_DISABLE_VDM: + case IOCTL_VIDEO_REGISTER_VDM: + WARN_(VIDEOPRT, "- IOCTL_VIDEO_*_VDM are UNIMPLEMENTED!\n"); + Status = STATUS_NOT_IMPLEMENTED; + break; + + case IOCTL_VIDEO_SET_OUTPUT_DEVICE_POWER_STATE: + case IOCTL_VIDEO_GET_OUTPUT_DEVICE_POWER_STATE: + WARN_(VIDEOPRT, "- IOCTL_VIDEO_GET/SET_OUTPUT_DEVICE_POWER_STATE are UNIMPLEMENTED!\n"); + Status = STATUS_NOT_IMPLEMENTED; + break; + + case IOCTL_VIDEO_SET_POWER_MANAGEMENT: + case IOCTL_VIDEO_GET_POWER_MANAGEMENT: + WARN_(VIDEOPRT, "- IOCTL_VIDEO_GET/SET_POWER_MANAGEMENT are UNIMPLEMENTED!\n"); + Status = STATUS_NOT_IMPLEMENTED; + break; + + case IOCTL_VIDEO_QUERY_SUPPORTED_BRIGHTNESS: + case IOCTL_VIDEO_QUERY_DISPLAY_BRIGHTNESS: + case IOCTL_VIDEO_SET_DISPLAY_BRIGHTNESS: + WARN_(VIDEOPRT, "- IOCTL_VIDEO_*_BRIGHTNESS are UNIMPLEMENTED!\n"); + Status = STATUS_NOT_IMPLEMENTED; + break; + case IOCTL_VIDEO_INIT_WIN32K_CALLBACKS: INFO_(VIDEOPRT, "- IOCTL_VIDEO_INIT_WIN32K_CALLBACKS\n"); Status = VideoPortInitWin32kCallbacks(DeviceObject, @@ -479,6 +764,11 @@ IntVideoPortDispatchDeviceControl( &Irp->IoStatus.Information); break; + case IOCTL_VIDEO_IS_VGA_DEVICE: + WARN_(VIDEOPRT, "- IOCTL_VIDEO_IS_VGA_DEVICE is UNIMPLEMENTED!\n"); + Status = STATUS_NOT_IMPLEMENTED; + break; + case IOCTL_VIDEO_USE_DEVICE_IN_SESSION: INFO_(VIDEOPRT, "- IOCTL_VIDEO_USE_DEVICE_IN_SESSION\n"); Status = VideoPortUseDeviceInSession(DeviceObject, @@ -487,71 +777,35 @@ IntVideoPortDispatchDeviceControl( &Irp->IoStatus.Information); break; + case IOCTL_VIDEO_PREPARE_FOR_EARECOVERY: + INFO_(VIDEOPRT, "- IOCTL_VIDEO_PREPARE_FOR_EARECOVERY\n"); + /* + * The Win32k Watchdog Timer detected that a thread spent more time + * in a display driver than the allotted time its threshold specified, + * and thus is going to attempt to recover by switching to VGA mode. + * If this attempt fails, the watchdog generates bugcheck 0xEA + * "THREAD_STUCK_IN_DEVICE_DRIVER". + * + * Prepare the recovery by resetting the display adapters to + * standard VGA 80x25 text mode. + */ + IntVideoPortResetDisplayParametersEx(80, 25, FALSE); + Status = STATUS_SUCCESS; + break; + default: /* Forward to the Miniport Driver */ Status = VideoPortForwardDeviceControl(DeviceObject, Irp); break; } - INFO_(VIDEOPRT, "- Returned status: %x\n", Irp->IoStatus.Status); + INFO_(VIDEOPRT, "- Returned status: 0x%x\n", Status); Irp->IoStatus.Status = Status; IoCompleteRequest(Irp, IO_NO_INCREMENT); - return Status; } -/* - * IntVideoPortWrite - * - * This is a bit of a hack. We want to take ownership of the display as late - * as possible, just before the switch to graphics mode. Win32k knows when - * this happens, we don't. So we need Win32k to inform us. This could be done - * using an IOCTL, but there's no way of knowing which IOCTL codes are unused - * in the communication between GDI driver and miniport driver. So we use - * IRP_MJ_WRITE as the signal that win32k is ready to switch to graphics mode, - * since we know for certain that there is no read/write activity going on - * between GDI and miniport drivers. - * We don't actually need the data that is passed, we just trigger on the fact - * that an IRP_MJ_WRITE was sent. - * - * Run Level - * PASSIVE_LEVEL - */ -NTSTATUS -NTAPI -IntVideoPortDispatchWrite( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp) -{ - PIO_STACK_LOCATION piosStack = IoGetCurrentIrpStackLocation(Irp); - PVIDEO_PORT_DEVICE_EXTENSION DeviceExtension; - NTSTATUS nErrCode; - - DeviceExtension = (PVIDEO_PORT_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - /* - * Storing the device extension pointer in a static variable is an - * ugly hack. Unfortunately, we need it in IntVideoPortResetDisplayParameters - * and InbvNotifyDisplayOwnershipLost doesn't allow us to pass a userdata - * parameter. On the bright side, the DISPLAY device is opened - * exclusively, so there can be only one device extension active at - * any point in time. - * - * FIXME: We should process all opened display devices in - * IntVideoPortResetDisplayParameters. - */ - ResetDisplayParametersDeviceExtension = DeviceExtension; - InbvNotifyDisplayOwnershipLost(IntVideoPortResetDisplayParameters); - - nErrCode = STATUS_SUCCESS; - Irp->IoStatus.Information = piosStack->Parameters.Write.Length; - Irp->IoStatus.Status = nErrCode; - IoCompleteRequest(Irp, IO_NO_INCREMENT); - - return nErrCode; -} - NTSTATUS NTAPI IntVideoPortPnPStartDevice( diff --git a/win32ss/drivers/videoprt/int10.c b/win32ss/drivers/videoprt/int10.c index f83ed0f3dd2..caa50ddfc35 100644 --- a/win32ss/drivers/videoprt/int10.c +++ b/win32ss/drivers/videoprt/int10.c @@ -393,7 +393,7 @@ VideoPortInt10( INT10_BIOS_ARGUMENTS Int10BiosArguments; VP_STATUS Status; - if (!CsrssInitialized) + if (!CsrProcess) { return ERROR_INVALID_PARAMETER; } diff --git a/win32ss/drivers/videoprt/videoprt.c b/win32ss/drivers/videoprt/videoprt.c index 812f55ad6af..6e75d563ea9 100644 --- a/win32ss/drivers/videoprt/videoprt.c +++ b/win32ss/drivers/videoprt/videoprt.c @@ -30,10 +30,10 @@ /* GLOBAL VARIABLES ***********************************************************/ -ULONG CsrssInitialized = FALSE; -PKPROCESS Csrss = NULL; +PKPROCESS CsrProcess = NULL; ULONG VideoPortDeviceNumber = 0; KMUTEX VideoPortInt10Mutex; +RTL_STATIC_LIST_HEAD(HwResetAdaptersList); /* PRIVATE FUNCTIONS **********************************************************/ @@ -406,6 +406,14 @@ IntVideoPortFindAdapter( goto Failure; } + /* If the device can be reset, insert it in the list of resettable adapters */ + InitializeListHead(&DeviceExtension->HwResetListEntry); + if (DriverExtension->InitializationData.HwResetHw != NULL) + { + InsertTailList(&HwResetAdaptersList, + &DeviceExtension->HwResetListEntry); + } + /* Query children of the device. */ VideoPortEnumerateChildren(&DeviceExtension->MiniPortDeviceExtension, NULL); @@ -427,9 +435,9 @@ IntAttachToCSRSS( PKAPC_STATE ApcState) { *CallingProcess = (PKPROCESS)PsGetCurrentProcess(); - if (*CallingProcess != Csrss) + if (*CallingProcess != CsrProcess) { - KeStackAttachProcess(Csrss, ApcState); + KeStackAttachProcess(CsrProcess, ApcState); } } @@ -439,7 +447,7 @@ IntDetachFromCSRSS( PKPROCESS *CallingProcess, PKAPC_STATE ApcState) { - if (*CallingProcess != Csrss) + if (*CallingProcess != CsrProcess) { KeUnstackDetachProcess(ApcState); } @@ -516,10 +524,8 @@ VideoPortInitialize( DriverObject->MajorFunction[IRP_MJ_CLOSE] = IntVideoPortDispatchClose; DriverObject->MajorFunction[IRP_MJ_DEVICE_CONTROL] = IntVideoPortDispatchDeviceControl; - DriverObject->MajorFunction[IRP_MJ_INTERNAL_DEVICE_CONTROL] = + DriverObject->MajorFunction[IRP_MJ_SHUTDOWN] = IntVideoPortDispatchDeviceControl; - DriverObject->MajorFunction[IRP_MJ_WRITE] = - IntVideoPortDispatchWrite; // ReactOS-specific hack DriverObject->DriverUnload = IntVideoPortUnload; /* Determine type of the miniport driver */ @@ -608,8 +614,8 @@ VideoPortInitialize( DriverExtension->HwContext = HwContext; /* - * Plug & Play drivers registers the device in AddDevice routine. For - * legacy drivers we must do it now. + * Plug & Play drivers registers the device in AddDevice routine. + * For legacy drivers we must do it now. */ if (LegacyDetection) { @@ -617,7 +623,7 @@ VideoPortInitialize( if (HwInitializationData->HwInitDataSize != SIZE_OF_NT4_VIDEO_HW_INITIALIZATION_DATA) { - /* power management */ + /* Power management */ DriverObject->MajorFunction[IRP_MJ_POWER] = IntVideoPortDispatchPower; } diff --git a/win32ss/drivers/videoprt/videoprt.h b/win32ss/drivers/videoprt/videoprt.h index 3490e5625ba..94289dc031a 100644 --- a/win32ss/drivers/videoprt/videoprt.h +++ b/win32ss/drivers/videoprt/videoprt.h @@ -32,11 +32,14 @@ #include <windef.h> #include <wdmguid.h> -#define TAG_VIDEO_PORT 'PDIV' -#define TAG_VIDEO_PORT_BUFFER '\0mpV' -#define TAG_REQUEST_PACKET 'qRpV' +/* PSEH for SEH Support */ +#include <pseh/pseh2.h> -#define GUID_STRING_LENGTH 38 * sizeof(WCHAR) +#define TAG_VIDEO_PORT 'PDIV' +#define TAG_VIDEO_PORT_BUFFER '\0mpV' +#define TAG_REQUEST_PACKET 'qRpV' + +#define GUID_STRING_LENGTH (38 * sizeof(WCHAR)) typedef struct _VIDEO_PORT_ADDRESS_MAPPING { @@ -102,6 +105,7 @@ typedef struct _VIDEO_PORT_DEVICE_EXTENSTION AGP_BUS_INTERFACE_STANDARD AgpInterface; KMUTEX DeviceLock; LIST_ENTRY DmaAdapterList, ChildDeviceList; + LIST_ENTRY HwResetListEntry; ULONG SessionId; CHAR MiniPortDeviceExtension[1]; } VIDEO_PORT_DEVICE_EXTENSION, *PVIDEO_PORT_DEVICE_EXTENSION; @@ -204,11 +208,6 @@ IntVideoPortDispatchSystemControl( IN PDEVICE_OBJECT DeviceObject, IN PIRP Irp); -NTSTATUS NTAPI -IntVideoPortDispatchWrite( - IN PDEVICE_OBJECT DeviceObject, - IN PIRP Irp); - VOID NTAPI IntVideoPortUnload(PDRIVER_OBJECT DriverObject); @@ -244,10 +243,10 @@ IntVideoPortMapPhysicalMemory( /* videoprt.c */ -extern ULONG CsrssInitialized; -extern PKPROCESS Csrss; +extern PKPROCESS CsrProcess; extern ULONG VideoPortDeviceNumber; extern KMUTEX VideoPortInt10Mutex; +extern LIST_ENTRY HwResetAdaptersList; VOID FASTCALL IntAttachToCSRSS(PKPROCESS *CallingProcess, PKAPC_STATE ApcState); diff --git a/win32ss/gdi/eng/device.c b/win32ss/gdi/eng/device.c index 7691aad92f5..21f63f86a7c 100644 --- a/win32ss/gdi/eng/device.c +++ b/win32ss/gdi/eng/device.c @@ -8,7 +8,9 @@ */ #include <win32k.h> -DBG_DEFAULT_CHANNEL(EngDev) +#include <ntddvdeo.h> + +DBG_DEFAULT_CHANNEL(EngDev); PGRAPHICS_DEVICE gpPrimaryGraphicsDevice; PGRAPHICS_DEVICE gpVgaGraphicsDevice; @@ -156,6 +158,70 @@ EngpPopulateDeviceModeList( return TRUE; } +extern VOID +UserRefreshDisplay(IN PPDEVOBJ ppdev); + +// PVIDEO_WIN32K_CALLOUT +VOID +NTAPI +VideoPortCallout( + _In_ PVOID Params) +{ +/* + * IMPORTANT NOTICE!! On Windows XP/2003 this function triggers the creation of + * a specific VideoPortCalloutThread() system thread using the same mechanism + * as the RIT/desktop/Ghost system threads. + */ + + PVIDEO_WIN32K_CALLBACKS_PARAMS CallbackParams = (PVIDEO_WIN32K_CALLBACKS_PARAMS)Params; + + TRACE("VideoPortCallout(0x%p, 0x%x)\n", + CallbackParams, CallbackParams ? CallbackParams->CalloutType : -1); + + if (!CallbackParams) + return; + + switch (CallbackParams->CalloutType) + { + case VideoFindAdapterCallout: + { + TRACE("VideoPortCallout: VideoFindAdapterCallout called - Param = %s\n", + CallbackParams->Param ? "TRUE" : "FALSE"); + if (CallbackParams->Param == TRUE) + { + /* Re-enable the display */ + UserRefreshDisplay(gppdevPrimary); + } + else + { + /* Disable the display */ + NOTHING; // Nothing to do for the moment... + } + + CallbackParams->Status = STATUS_SUCCESS; + break; + } + + case VideoPowerNotifyCallout: + case VideoDisplaySwitchCallout: + case VideoEnumChildPdoNotifyCallout: + case VideoWakeupCallout: + case VideoChangeDisplaySettingsCallout: + case VideoPnpNotifyCallout: + case VideoDxgkDisplaySwitchCallout: + case VideoDxgkMonitorEventCallout: + case VideoDxgkFindAdapterTdrCallout: + ERR("VideoPortCallout: CalloutType 0x%x is UNIMPLEMENTED!\n", CallbackParams->CalloutType); + CallbackParams->Status = STATUS_NOT_IMPLEMENTED; + break; + + default: + ERR("VideoPortCallout: Unknown CalloutType 0x%x\n", CallbackParams->CalloutType); + CallbackParams->Status = STATUS_UNSUCCESSFUL; + break; + } +} + PGRAPHICS_DEVICE NTAPI EngpRegisterGraphicsDevice( @@ -168,10 +234,10 @@ EngpRegisterGraphicsDevice( PDEVICE_OBJECT pDeviceObject; PFILE_OBJECT pFileObject; NTSTATUS Status; + VIDEO_WIN32K_CALLBACKS Win32kCallbacks; + ULONG ulReturn; PWSTR pwsz; ULONG cj; - SIZE_T cjWritten; - BOOL bEnable = TRUE; TRACE("EngpRegisterGraphicsDevice(%wZ)\n", pustrDeviceName); @@ -197,13 +263,34 @@ EngpRegisterGraphicsDevice( return NULL; } - /* Enable the device */ - EngFileWrite(pFileObject, &bEnable, sizeof(BOOL), &cjWritten); - /* Copy the device and file object pointers */ pGraphicsDevice->DeviceObject = pDeviceObject; pGraphicsDevice->FileObject = pFileObject; + /* Initialize and register the device with videoprt for Win32k callbacks */ + Win32kCallbacks.PhysDisp = pGraphicsDevice; + Win32kCallbacks.Callout = VideoPortCallout; + // Reset the data being returned prior to the call. + Win32kCallbacks.bACPI = FALSE; + Win32kCallbacks.pPhysDeviceObject = NULL; + Win32kCallbacks.DualviewFlags = 0; + Status = (NTSTATUS)EngDeviceIoControl((HANDLE)pDeviceObject, + IOCTL_VIDEO_INIT_WIN32K_CALLBACKS, + &Win32kCallbacks, + sizeof(Win32kCallbacks), + &Win32kCallbacks, + sizeof(Win32kCallbacks), + &ulReturn); + if (Status != ERROR_SUCCESS) + { + ERR("EngDeviceIoControl(0x%p, IOCTL_VIDEO_INIT_WIN32K_CALLBACKS) failed, Status 0x%lx\n", + pDeviceObject, Status); + } + // TODO: Set flags according to the results. + // if (Win32kCallbacks.bACPI) + // if (Win32kCallbacks.DualviewFlags & ???) + // Win32kCallbacks.pPhysDeviceObject; + /* Copy the device name */ RtlStringCbCopyNW(pGraphicsDevice->szNtDeviceName, sizeof(pGraphicsDevice->szNtDeviceName), diff --git a/win32ss/user/ntuser/display.c b/win32ss/user/ntuser/display.c index cebaf8c1148..be0335c52da 100644 --- a/win32ss/user/ntuser/display.c +++ b/win32ss/user/ntuser/display.c @@ -285,6 +285,45 @@ InitVideo(VOID) return STATUS_SUCCESS; } +VOID +UserRefreshDisplay(IN PPDEVOBJ ppdev) +{ + ULONG_PTR ulResult; + // PVOID pvOldCursor; + + // TODO: Re-enable the cursor reset code once this function becomes called + // from within a Win32 thread... Indeed UserSetCursor() requires this, but + // at the moment this function is directly called from a separate thread + // from within videoprt, instead of by a separate win32k system thread. + + if (!ppdev) + return; + + PDEVOBJ_vReference(ppdev); + + /* Remove mouse pointer */ + // pvOldCursor = UserSetCursor(NULL, TRUE); + + /* Do the mode switch -- Use the actual same current mode */ + ulResult = PDEVOBJ_bSwitchMode(ppdev, ppdev->pdmwDev); + ASSERT(ulResult); + + /* Restore mouse pointer, no hooks called */ + // pvOldCursor = UserSetCursor(pvOldCursor, TRUE); + // ASSERT(pvOldCursor == NULL); + + /* Update the system metrics */ + InitMetrics(); + + /* Set new size of the monitor */ + // UserUpdateMonitorSize((HDEV)ppdev); + + //co_IntShowDesktop(pdesk, ppdev->gdiinfo.ulHorzRes, ppdev->gdiinfo.ulVertRes); + UserRedrawDesktop(); + + PDEVOBJ_vRelease(ppdev); +} + NTSTATUS NTAPI UserEnumDisplayDevices(

1 0

[reactos] 01/01: [USERENV] Improve Japanese translation again
by Katayama Hirofumi MZ 01 Dec '19

01 Dec '19

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c076d5819cfacb22a1156… commit c076d5819cfacb22a115664a2eca935dd410eef7 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Mon Dec 2 07:05:57 2019 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Mon Dec 2 07:05:57 2019 +0900 [USERENV] Improve Japanese translation again --- dll/win32/userenv/lang/ja-JP.rc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dll/win32/userenv/lang/ja-JP.rc b/dll/win32/userenv/lang/ja-JP.rc index c2b5f2120d7..df17904974a 100644 --- a/dll/win32/userenv/lang/ja-JP.rc +++ b/dll/win32/userenv/lang/ja-JP.rc @@ -20,5 +20,5 @@ BEGIN IDS_TEMP "Local Settings\\Temp" IDS_CACHE "Local Settings\\Temporary Internet Files" IDS_HISTORY "Local Settings\\History" - IDS_COOKIES "Cookies" + IDS_COOKIES "クッキー" END

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