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

[Brendan Shanks (＠bshanks)]

Brendan Shanks (@bshanks) commented about dlls/winemac.drv/cocoa_display.m:
> +        uint32_t vendor_number_edid, model_number_edid, serial_number_edid;
> +        const unsigned char *edid_ptr;
> +        IOAVServiceRef avservice;
> +        IOReturn edid_result;
> +
> +        avservice = pIOAVServiceCreateWithService(kCFAllocatorDefault, service);
> +        IOObjectRelease(service);
> +        if (!avservice)
> +            continue;
> +
> +        edid_result = pIOAVServiceCopyEDID(avservice, &edid);
> +        CFRelease(avservice);
> +        if (edid_result != kIOReturnSuccess || !edid || CFDataGetLength(edid) < 13)
> +        {
> +            if (edid)
> +                CFRelease(edid);
> `edid` also needs to be set to NULL here, otherwise it could be returned if this is the last DCPAVServiceProxy.

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