Ros-dev

ros-dev@reactos.org

5791 discussions

Re: [ros-dev] [ros-builds] Linux_AMD64_2 VMWPlayer-Test: build failed
by Pierre Schweitzer 20 May '13

20 May '13

Hi, Please note that r59047 broke VMware testbot. It times out in setupapi:install and fails to restart afterwards. Regards, Pierre On 20/05/2013 15:01, buildbot(a)reactos.org wrote: > The Buildbot has detected a failed build on builder Linux_AMD64_2 VMWPlayer-Test while building ReactOS. > Full details are available at: > http://build.reactos.org/builders/Linux_AMD64_2%20VMWPlayer-Test/builds/869 > > Buildbot URL: http://build.reactos.org/ > > Buildslave for this Build: Linux_AMD64_2 > > Build Reason: Triggerable(Linux_AMD64_1 KVM-Test Trigger) > Build Source Stamp: 59050 > Blamelist: > > BUILD FAILED: failed test > > sincerely, > -The Buildbot > > > > -- Pierre Schweitzer<pierre at reactos.org> System Administrator ReactOS Foundation

2 2

Important Test regressions due to FastFat changes
by victor martinez 17 May '13

17 May '13

The following tests are introducing several bugs/regressions: [ros-diffs] [ekohl] 59021: [SYSSETUP] Enable the restore privilege before the call to NtUnloadKey. Disable it immediately after the call to NtUnloadKey. ekohl at svn.reactos.org ^This one introduces a potential regression at: user32:clipboard From 49216 tests executed, aftwerdars 49208 (-8)shell32:appbar. From 61 test executed, afterwards 58 [ros-diffs] [ekohl] 59022: [FASTFAT] Allow only readonly, hidden, system and archive files attributes to be set for existing files. Other file attributes must be ignored. This fixes several tests. ekohl at svn.reactos.org ^This introduces a whole bunch of regressions easily spoted. ole32:storage32oleaut32:typelibregedit:regeditsetupapi:miscsetupapi:parserEtc,Etc [ros-diffs] [ekohl] 59023: [FASTFAT] Implement a special case for existing files which are opened using the FILE_OVERWRITE or FILE_OVERWRITE_IF file disposition: New file attributes must be ORed with the exist... ekohl at svn.reactos.org ^The following regresses: zz_notepad++_6.1.2:1.installzz_notepad++_6.1.2:2.SaveAsDesktopzz_notepad++_6.1.2:3.OpenDocumentzz_notepad++_6.1.2:4.Ctrl-F.FindDialogzz_notepad++_6.1.2:5.Send_InputAmong others Comparison chart: http://www.reactos.org/sites/all/modules/reactos/testman/compare.php?ids=17…

2 1

Unexpected downtime
by Pierre Schweitzer 14 May '13

14 May '13

Hi, due to critical load issues on reactos.org, the webserver has been taken down now for an extra maintenance to address the issues. Affected services are the following: reactos.org, doxygen.reactos.org (still reachable but slow), jira.reactos.org (no auth), code.reactos.org (no auth). We cannot provide any ETA for the end of the maintenance. We are sorry for the caused inconveniences. With my best regards, -- Pierre Schweitzer<pierre at reactos.org> System Administrator ReactOS Foundation

1 2

Re: [ros-dev] [ros-diffs] [ekohl] 59013: [LSASRV] - LsapCheckLogonProcess: Open client process using the PROCESS_DUP_HANDLE access right because we need to duplicate the logon token back into the clients handle table.
by Thomas Faber 14 May '13

14 May '13

On 2013-05-13 15:05, ekohl(a)svn.reactos.org wrote: > Status = NtReplyWaitReceivePort(AuthPortHandle, > (PVOID*)&LogonContext, > - &ReplyMsg->h, > - &RequestMsg.h); > + (PPORT_MESSAGE)ReplyMsg, > + (PPORT_MESSAGE)&RequestMsg); As the Guy Who Officially Hates Casts™, I have to ask: wasn't the previous version better for type safety? With the cast, an additional [or missing] & can slip in and the code will compile without complaint. Best, Thomas

