[ros-dev] RE: [ros-svn] [ion] 17650: - Fix kernel32 and ntoskrnl
ionucu at videotron.ca
Mon Sep 5 18:51:33 CEST 2005
Ge van Geldorp wrote:
>Ok, I have to wonder, did you bother to do ANY testing on your
>DEVOBJ_EXTENSION/EXTENDED_DEVOBJ_EXTENSION changes at all?
>Like, uhm, just
>trying to boot?
>'cause it seems very unlikely to me that you were able to
I was, but since I hadn't worked on ReactOS in about two weeks I forgot
to properly install the new files, and I was still running the old source.
>IoCreateDevice allocates only enough room for
>DEVOBJ_EXTENSION, but the allocated pointer is cast to
>PEXTENDED_DEVOBJ_EXTENSION (a much larger struct) all over the place, so
>we're reading/writing outside allocated memory in a zillion places.
Yes, I just noticed that now, thanks for fixing it...
>Gé van Geldorp.
Mistakes happen... do we really have to lash out on each others
everytime? I'm starting to understand why so many people are getting
tired of working on the project. People take time to work on complex, or
simply long and ugly things, and instead of a single ounce of respect or
thanks, they get yelled at for accidentally breaking something. It took
me four hours to properly ensure everything built properly... do you
really think I didn't bother to test?
More information about the Ros-dev