Ros-dev

ros-dev@reactos.org

5788 discussions

Re: [ros-dev] [ros-diffs] [hbelusca] 57504: [NTOSKRNL] Rearrange the NtQuerySystemEnvironmentValue code to have successive logical checks.

by Timo Kreuzer

This changes the logic. I'm not sure how exact the previous version was, but these changes should be tested. Am 06.10.2012 21:50, schrieb hbelusca(a)svn.reactos.org: > Author: hbelusca > Date: Sat Oct 6 19:50:17 2012 > New Revision: 57504 > > URL: http://svn.reactos.org/svn/reactos?rev=57504&view=rev > Log: > [NTOSKRNL] > Rearrange the NtQuerySystemEnvironmentValue code to have successive logical checks. > > Modified: > trunk/reactos/ntoskrnl/ex/sysinfo.c > > Modified: trunk/reactos/ntoskrnl/ex/sysinfo.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/sysinfo.c?rev=… > ============================================================================== > --- trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/ex/sysinfo.c [iso-8859-1] Sat Oct 6 19:50:17 2012 > @@ -235,44 +235,40 @@ > _SEH2_YIELD(return _SEH2_GetExceptionCode()); > } > _SEH2_END; > - > - } > - > - /* Allocate a buffer for the value */ > - AnsiValueBuffer = ExAllocatePoolWithTag(NonPagedPool, ValueBufferLength, 'pmeT'); > - if (AnsiValueBuffer == NULL) > - { > - return STATUS_INSUFFICIENT_RESOURCES; > - } > - > - /* > - * Copy the name to kernel space if necessary and convert it to ANSI. > - */ > + } > + > + /* According to NTInternals the SeSystemEnvironmentName privilege is required! */ > + if (!SeSinglePrivilegeCheck(SeSystemEnvironmentPrivilege, PreviousMode)) > + { > + DPRINT1("NtQuerySystemEnvironmentValue: Caller requires the SeSystemEnvironmentPrivilege privilege!\n"); > + return STATUS_PRIVILEGE_NOT_HELD; > + } > + > + /* Copy the name to kernel space if necessary */ > Status = ProbeAndCaptureUnicodeString(&WName, PreviousMode, VariableName); > if (!NT_SUCCESS(Status)) > { > return Status; > } > > - /* > - * according to ntinternals the SeSystemEnvironmentName privilege is required! > - */ > - if (!SeSinglePrivilegeCheck(SeSystemEnvironmentPrivilege, PreviousMode)) > - { > - ReleaseCapturedUnicodeString(&WName, PreviousMode); > - DPRINT1("NtQuerySystemEnvironmentValue: Caller requires the SeSystemEnvironmentPrivilege privilege!\n"); > - return STATUS_PRIVILEGE_NOT_HELD; > - } > - > - /* Convert the value name to ansi and release the captured unicode string */ > + /* Convert the name to ANSI and release the captured UNICODE string */ > Status = RtlUnicodeStringToAnsiString(&AName, &WName, TRUE); > ReleaseCapturedUnicodeString(&WName, PreviousMode); > if (!NT_SUCCESS(Status)) return Status; > > - /* Get the environment variable */ > + /* Allocate a buffer for the ANSI environment variable */ > + AnsiValueBuffer = ExAllocatePoolWithTag(NonPagedPool, ValueBufferLength, 'pmeT'); > + if (AnsiValueBuffer == NULL) > + { > + RtlFreeAnsiString(&AName); > + return STATUS_INSUFFICIENT_RESOURCES; > + } > + > + /* Get the environment variable and free the ANSI name */ > Result = HalGetEnvironmentVariable(AName.Buffer, > (USHORT)ValueBufferLength, > AnsiValueBuffer); > + RtlFreeAnsiString(&AName); > > /* Check if we had success */ > if (Result == ESUCCESS) > @@ -280,13 +276,13 @@ > /* Copy the result back to the caller. */ > _SEH2_TRY > { > - /* Initialize ansi string from the result */ > + /* Initialize ANSI string from the result */ > RtlInitAnsiString(&AValue, AnsiValueBuffer); > > - /* Initialize a unicode string from the callers buffer */ > + /* Initialize a UNICODE string from the callers buffer */ > RtlInitEmptyUnicodeString(&WValue, ValueBuffer, ValueBufferLength); > > - /* Convert the result to unicode */ > + /* Convert the result to UNICODE */ > Status = RtlAnsiStringToUnicodeString(&WValue, &AValue, FALSE); > > if (ReturnLength != NULL) > @@ -305,8 +301,7 @@ > Status = STATUS_UNSUCCESSFUL; > } > > - /* Cleanup allocated resources. */ > - RtlFreeAnsiString(&AName); > + /* Free the allocated ANSI value buffer */ > ExFreePoolWithTag(AnsiValueBuffer, 'pmeT'); > > return Status; > > >