1 0

Re: [ros-dev] [ros-diffs] [hbelusca] 58971: [FREELDR] - Fix dprints (be careful when "Status" variables are booleans). - Don't fail when trying to load an non-existent KD transport dll.
by Timo Kreuzer 10 May '13

10 May '13

I suggest NEVER to use Status as the name for anything other than NTSTATUS variables. Otherwise problems are almost preprogrammed (seen this multiple times). It Should be called Result or something. This is one of those cases where hungarian notation really makes a difference, Colin. ;-) Am 08.05.2013 19:52, schrieb hbelusca(a)svn.reactos.org: > Author: hbelusca > Date: Wed May 8 17:52:16 2013 > New Revision: 58971 > > URL: http://svn.reactos.org/svn/reactos?rev=58971&view=rev > Log: > [FREELDR] > - Fix dprints (be careful when "Status" variables are booleans). > - Don't fail when trying to load an non-existent KD transport dll. > > Modified: > branches/kd++/boot/freeldr/freeldr/disk/scsiport.c > branches/kd++/boot/freeldr/freeldr/windows/peloader.c > branches/kd++/boot/freeldr/freeldr/windows/winldr.c > > Modified: branches/kd++/boot/freeldr/freeldr/disk/scsiport.c > URL: http://svn.reactos.org/svn/reactos/branches/kd%2B%2B/boot/freeldr/freeldr/d… > ============================================================================== > --- branches/kd++/boot/freeldr/freeldr/disk/scsiport.c [iso-8859-1] (original) > +++ branches/kd++/boot/freeldr/freeldr/disk/scsiport.c [iso-8859-1] Wed May 8 17:52:16 2013 > @@ -1568,7 +1568,7 @@ > ULONG ImportTableSize; > PLDR_DATA_TABLE_ENTRY BootDdDTE, FreeldrDTE; > CHAR NtBootDdPath[MAX_PATH]; > - PVOID ImageBase; > + PVOID ImageBase = NULL; > ULONG (NTAPI *EntryPoint)(IN PVOID DriverObject, IN PVOID RegistryPath); > BOOLEAN Status; > > > Modified: branches/kd++/boot/freeldr/freeldr/windows/peloader.c > URL: http://svn.reactos.org/svn/reactos/branches/kd%2B%2B/boot/freeldr/freeldr/w… > ============================================================================== > --- branches/kd++/boot/freeldr/freeldr/windows/peloader.c [iso-8859-1] (original) > +++ branches/kd++/boot/freeldr/freeldr/windows/peloader.c [iso-8859-1] Wed May 8 17:52:16 2013 > @@ -255,11 +255,12 @@ > return TRUE; > } > > -/* WinLdrLoadImage loads the specified image from the file (it doesn't > - perform any additional operations on the filename, just directly > - calls the file I/O routines), and relocates it so that it's ready > - to be used when paging is enabled. > - Addressing mode: physical > +/* > + * WinLdrLoadImage loads the specified image from the file (it doesn't > + * perform any additional operations on the filename, just directly > + * calls the file I/O routines), and relocates it so that it's ready > + * to be used when paging is enabled. > + * Addressing mode: physical > */ > BOOLEAN > WinLdrLoadImage(IN PCHAR FileName, > @@ -429,7 +430,6 @@ > /* If loading failed - return right now */ > if (Status != ESUCCESS) > return FALSE; > - > > /* Relocate the image, if it needs it */ > if (NtHeaders->OptionalHeader.ImageBase != (ULONG_PTR)VirtualBase) > @@ -756,7 +756,7 @@ > { > CHAR FullDllName[256]; > BOOLEAN Status; > - PVOID BasePA; > + PVOID BasePA = NULL; > > /* Prepare the full path to the file to be loaded */ > strcpy(FullDllName, DirectoryPath); > @@ -781,7 +781,7 @@ > DataTableEntry); > if (!Status) > { > - ERR("WinLdrAllocateDataTableEntry() failed with Status=0x%X\n", Status); > + ERR("WinLdrAllocateDataTableEntry() failed\n"); > return Status; > } > > @@ -791,7 +791,7 @@ > Status = WinLdrScanImportDescriptorTable(ModuleListHead, DirectoryPath, *DataTableEntry); > if (!Status) > { > - ERR("WinLdrScanImportDescriptorTable() failed with Status=0x%X\n", Status); > + ERR("WinLdrScanImportDescriptorTable() failed\n"); > return Status; > } > > > Modified: branches/kd++/boot/freeldr/freeldr/windows/winldr.c > URL: http://svn.reactos.org/svn/reactos/branches/kd%2B%2B/boot/freeldr/freeldr/w… > ============================================================================== > --- branches/kd++/boot/freeldr/freeldr/windows/winldr.c [iso-8859-1] (original) > +++ branches/kd++/boot/freeldr/freeldr/windows/winldr.c [iso-8859-1] Wed May 8 17:52:16 2013 > @@ -237,7 +237,7 @@ > CHAR DllName[1024]; > PCHAR DriverNamePos; > BOOLEAN Status; > - PVOID DriverBase; > + PVOID DriverBase = NULL; > > // Separate the path to file name and directory path > _snprintf(DriverPath, sizeof(DriverPath), "%wZ", FilePath); > @@ -258,7 +258,6 @@ > } > > TRACE("DriverPath: %s, DllName: %s, LPB\n", DriverPath, DllName); > - > > // Check if driver is already loaded > Status = WinLdrCheckForLoadedDll(LoadOrderListHead, DllName, DriverDTE); > @@ -424,7 +423,7 @@ > } > > static > -PVOID > +BOOLEAN > LoadModule( > PLOADER_PARAMETER_BLOCK LoaderBlock, > PCCH Path, > @@ -434,10 +433,10 @@ > BOOLEAN IsKdTransportDll, > ULONG Percentage) > { > + BOOLEAN Status; > CHAR FullFileName[MAX_PATH]; > CHAR ProgressString[256]; > - NTSTATUS Status; > - PVOID BaseAdress; > + PVOID BaseAdress = NULL; > > UiDrawBackdrop(); > sprintf(ProgressString, "Loading %s...", File); > @@ -448,8 +447,12 @@ > strcat(FullFileName, File); > > Status = WinLdrLoadImage(FullFileName, MemoryType, &BaseAdress); > - TRACE("%s loaded with status %d at %p\n", > - File, Status, BaseAdress); > + if (!Status) > + { > + TRACE("Loading %s failed\n", File); > + return FALSE; > + } > + TRACE("%s loaded successfully at %p\n", File, BaseAdress); > > strcpy(FullFileName, "WINDOWS\\SYSTEM32\\"); > strcat(FullFileName, File); > @@ -458,13 +461,13 @@ > * the Kernel Debugger Transport DLL, to make the > * PE loader happy. > */ > - WinLdrAllocateDataTableEntry(&LoaderBlock->LoadOrderListHead, > - (IsKdTransportDll ? "KDCOM.DLL" : File), > - FullFileName, > - BaseAdress, > - Dte); > - > - return BaseAdress; > + Status = WinLdrAllocateDataTableEntry(&LoaderBlock->LoadOrderListHead, > + (IsKdTransportDll ? "KDCOM.DLL" : File), > + FullFileName, > + BaseAdress, > + Dte); > + > + return Status; > } > > static > @@ -679,11 +682,11 @@ > UiDrawBackdrop(); > UiDrawProgressBarCenter(15, 100, "Loading system hive..."); > Status = WinLdrInitSystemHive(LoaderBlock, BootPath); > - TRACE("SYSTEM hive loaded with status %d\n", Status); > + TRACE("SYSTEM hive %s\n", (Status ? "loaded" : "not loaded")); > > /* Load NLS data, OEM font, and prepare boot drivers list */ > Status = WinLdrScanSystemHive(LoaderBlock, BootPath); > - TRACE("SYSTEM hive scanned with status %d\n", Status); > + TRACE("SYSTEM hive %s\n", (Status ? "scanned" : "not scanned")); > > /* Finish loading */ > LoadAndBootWindowsCommon(OperatingSystemVersion, > > >

