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.