Thanks. That's a good point. In Windows,������GdipEnumerateMetafileSrcRectDestRectI is defined in gdiplusflat.h as������ :������
GpStatus WINGDIPAPI
GdipEnumerateMetafileSrcRectDestRectI(
������ ������ GpGraphics * ������ ������ ������ ������ ������ ������graphics,
������ ������ GDIPCONST GpMetafile * ������metafile,
������ ������ GDIPCONST Rect & ������ ������ ������ ������destRect,
������ ������ GDIPCONST Rect & ������ ������ ������ ������srcRect,
������ ������ Unit ������ ������ ������ ������ ������ ������ ������ ������ ������ ������srcUnit,
������ ������ EnumerateMetafileProc ������ callback,
������ ������ VOID * ������ ������ ������ ������ ������ ������ ������ ������ ������callbackData,
������ ������ GDIPCONST GpImageAttributes * ������ ������ imageAttributes);������

it means the arguments normally cannot be null because the compiler would block.������
We could call the function with null value of two arguments������when we use GetProcAddress.������
In that case, the program gets a crash in Windows. if a developer made a SEH block, the program should go to the exception handler.������
So, I think we should not check for null pointers to make it work the same.������

2021��������� 5��������� 17��������� (���������) ������������������ 1:06, Esme Povirk (they/them) <esme@codeweavers.com>������������������ ������������������:
We should check if destRect or srcRect are NULL before dereferencing
them. Based on the float version, a NULL destRect should result in an
InvalidParameter return, and a NULL srcRect should be passed through
as NULL.