2 1

Re: [ros-dev] [ros-diffs] [pschweitzer] 58960: [RTL] Finally reenable RtlAcquirePrivilege
by Jérôme Gardou 06 May '13

06 May '13

Hey Pierre! Nice patch series, that takes ReactOS nearer to being secure. A remark though : > /* > - * enable the SeSystemtimePrivilege privilege > + * Call SetLocalTime twice to ensure correct results > */ > + Ret = SetLocalTime(&SetupData->SystemTime) && > + SetLocalTime(&SetupData->SystemTime); This is a bit confusing, and the comment doesn't really help. Why would calling the function twice ansure correct result ? Also, this is likely to be optimized away by the compiler, or at least produce a report in static code analysers. I know this isn't pure chance. Any hint as to why you wrote it like this ? Cheers Jérôme

4 4

getdate.exe being pulled from SVN repository
by J. C. Jones 05 May '13

05 May '13

Small nit-pick: .\reactos\reactos\tools\getdate.exe is being pulled from SVN even though it is being generated by getdate.cmd and getdate.c that are both in the same directory? -John

3 3

Planned services downtime
by Pierre Schweitzer 03 May '13

03 May '13

Hi, on the 3rd of May, from 1PM to 6PM CET, the following services will down due to a power maintenance: -> ISO upload -> ISO download -> Doxygen -> cppcheck -> Linux buildbot -> KVM testbot -> VMware testbot I warmly recommend you don't commit during that period then, as remaining builder, Windows one, won't be able to upload its ISOs. Actually, I could also take the opportunity of such downtime to take SVN (svn, git, buildbot master) server down for maintenance as it will barely harm. I'll keep you informed. I'll push reminders before it happens. Sorry for the services interruption. With my best regards, -- Pierre Schweitzer<pierre at reactos.org> System Administrator ReactOS Foundation