11 years, 11 months

Re: [ros-dev] [ros-diffs] [tfaber] 57485: [SHELL32] - Fix what might finally be the last missing "this->" instances left over from the C++ conversion (used a regex this time)

by Timo Kreuzer

Shouldn't member variables rather be prefixed with m_? Am 05.10.2012 01:46, schrieb tfaber(a)svn.reactos.org: > Author: tfaber > Date: Thu Oct 4 23:46:59 2012 > New Revision: 57485 > > URL: http://svn.reactos.org/svn/reactos?rev=57485&view=rev > Log: > [SHELL32] > - Fix what might finally be the last missing "this->" instances left over from the C++ conversion (used a regex this time) > > Modified: > trunk/reactos/dll/win32/shell32/folders/cpanel.cpp > trunk/reactos/dll/win32/shell32/folders/mycomp.cpp > trunk/reactos/dll/win32/shell32/shlview.cpp > > Modified: trunk/reactos/dll/win32/shell32/folders/cpanel.cpp > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/folders/… > ============================================================================== > --- trunk/reactos/dll/win32/shell32/folders/cpanel.cpp [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/shell32/folders/cpanel.cpp [iso-8859-1] Thu Oct 4 23:46:59 2012 > @@ -540,7 +540,7 @@ > // > pObj = (IContextMenu *)this; > this->apidl = apidl; > - cidl = cidl; > + this->cidl = cidl; > pObj->AddRef(); > hr = S_OK; > } else if (IsEqualIID(riid, IID_IDataObject) && (cidl >= 1)) { > > Modified: trunk/reactos/dll/win32/shell32/folders/mycomp.cpp > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/folders/… > ============================================================================== > --- trunk/reactos/dll/win32/shell32/folders/mycomp.cpp [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/shell32/folders/mycomp.cpp [iso-8859-1] Thu Oct 4 23:46:59 2012 > @@ -728,7 +728,7 @@ > > wcscpy(sName, lpName); > SHFree(sName); > - sName = sName; > + this->sName = sName; > TRACE("result %s\n", debugstr_w(sName)); > return S_OK; > } > > Modified: trunk/reactos/dll/win32/shell32/shlview.cpp > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/shlview.… > ============================================================================== > --- trunk/reactos/dll/win32/shell32/shlview.cpp [iso-8859-1] (original) > +++ trunk/reactos/dll/win32/shell32/shlview.cpp [iso-8859-1] Thu Oct 4 23:46:59 2012 > @@ -2828,7 +2828,7 @@ > FIXME("partial stub: %p %08x %08x %p\n", this, aspects, advf, pAdvSink); > > /* FIXME: we set the AdviseSink, but never use it to send any advice */ > - pAdvSink = pAdvSink; > + this->pAdvSink = pAdvSink; > dwAspects = aspects; > dwAdvf = advf; > > > >

11 years, 11 months

Re: [ros-dev] [ros-diffs] [hbelusca] 57462: [REGEDIT] - WCHAR ==> TCHAR in (before a REALLY conversion of regedit into UNICODE) - Correct some mistakes in displayed strings - Improve informative / warning / error messages w...

by Timo Kreuzer

