Interesting that the wine code doesn't check for NULL at HeapAlloc at all
- could these be submitted to wine?

Additionally, at least one HeapAlloc isn't checked also in our build:

piDx = HeapAlloc(GetProcessHeap(),0,sizeof(INT) * (*count));

Perhaps a

#ifdef __REACTOS__
        if (piDx == NULL)
        {
             return FALSE; //Not sure about this, just looked at the diff ;)
        }
#endif

would be usefull there.

Best regards,
Michael Fritscher

> Author: akhaldi
> Date: Sat May 30 17:14:16 2015
> New Revision: 67974
>
> URL: http://svn.reactos.org/svn/reactos?rev=67974&view=rev
> Log:
> [USER32] Sync edit.c with Wine Staging 1.7.37. CORE-9246
>
> Modified:
>     trunk/reactos/media/doc/README.WINE
>     trunk/reactos/win32ss/user/user32/controls/edit.c
>





_______________________________________________
Ros-dev mailing list
[email protected]
http://www.reactos.org/mailman/listinfo/ros-dev

Reply via email to