Ros-dev

ros-dev@reactos.org

5788 discussions

Re: [ros-dev] [ros-diffs] 01/01: [SERVICES] Fix timeout when a service is stopped

by Pierre Schweitzer

Hey Eric, What's the purpose of that "Sleep(2000);"? Is it a debug leftover? Cheers, Pierre Le 05/10/2018 à 08:43, Eric Kohl a écrit : > https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f6d81f225747e4b1b3716… > > commit f6d81f225747e4b1b3716e9ca8629f77dcc52b1a > Author: Eric Kohl <eric.kohl(a)reactos.org> > AuthorDate: Fri Oct 5 08:41:23 2018 +0200 > Commit: Eric Kohl <eric.kohl(a)reactos.org> > CommitDate: Fri Oct 5 08:41:23 2018 +0200 > > [SERVICES] Fix timeout when a service is stopped > > RSetServiceStatus: Send the stop command for the dispatcher thread from a separate thread. > > Fixes CORE-15064 > --- > base/system/services/rpcserver.c | 59 ++++++++++++++++++++++++++++++++++------ > 1 file changed, 50 insertions(+), 9 deletions(-) > > diff --git a/base/system/services/rpcserver.c b/base/system/services/rpcserver.c > index 7f251c8760..8e383a5bdc 100644 > --- a/base/system/services/rpcserver.c > +++ b/base/system/services/rpcserver.c > @@ -1671,6 +1671,43 @@ ScmIsValidServiceState(DWORD dwCurrentState) > } > > > +static > +DWORD > +WINAPI > +ScmStopThread( > + _In_ PVOID pParam) > +{ > + PSERVICE pService; > + > + DPRINT("ScmStopThread(%p)\n", pParam); > + > + pService = (PSERVICE)pParam; > + > + if (pService->lpImage->dwImageRunCount != 0) > + return 0; > + > + Sleep(2000); > + > + /* Lock the service database exclusively */ > + ScmLockDatabaseExclusive(); > + > + /* Stop the dispatcher thread */ > + ScmControlService(pService->lpImage->hControlPipe, > + L"", > + (SERVICE_STATUS_HANDLE)pService, > + SERVICE_CONTROL_STOP); > + > + /* Remove the service image */ > + ScmRemoveServiceImage(pService->lpImage); > + > + /* Unlock the service database */ > + ScmUnlockDatabase(); > + > + DPRINT("ScmStopThread done!\n"); > + return 0; > +} > + > + > /* Function 7 */ > DWORD > WINAPI > @@ -1683,6 +1720,8 @@ RSetServiceStatus( > DWORD dwPreviousType; > LPCWSTR lpLogStrings[2]; > WCHAR szLogBuffer[80]; > + HANDLE hStopThread = NULL; > + DWORD dwStopThreadId; > UINT uID; > > DPRINT("RSetServiceStatus() called\n"); > @@ -1762,15 +1801,17 @@ RSetServiceStatus( > /* If we just stopped the last running service... */ > if (lpService->lpImage->dwImageRunCount == 0) > { > - /* Stop the dispatcher thread */ > - ScmControlService(lpService->lpImage->hControlPipe, > - L"", > - (SERVICE_STATUS_HANDLE)lpService, > - SERVICE_CONTROL_STOP); > - > - /* Remove the service image */ > - ScmRemoveServiceImage(lpService->lpImage); > - lpService->lpImage = NULL; > + /* Run the stop thread to stop the service dispatcher */ > + hStopThread = CreateThread(NULL, > + 0, > + (LPTHREAD_START_ROUTINE)ScmStopThread, > + (LPVOID)lpService, > + 0, > + &dwStopThreadId); > + if (hStopThread != NULL) > + { > + CloseHandle(hStopThread); > + } > } > } > > -- Pierre Schweitzer <pierre at reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

6 years, 2 months

WidenPath

by katahiromz

About NTGDI Should we use PATHOBJ for WidenPath and FlattenPath?

6 years, 2 months

Unused bandwidth at home

by Javier Agustìn Fernàndez Arroyo

Hello all, I have just set optical fiber up at my home. The thing is, im completely sure it will be infra-used Is there anything in ROS i might do for help with that?

6 years, 3 months

Re: [ros-dev] [ros-diffs] 01/01: [ADVAPI32][SERVICES] Add (dummy) password encryption/decryption functions to CreateServiceA/W and ChangeServiceConfigA/W in order to prepare to pass encrypted passwords to the service manager

by Pierre Schweitzer

