<ion(a)cvs.reactos.com> wrote:
> CVSROOT: /CVS/ReactOS
> Module name: reactos
> Repository: reactos/ntoskrnl/mm/
> Changes by: ion(a)mok.osexperts.com 04/10/04 14:38:49
>
> Modified files:
> ./: config rules.mak
> reactos/include/ntdll/: rtl.h
> reactos/ntoskrnl/: Makefile
> reactos/ntoskrnl/include/internal/i386/: mm.h
> reactos/ntoskrnl/ke/i386/: multiboot.S
> reactos/ntoskrnl/mm/: mminit.c
>
> Log message:
> Un-hardcoded the 0xC0000000 value, and stuck the define in one single
> place both for ASM and C code. This define is 0x80000000 by default, but
> automatically sets itself to 0xC0000000 if 3GB is set to 1 in
> reactos/config. This option is turned on by default for now, because the
> Default heap is at 0xA0000000, which doesn't work on a non 3GB system.
> Lower values seem to crash Win32K but I fill figure out a solution.
I tried a NULL base address in the call to RtlCreateHeap (in
lib/ntdll/ldr/startup.c) and regedit (WIN32 app) works very well. The NULL
base address enables the default bottom-up memory allocation so the heap
will be allocated from the bootom of the process address space just like
Windows does. HEAP_BASE should have been removed ages ago.
Regards,
Eric
Is Sanos`s implementation of ndis and rtl8139 driver etc.
interesting for ReactOS developers ?
http://www.jbox.dk/sanos/index.htm
Regards, David
--
David Kredba <kredba at ibot.cas.cz>
GPG: ID 1024D/5B6B02DE
Fingerprint: F0B3312596BEDCF91DFB 0699E06AACD75B6B02DE
>From current CVS, running in whole-page alloc mode, I see a buffer
overrun in lib/rtl/acl.c early in the boot process. What I see is
a few calls to RtlpAddKnownAce which slowly overgrow the available
buffer. The following patch fixes it, but I don't know if it's
right as I'm not sure if AclSize should be adjusted. The reason
a patch like this is needed is that AclSize is checked against
the size of the Sid to be copied in, but that size never changes
so as many sids as desired could be copied in without the check
at line 136 going off, thus overrunning available space.
--
Hey, Adam Smith, keep your invisible hands to yourself!
Hi Hartmut,
Thanks for the commit, I wasn't sure how to fix those!
Best Regards,
Alex Ionescu
Hartmut Birr wrote:
>CVSROOT: /CVS/ReactOS
>Module name: reactos
>Repository: reactos/ntoskrnl/ke/i386/
>Changes by: hbirr(a)mok.osexperts.com 04/10/05 11:53:28
>
>Modified files:
> reactos/ntoskrnl/mm/i386/: page.c
> reactos/ntoskrnl/ke/i386/: multiboot.S
>
>Log message:
> - Removed some hard coded values, which depend on a kernel base of 0xc0000000.
>
>_______________________________________________
>Ros-cvs mailing list
>Ros-cvs(a)reactos.com
>http://reactos.com/mailman/listinfo/ros-cvs
>
>
>
Hi,
I just want to make a few annotations to the new dhcpd.
Why do we have to add these useless files, like shell scripts, makefile.am,
makefile.in, etc. to the CVS tree if we want to make dhcpd a real service?
The source code will loose its portability anyway. And why keep this rather
complex directory structure for such a simple utility?
Btw, the dhcp server looks like a hack to me. It is leaking lots of memory
because there is not a single call to free() in the source and the dhcp
options handling it pretty crude. And it doesn't seem to work with a Win2K
client.
I'd suggest to use the Moreton dhcp server (dhcp-win-0.8.25.tar) instead. It
has got a much better internal structure and it works properly. I ran it on
Win98SE and tested it with a Win2K client. The only disadvantage of the
Moreton server is that it uses a binary config file.
Regards,
Eric
Hi,
why is the drive letter added to users path if the install drive isn't C?
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft")
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft\Internet
Explorer")
CreateDirectory("G:\Documents and
Settings\Administrator.REACTOS_G\Application Data\Microsoft\Internet
Explorer\Quick Launch")
- Hartmut
In regards to the earlier change, this is why i made it ;)
The broodwar installer works, along with d2, and the d2 expansion.
Starcraft's installer fails with an odd error mentioning an invalid
window handle.