Please don't use TCHAR. Make everything WCHAR and use widechar functions explicitly. Am 03.10.2012 01:29, schrieb hbelusca(a)svn.reactos.org: > Author: hbelusca > Date: Tue Oct 2 23:29:22 2012 > New Revision: 57462 > > URL: http://svn.reactos.org/svn/reactos?rev=57462&view=rev > Log: > [REGEDIT] > - WCHAR ==> TCHAR in (before a REALLY conversion of regedit into UNICODE) > - Correct some mistakes in displayed strings > - Improve informative / warning / error messages when importing registry files > - Now, try to import each registry file specified in the command line and don't fail miserably when one of them is missing, but continue with the next one (as Windows' regedit does). > Patch by Edijs Kolesnikovičs (2/2) adapted by me. > > CORE-6239 #comment Fixed in r57462 with some adaptations. Thanks ;) #resolve > > Modified: > trunk/reactos/base/applications/regedit/regedit.c > > Modified: trunk/reactos/base/applications/regedit/regedit.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/applications/regedit/… > ============================================================================== > --- trunk/reactos/base/applications/regedit/regedit.c [iso-8859-1] (original) > +++ trunk/reactos/base/applications/regedit/regedit.c [iso-8859-1] Tue Oct 2 23:29:22 2012 > @@ -23,30 +23,31 @@ > > static const char *usage = > "Usage:\n" > - " regedit filename\n" > + " regedit filenames\n" > " regedit /E filename [regpath]\n" > " regedit /D regpath\n" > "\n" > - "filename - registry file name\n" > - "regpath - name of the registry key\n" > + "filenames - List of registry files names\n" > + "filename - Registry file name\n" > + "regpath - Name of the registry key\n" > "\n" > "When is called without any switches adds contents of the specified\n" > - "registry file to the registry\n" > + "registry files to the registry.\n" > "\n" > "Switches:\n" > - " /E - exports contents of the specified registry key to the specified\n" > - " file. Exports the whole registry if no key is specified.\n" > - " /D - deletes specified registry key\n" > - " /S - silent execution, can be used with any other switch.\n" > - " The only existing mode, exists for compatibility with Windows regedit.\n" > - " /V - advanced mode, can be used with any other switch.\n" > - " Ignored, exists for compatibility with Windows regedit.\n" > - " /L - location of system.dat file. Can be used with any other switch.\n" > - " Ignored. Exists for compatibility with Windows regedit.\n" > - " /R - location of user.dat file. Can be used with any other switch.\n" > - " Ignored. Exists for compatibility with Windows regedit.\n" > - " /? - print this help. Any other switches are ignored.\n" > - " /C - create registry from. Not implemented.\n" > + " /E - Exports contents of the specified registry key to the specified\n" > + " file. Exports the whole registry if no key is specified.\n" > + " /D - Deletes specified registry key\n" > + " /S - Silent execution, can be used with any other switch.\n" > + " The only existing mode, exists for compatibility with Windows regedit.\n" > + " /V - Advanced mode, can be used with any other switch.\n" > + " Ignored, exists for compatibility with Windows regedit.\n" > + " /L - Location of system.dat file. Can be used with any other switch.\n" > + " Ignored. Exists for compatibility with Windows regedit.\n" > + " /R - Location of user.dat file. Can be used with any other switch.\n" > + " Ignored. Exists for compatibility with Windows regedit.\n" > + " /? - Print this help. Any other switches are ignored.\n" > + " /C - Create registry from. Not implemented.\n" > "\n" > "The switches are case-insensitive, can be prefixed either by '-' or '/'.\n" > "This program is command-line compatible with Microsoft Windows\n" > @@ -74,10 +75,10 @@ > * where *s[0] is the first symbol of the file name. > * file_name - buffer to write the file name to. > */ > -void get_file_name(LPWSTR *command_line, LPWSTR file_name) > -{ > - WCHAR *s = *command_line; > - int pos = 0; /* position of pointer "s" in *command_line */ > +void get_file_name(LPTSTR *command_line, LPTSTR file_name) > +{ > + TCHAR *s = *command_line; > + int pos = 0; /* position of pointer "s" in *command_line */ > file_name[0] = 0; > > if (!s[0]) > @@ -85,11 +86,11 @@ > return; > } > > - if (s[0] == L'"') > + if (s[0] == _T('"')) > { > s++; > (*command_line)++; > - while(s[0] != L'"') > + while(s[0] != _T('"')) > { > if (!s[0]) > { > @@ -102,7 +103,7 @@ > } > else > { > - while(s[0] && !iswspace(s[0])) > + while(s[0] && !_istspace(s[0])) > { > s++; > pos++; > @@ -110,13 +111,13 @@ > } > memcpy(file_name, *command_line, pos * sizeof(WCHAR)); > /* remove the last backslash */ > - if (file_name[pos - 1] == L'\\') > - { > - file_name[pos - 1] = L'\0'; > + if (file_name[pos - 1] == _T('\\')) > + { > + file_name[pos - 1] = _T('\0'); > } > else > { > - file_name[pos] = L'\0'; > + file_name[pos] = _T('\0'); > } > > if (s[0]) > @@ -124,7 +125,7 @@ > s++; > pos++; > } > - while(s[0] && iswspace(s[0])) > + while(s[0] && _istspace(s[0])) > { > s++; > pos++; > @@ -132,88 +133,116 @@ > (*command_line) += pos; > } > > -BOOL PerformRegAction(REGEDIT_ACTION action, LPWSTR s, BOOL silent) > -{ > - TCHAR szTitle[256], szText[256]; > +BOOL PerformRegAction(REGEDIT_ACTION action, LPTSTR s, BOOL silent) > +{ > switch (action) > { > - case ACTION_ADD: > - { > - WCHAR filename[MAX_PATH]; > - FILE *fp; > - > - get_file_name(&s, filename); > - if (!filename[0]) > - { > - fprintf(stderr, "%s: No file name is specified\n", getAppName()); > - fprintf(stderr, usage); > - exit(4); > - } > - > - while(filename[0]) > - { > - fp = _wfopen(filename, L"r"); > - if (fp == NULL) > - { > - LPSTR p = GetMultiByteString(filename); > - perror(""); > - fprintf(stderr, "%s: Can't open file \"%s\"\n", getAppName(), p); > - HeapFree(GetProcessHeap(), 0, p); > - exit(5); > - } > - import_registry_file(fp); > + case ACTION_ADD: > + { > + TCHAR szTitle[512], szText[512]; > + TCHAR filename[MAX_PATH]; > + FILE *fp; > + > get_file_name(&s, filename); > + if (!filename[0]) > + { > + fprintf(stderr, "%s: No file name is specified\n", getAppName()); > + fprintf(stderr, usage); > + exit(4); > + } > + > LoadString(hInst, IDS_APP_TITLE, szTitle, COUNT_OF(szTitle)); > - LoadString(hInst, IDS_IMPORT_OK, szText, COUNT_OF(szText)); > - /* show successful import */ > - if (!silent) > - MessageBox(NULL, szText, szTitle, MB_OK); > - } > - break; > - } > - case ACTION_DELETE: > - { > - WCHAR reg_key_name[KEY_MAX_LEN]; > - get_file_name(&s, reg_key_name); > - if (!reg_key_name[0]) > - { > - fprintf(stderr, "%s: No registry key is specified for removal\n", getAppName()); > - fprintf(stderr, usage); > - exit(6); > - } > - delete_registry_key(reg_key_name); > - break; > - } > - case ACTION_EXPORT: > - { > - WCHAR filename[MAX_PATH]; > - > - filename[0] = _T('\0'); > - get_file_name(&s, filename); > - if (!filename[0]) > - { > - fprintf(stderr, "%s: No file name is specified\n", getAppName()); > - fprintf(stderr, usage); > - exit(7); > - } > - > - if (s[0]) > - { > - WCHAR reg_key_name[KEY_MAX_LEN]; > + > + while (filename[0]) > + { > + /* Request import confirmation */ > + if (!silent) > + { > + LoadString(hInst, IDS_IMPORT_PROMPT, szText, COUNT_OF(szText)); > + > + if (InfoMessageBox(NULL, MB_YESNO | MB_ICONWARNING, szTitle, szText, filename) != IDYES) > + goto cont; > + } > + > + fp = _tfopen(filename, _T("r")); > + if (fp != NULL) > + { > + import_registry_file(fp); > + > + /* Show successful import */ > + if (!silent) > + { > + LoadString(hInst, IDS_IMPORT_OK, szText, COUNT_OF(szText)); > + InfoMessageBox(NULL, MB_OK | MB_ICONINFORMATION, szTitle, szText, filename); > + } > + } > + else > + { > + LPSTR p = GetMultiByteString(filename); > + perror(""); > + fprintf(stderr, "%s: Can't open file \"%s\"\n", getAppName(), p); > + HeapFree(GetProcessHeap(), 0, p); > + > + /* Error opening the file */ > + if (!silent) > + { > + LoadString(hInst, IDS_IMPORT_ERROR, szText, COUNT_OF(szText)); > + InfoMessageBox(NULL, MB_OK | MB_ICONERROR, szTitle, szText, filename); > + } > + } > + > +cont: > + get_file_name(&s, filename); > + } > + break; > + } > + > + case ACTION_DELETE: > + { > + TCHAR reg_key_name[KEY_MAX_LEN]; > get_file_name(&s, reg_key_name); > - export_registry_key(filename, reg_key_name, REG_FORMAT_4); > - } > - else > - { > - export_registry_key(filename, NULL, REG_FORMAT_4); > - } > - break; > - } > - default: > - fprintf(stderr, "%s: Unhandled action!\n", getAppName()); > - exit(8); > - break; > - } > + if (!reg_key_name[0]) > + { > + fprintf(stderr, "%s: No registry key is specified for removal\n", getAppName()); > + fprintf(stderr, usage); > + exit(6); > + } > + delete_registry_key(reg_key_name); > + break; > + } > + > + case ACTION_EXPORT: > + { > + TCHAR filename[MAX_PATH]; > + > + filename[0] = _T('\0'); > + get_file_name(&s, filename); > + if (!filename[0]) > + { > + fprintf(stderr, "%s: No file name is specified\n", getAppName()); > + fprintf(stderr, usage); > + exit(7); > + } > + > + if (s[0]) > + { > + TCHAR reg_key_name[KEY_MAX_LEN]; > + get_file_name(&s, reg_key_name); > + export_registry_key(filename, reg_key_name, REG_FORMAT_4); > + } > + else > + { > + export_registry_key(filename, NULL, REG_FORMAT_4); > + } > + break; > + } > + > + default: > + fprintf(stderr, "%s: Unhandled action!\n", getAppName()); > + exit(8); > + break; > + } > + > return TRUE; > } > > @@ -233,35 +262,35 @@ > else > { > fprintf(stderr, "%s: Alphabetic character is expected after '%c' " > - "in swit ch specification\n", getAppName(), *(s - 1)); > + "in switch specification\n", getAppName(), *(s - 1)); > } > exit(1); > } > > -BOOL ProcessCmdLine(LPWSTR lpCmdLine) > +BOOL ProcessCmdLine(LPTSTR lpCmdLine) > { > BOOL silent = FALSE; > REGEDIT_ACTION action = ACTION_UNDEF; > - LPWSTR s = lpCmdLine; /* command line pointer */ > - WCHAR ch = *s; /* current character */ > - > - while (ch && ((ch == L'-') || (ch == L'/'))) > - { > - WCHAR chu; > - WCHAR ch2; > + LPTSTR s = lpCmdLine; /* command line pointer */ > + TCHAR ch = *s; /* current character */ > + > + while (ch && ((ch == _T('-')) || (ch == _T('/')))) > + { > + TCHAR chu; > + TCHAR ch2; > > s++; > ch = *s; > ch2 = *(s + 1); > - chu = (WCHAR)towupper(ch); > - if (!ch2 || iswspace(ch2)) > - { > - if (chu == L'S') > + chu = _totupper(ch); > + if (!ch2 || _istspace(ch2)) > + { > + if (chu == _T('S')) > { > /* Silence dialogs */ > silent = TRUE; > } > - else if (chu == L'V') > + else if (chu == _T('V')) > { > /* Ignore this switch */ > } > @@ -269,41 +298,41 @@ > { > switch (chu) > { > - case L'D': > - action = ACTION_DELETE; > - break; > - case L'E': > - action = ACTION_EXPORT; > - break; > - case L'?': > - fprintf(stderr, usage); > - exit(3); > - break; > - default: > - error_unknown_switch(chu, s); > - break; > + case _T('D'): > + action = ACTION_DELETE; > + break; > + case _T('E'): > + action = ACTION_EXPORT; > + break; > + case _T('?'): > + fprintf(stderr, usage); > + exit(3); > + break; > + default: > + error_unknown_switch(chu, s); > + break; > } > } > s++; > } > else > { > - if (ch2 == L':') > + if (ch2 == _T(':')) > { > switch (chu) > { > - case L'L': > - /* fall through */ > - case L'R': > - s += 2; > - while (*s && !iswspace(*s)) > - { > - s++; > - } > - break; > - default: > - error_unknown_switch(chu, s); > - break; > + case _T('L'): > + /* fall through */ > + case _T('R'): > + s += 2; > + while (*s && !_istspace(*s)) > + { > + s++; > + } > + break; > + default: > + error_unknown_switch(chu, s); > + break; > } > } > else > @@ -315,7 +344,7 @@ > } > /* skip spaces to the next parameter */ > ch = *s; > - while (ch && iswspace(ch)) > + while (ch && _istspace(ch)) > { > s++; > ch = *s; > @@ -323,25 +352,10 @@ > } > > if (*s && action == ACTION_UNDEF) > - { > - if (!silent) > - { > - TCHAR szTitle[256], szText[256]; > - LoadString(hInst, IDS_APP_TITLE, szTitle, COUNT_OF(szTitle)); > - LoadString(hInst, IDS_IMPORT_PROMPT, szText, COUNT_OF(szText)); > - /* request import confirmation */ > - if (MessageBox(NULL, szText, szTitle, MB_YESNO) == IDYES) > - action = ACTION_ADD; > - else > - return TRUE; > - } > - else > - { > - action = ACTION_ADD; > - } > - } > - if (action == ACTION_UNDEF) > + action = ACTION_ADD; > + > + if (action != ACTION_UNDEF) > + return PerformRegAction(action, s, silent); > + else > return FALSE; > - > - return PerformRegAction(action, s, silent); > -} > +} > > >

11 years, 11 months

Everithing is down?

by Александр

11 years, 11 months

General life fault

by caemyr＠myopera.com

Hiya Unfortunately, my lappy has died yesterday. I`ll RMA it this week (its just 6mo old...) but who knows how long will it take. I`ll get one as replacement from my work tomorow, but still this means no testing for me, just the web works for few next weeks. -- With best regards Caemyr

11 years, 11 months

Re: [ros-dev] [ros-diffs] [hbelusca] 57400: [NTOSKRNL] Coverity code defects fixes : - Cache: CID 701441 - Config: CIDs 716570, 716669, 716760 - Dbgk: Kdbg: CIDs 716571, 515128/9, 500432 - Ex: CIDs 500156/7, 515122, 716200/...

by Pierre Schweitzer

What's the purpose of such ASSERT? > Modified: trunk/reactos/ntoskrnl/fsrtl/notify.c > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/fsrtl/notify.c?re… > ============================================================================== > --- trunk/reactos/ntoskrnl/fsrtl/notify.c [iso-8859-1] (original) > +++ trunk/reactos/ntoskrnl/fsrtl/notify.c [iso-8859-1] Thu Sep 27 17:16:31 2012 > @@ -586,6 +586,13 @@ > /* Allocate new notification */ > NotifyChange = ExAllocatePoolWithTag(PagedPool | POOL_RAISE_IF_ALLOCATION_FAILURE, > sizeof(NOTIFY_CHANGE), 'FSrN'); > + > + /* > + * If NotifyChange == NULL then an > + * exception was already raised. > + */ > + ASSERT(NotifyChange != NULL); > + > RtlZeroMemory(NotifyChange, sizeof(NOTIFY_CHANGE)); > > /* Set basic information */ > -- Pierre Schweitzer <pierre(a)reactos.org> System Administrator ReactOS Foundation

11 years, 12 months

Monthly meeting - September 2012

by Aleksey Bragin

Hello, Let me invite you to the monthly status meeting taking place last Thursday of this month, 28th of September, 19:00 UTC. The meeting will be at irc://dev.reactos.org (Port 6667, no SSL) in the channel #meeting, or as Pierre/Colin say. Note that the IRC service will only be started shortly before the meeting. Your participation passwords will be emailed to you shortly before the meeting starts, and they are going to be different once again as they are not stored in any database. Hopefully it's not much of inconvenience. If someone still is not getting passwords sent before a meeting - please email Colin or Pierre before the meeting started to get one. In order to save time, let's choose who is going to be the minute taker on the upcoming meeting. As usual, Z98 is welcome to volunteer. The agenda will be posted shortly before the meeting, suggestions are welcome (send them to my email as usual). With the best regards, Aleksey Bragin.

11 years, 12 months

Fwd: buildbot failure in ReactOS on Windows_AMD64_1 VBox-Test

by caemyr＠myopera.com

@Giannis - as you required: http://svn.reactos.org/svn/reactos?view=revision&revision=57330 This commit is causing the following crash in user32_apitest:desktop (reported in CORE-6682 desktop.c:83: Test failed: 3: expected error 0xc0000142 in child process got 0x0 (../../win32ss/user/ntuser/desktop.c:340) err: Failed to reference desktop WinSta0/Default PID: --! (../../win32ss/user/ntuser/main.c:333) err: Failed to assign default dekstop and winsta to process (../../win32ss/user/ntuser/main.c:374) err: UserCreateThreadInfo failed! Freeing pti 0xB09C5A70 for TID 00000590 (../../dll/ntdll/ldr/ldrinit.c:829) LDR: DLL_PROCESS_ATTACH for dll "user32.dll" (InitRoutine: 77ABADBF) failed (../../dll/ntdll/ldr/ldrinit.c:2111) LDR: LdrpProcessInitialization failed running initialization routines; status c0000142 (../../dll/ntdll/ldr/ldrinit.c:2140) LDR: Process initialization failure for C:\ReactOS\bin\user32_apitest.exe; NTSTATUS = c0000142 (../../subsystems/win32/csrss/csrsrv/api/wapi.c:1221) Exception in 59c.590. Killing... (../../win32ss/user/ntuser/desktop.c:340) err: Failed to reference desktop NonExistantDesktop PID: --! (../../win32ss/user/ntuser/main.c:333) err: Failed to assign default dekstop and winsta to process (../../win32ss/user/ntuser/main.c:374) err: UserCreateThreadInfo failed! Freeing pti 0xB097F008 for TID 00000588 (../../dll/ntdll/ldr/ldrinit.c:829) LDR: DLL_PROCESS_ATTACH for dll "user32.dll" (InitRoutine: 77ABADBF) failed (../../dll/ntdll/ldr/ldrinit.c:2111) LDR: LdrpProcessInitialization failed running initialization routines; status c0000142 (../../dll/ntdll/ldr/ldrinit.c:2140) LDR: Process initialization failure for C:\ReactOS\bin\user32_apitest.exe; NTSTATUS = c0000142 (../../subsystems/win32/csrss/csrsrv/api/wapi.c:1221) Exception in 594.588. Killing... (../../win32ss/user/ntuser/desktop.c:308) err: Failed to reference window station \Windows\WindowStations\NonExistantWinsta PID: --! (../../win32ss/user/ntuser/main.c:333) err: Failed to assign default dekstop and winsta to process (../../win32ss/user/ntuser/main.c:374) err: UserCreateThreadInfo failed! Freeing pti 0xB0A444F8 for TID 00000580 (../../dll/ntdll/ldr/ldrinit.c:829) LDR: DLL_PROCESS_ATTACH for dll "user32.dll" (InitRoutine: 77ABADBF) failed (../../dll/ntdll/ldr/ldrinit.c:2111) LDR: LdrpProcessInitialization failed running initialization routines; status c0000142 (../../dll/ntdll/ldr/ldrinit.c:2140) LDR: Process initialization failure for C:\ReactOS\bin\user32_apitest.exe; NTSTATUS = c0000142 (../../subsystems/win32/csrss/csrsrv/api/wapi.c:1221) Exception in 58c.580. Killing... (../../ntoskrnl/mm/ARM3/section.c:386) Out of system view space Assertion 'Base' failed at ../../ntoskrnl/mm/ARM3/section.c line 1041 [7h Entered debugger on embedded INT3 at 0x0008:0x8092582e. kdb:> bt Eip: <NTOSKRNL.EXE:12582f (../../lib/rtl/i386/debug_asm.S:35 (_DbgBreakPoint@0))> Frames: <NTOSKRNL.EXE:b773f (../../ntoskrnl/mm/ARM3/section.c:1041 (MiMapViewInSystemSpace@16))> <NTOSKRNL.EXE:b78ee (../../ntoskrnl/mm/ARM3/section.c:2655 (MmMapViewInSessionSpace@12))> <win32k.sys:148c8 (../../win32ss/user/ntuser/misc/usrheap.c:201 (UserCreateHeap))> <win32k.sys:359e1 (../../include/ddk/wdm.h:0 (UserDestroyThreadInfo@4))> <NTOSKRNL.EXE:11100e (../../ntoskrnl/ke/i386/traphdlr.c:1629 (@KiFastCallEntryHandler@8))> <NTOSKRNL.EXE:10ebfd (../../ntoskrnl/ke/i386/trap.s:150 (_KiFastCallEntry))> <ntdll.dll:b065> <user32_apitest.exe:84f2> ----- Original message ----- From: buildbot(a)reactos.org To: caemyr(a)myopera.com Subject: buildbot failure in ReactOS on Windows_AMD64_1 VBox-Test Date: Wed, 19 Sep 2012 02:32:21 +0000 The Buildbot has finished a build on builder Windows_AMD64_1 VBox-Test while building ReactOS. Full details are available at: http://build.reactos.org/builders/Windows_AMD64_1%20VBox-Test/builds/6627 Buildbot URL: http://build.reactos.org/ Buildslave for this Build: Windows_AMD64_2 Build Reason: Triggerable(Windows_AMD64_1 VBox-Test Trigger) Build Source Stamp: 57330 Blamelist: ion BUILD FAILED: failed Cleanup sincerely, -The Buildbot -- With best regards Caemyr

12 years

Re: [ros-dev] [ros-diffs] [akhaldi] 57313: * Don't use -momit-leaf-frame-pointer in debug gcc builds, it messes up stack traces. * Remove it from the amd64 gcc builds too.

by Timo Kreuzer

Amd64 doesn't need a frame pointer. Backtraces are made based on unwind tables. At least if GCC conforms to the ABI. Am 16.09.2012 14:57, schrieb akhaldi(a)svn.reactos.org: > Author: akhaldi > Date: Sun Sep 16 12:57:22 2012 > New Revision: 57313 > > URL: http://svn.reactos.org/svn/reactos?rev=57313&view=rev > Log: > * Don't use -momit-leaf-frame-pointer in debug gcc builds, it messes up stack traces. > * Remove it from the amd64 gcc builds too. > > Modified: > trunk/reactos/cmake/gcc.cmake > > Modified: trunk/reactos/cmake/gcc.cmake > URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/gcc.cmake?rev=57313&… > ============================================================================== > --- trunk/reactos/cmake/gcc.cmake [iso-8859-1] (original) > +++ trunk/reactos/cmake/gcc.cmake [iso-8859-1] Sun Sep 16 12:57:22 2012 > @@ -83,12 +83,15 @@ > if(ARCH STREQUAL "i386") > add_compile_flags("-mpreferred-stack-boundary=3 -fno-set-stack-executable -fno-optimize-sibling-calls -fno-omit-frame-pointer") > if(OPTIMIZE STREQUAL "1") > - add_compile_flags("-ftracer -momit-leaf-frame-pointer") > + add_compile_flags("-ftracer") > + if(NOT CMAKE_BUILD_TYPE STREQUAL "Debug") > + add_compile_flags("-momit-leaf-frame-pointer") > + endif() > endif() > elseif(ARCH STREQUAL "amd64") > add_compile_flags("-mpreferred-stack-boundary=4") > if(OPTIMIZE STREQUAL "1") > - add_compile_flags("-ftracer -momit-leaf-frame-pointer") > + add_compile_flags("-ftracer") > endif() > elseif(ARCH STREQUAL "arm") > if(OPTIMIZE STREQUAL "1") > > >

12 years

Re: [ros-dev] [ros-diffs] [mnordell] 57288: Add comment explaining initialization.

by Thomas Faber

On 2012-09-13 02:52, mnordell(a)svn.reactos.org wrote: > --- trunk/reactos/subsystems/csr/csrsrv/process.c [iso-8859-1] (original) > +++ trunk/reactos/subsystems/csr/csrsrv/process.c [iso-8859-1] Thu Sep 13 00:52:12 2012 > @@ -1211,7 +1211,7 @@ > BOOLEAN FirstTry; > ULONG i; > PCSR_SERVER_DLL ServerDll; > - ULONG Result = 0; > + ULONG Result = 0; /* Intentionally invalid enumeratee to silence compiler warning */ > > /* Acquire process lock */ > CsrAcquireProcessLock(); Frankly, I still don't understand it :D As I recall, this warning isn't a false positive, is it? Thus one would logically "fix", not "silence" it.

12 years

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