[ros-dev] RE: [ros-svn] [ion] 17650: - Fix kernel32 and ntoskrnl build issues.

[Alex Ionescu]

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? 
>
Yes...

>Like, uhm, just
>trying to boot? 
>
Yes

>'cause it seems very unlikely to me that you were able to
>successfully boot. 
>
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?

Best regards,
Alex Ionescu