Hi Eric, For security reason, when freeing your password buffers (see below), you shouldn't be using ZeroMemory() before free, but SecureZeroMemory() to zero the buffer. The first one can be optimized (and thus removed) by the compiler. The second cannot. Cheers, Pierre Le 17/09/2018 à 16:36, Eric Kohl a écrit : > https://git.reactos.org/?p=reactos.git;a=commitdiff;h=5e2c4657ca10dea1154cb… > > commit 5e2c4657ca10dea1154cb43f16ee6962999ac7a4 > Author: Eric Kohl <eric.kohl(a)reactos.org> > AuthorDate: Mon Sep 17 16:34:48 2018 +0200 > Commit: Eric Kohl <eric.kohl(a)reactos.org> > CommitDate: Mon Sep 17 16:34:48 2018 +0200 > > [ADVAPI32][SERVICES] Add (dummy) password encryption/decryption functions to CreateServiceA/W and ChangeServiceConfigA/W in order to prepare to pass encrypted passwords to the service manager > --- > base/system/services/config.c | 24 ++++++++ > base/system/services/rpcserver.c | 49 ++++++++++++++-- > base/system/services/services.h | 7 +++ > dll/win32/advapi32/service/scm.c | 123 ++++++++++++++++++++++++++++++++------- > 4 files changed, 177 insertions(+), 26 deletions(-) > > diff --git a/base/system/services/rpcserver.c b/base/system/services/rpcserver.c > index aa64233350..454181bb66 100644 > --- a/base/system/services/rpcserver.c > +++ b/base/system/services/rpcserver.c > @@ -2216,12 +2232,23 @@ RChangeServiceConfigW( > dwError = ERROR_SUCCESS; > > if (dwError != ERROR_SUCCESS) > + { > + DPRINT1("ScmSetServicePassword failed (Error %lu)\n", dwError); > goto done; > + } > } > } > } > > done: > + if (lpClearTextPassword != NULL) > + { > + /* Wipe and release the password buffer */ > + ZeroMemory(lpClearTextPassword, > + (wcslen(lpClearTextPassword) + 1) * sizeof(WCHAR)); > + HeapFree(GetProcessHeap(), 0, lpClearTextPassword); > + } > + > if (hServiceKey != NULL) > RegCloseKey(hServiceKey); > > @@ -2612,6 +2645,14 @@ done: > if (hServiceKey != NULL) > RegCloseKey(hServiceKey); > > + if (lpClearTextPassword != NULL) > + { > + /* Wipe and release the password buffer */ > + ZeroMemory(lpClearTextPassword, > + (wcslen(lpClearTextPassword) + 1) * sizeof(WCHAR)); > + HeapFree(GetProcessHeap(), 0, lpClearTextPassword); > + } > + > if (dwError == ERROR_SUCCESS) > { > DPRINT("hService %p\n", hServiceHandle); -- Pierre Schweitzer <pierre at reactos.org> System & Network Administrator Senior Kernel Developer ReactOS Deutschland e.V.

6 years, 3 months

Re: [ros-dev] Win8 theme for ReactOS?

by theUser BL

> Isn't it fun that ppl simply don't understand a "NO" I give 'em in the > forums?! Thats only a comment of ONE developer. If your medical doctor syas you anything, do you never asks a second doctor again for the same thing, to see if he was right? > I even explained why and that GreenteaOS is not more than > stolen ROS code (*) The only problem you have had is, that he have removed the name "ReactOS" everywhere. But it is still al under the same license like ReactOS. > filled with questionable code sources like the > betaarchive forums. So "NO" means go on the other dev's nerves for a > ugly theme violating our and MS's copyrights? Why is an Win8-theme violating MS-copyright but - changing the color of WineMine to Microsofts WinMine - copying the XP explorer.exe 1:1, only the images on it are different - having a Look And Feel VERY similar to Win2000 - etc is all ok? Greetings theuserbl

6 years, 3 months

Win8 theme for ReactOS?

by theUser BL

A while ago I have asked in the forum, if the modified Lautus-theme of GreenteaOS could beintegrated in ReactOS https://www.reactos.org/forum/viewtopic.php?f=2&t=16788 Sadly the theme is called in GreenteaOS Lautus, too. Here the source of it https://github.com/GreenteaOS/Deprecated/tree/master/third-party/reactos/me… But it looks like Windows 8. Here a screenshot of the Greentea Windows 8 like theme on ReactOS. Choose one link. I have uploaded it multiple times on multiple places, because some picture-uploading-sides delete its files from time to time https://image.ibb.co/gTsPkU/Virtual_Box_React_OS_0_4_9_08_09_2018_16_22_18.… http://img4host.net/upload/081642005b93dfb8c233a.png https://img3.picload.org/image/dlpgdlow/virtualbox_reactos0.4.9_08_09_.png https://cdn3.imggmi.com/uploads/2018/9/8/e016c5a6842640ac5f7436ee2e9b04e7-f… https://i.imgur.com/mCUvsLy.png EmuandCo is against the integration of the Win8-theme in ReactOS. But he is the only ReactOS-developer in the ReactOS-forum. So I now also ask here, what the other developers thinking about it. Greetings theuserbl

6 years, 3 months

Contribution for Tracing APIs

by Biswapriyo Nath

I've some working code for event tracing APIs like StartTrace, OpenTrace, ControlTrace etc. Those works in Windows 10 but are not present in ReactOS Advapi.dll. How can I add those APIs?

6 years, 3 months

Virtual consoles using ntvdm

by Javier Agustìn Fernàndez Arroyo

A crazy idea .. What about implementing some kind of virtual consoles, just like Linux? The point is, using ntvdm subsystem, we could allow only console (DOS) apps to run in there. And some kind of "you are running a console. Switch to desktop to run this app!" Message is anyone tried to run another stuff What do you think? A side effect could be, allow remote connections, etc ... :)

6 years, 3 months

Valve support for ReactOS?

by Javier Agustìn Fernàndez Arroyo

i was thinking about Valve's Linux support (Steam play) and i was wondering about asking them to support ReactOS via development... Since Windows XP/2k3 are being abandoned, maybe ReactOS could still run Steam games... if they support it. what do you think?

6 years, 3 months

Broken booting since commit ae11022712db7a5a1e818e33a5f4052eac7e8e3c

by David Quintana (gigaherz)

I believe from the testbot log, that my commit made some DLLs grow out of their assigned memory allocation, but I don't know how to update the dll address mappings. Help appreciated.

6 years, 3 months

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-dev