1 4

reactos.inf
by Stefan Naumann 03 May '13

03 May '13

Hello. I am the guy writing an updater for ReactOS. Recent tests show some difficulties and errors in data from reactos bootcds; mainly reactos.inf: My program reads reactos.inf, puts the filepaths together and copies files to the specified positions in folder-tree (I'm talking about the server's side). It seems that reactos.inf contains pretty weird data; opening reactos.inf with gedit shows it intact; reading it with the program gets the following things out (shortened): " -- 6 -->cache/files/��" should be "cache/files/inf" (with cache/files" beeing the path of all cached files, unpacked from bootcd-ISO. and (in the same context): " -- 5 = system32\drivers\etc -- 6 = inf -- 7 = bin -- �Í = bin\data -- 9 = media -- 10 = Microsoft.NET " ("--" comes from debug-function of my iniparser). Is this intended to be as is or not? Could someone please fix that or let me know whether I should think about doing something differently. I'm working and testing with Xubuntu 13.04 amd64 - always getting latest dbg-BootCD-ISO. Thanks peope. Regards Stefan Naumann

2 4

Re: [ros-dev] [ros-diffs] [hbelusca] 58902: [FREELDR] - Fix some function parameter types, so to avoid unuseful casts (from pointers to const strings to pointers to strings in particular). - The big thing: make FreeLdr load...
by Timo Kreuzer 02 May '13

02 May '13

I'm not a fan of single line conditional statements. I'd prefer if we had a style rule for that. Any thoughts on that? Am 01.05.2013 19:12, schrieb hbelusca(a)svn.reactos.org: > Author: hbelusca > Date: Wed May 1 17:12:56 2013 > New Revision: 58902 > do > { > - while (*LoadOptions == '/') > - ++LoadOptions; > - > + while (*LoadOptions == '/') ++LoadOptions; > *NewLoadOptions++ = *LoadOptions; > } while (*LoadOptions++); > >

6 7

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-dev