Re: [PATCH 0/4] MR8925: winemac.drv: Add EDID support.

[Brendan Shanks (＠bshanks)]

Brendan Shanks (@bshanks) commented about dlls/winemac.drv/cocoa_display.m:
> +    {
> +        uint32_t vendor_number, model_number, serial_number;
> +        const unsigned char *edid_ptr;
> +        IOAVServiceRef avservice;
> +        CFDataRef edid = NULL;
> +        IOReturn edid_result;
> +
> +        avservice = pIOAVServiceCreateWithService(kCFAllocatorDefault, service);
> +        if (!avservice)
> +        {
> +            IOObjectRelease(service);
> +            continue;
> +        }
> +
> +        edid_result = pIOAVServiceCopyEDID(avservice, &edid);
> +        if (edid_result != kIOReturnSuccess || !edid)
> Maybe also a good idea to call `CFDataGetLength()` here and ensure the CFData is large enough that the `edid_ptr` accesses below won't be reading past the buffer size.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8925#note_117488