[PATCH 5/9] msscript.ocx: Implement ScriptProcedureCollection::get_Item.

[Jacek Caban]

On 11.08.2020 17:38, Gabriel Ivăncescu wrote:
> On 11/08/2020 14:51, Jacek Caban wrote:
>> Hi Gabriel,
>>
>> On 10.08.2020 16:58, Gabriel Ivăncescu wrote:
>>> +        case DESCKIND_TYPECOMP:
>>> +            ITypeComp_Release(bindptr.lptcomp);
>>> +            break;
>>> +        default:
>>> +            break;
>>> +        }
>>> +        return CTL_E_ILLEGALFUNCTIONCALL;
>>
>>
>> You leak typeinfo here.
>>
>>
>> Thanks,
>>
>> Jacek
>>
>
> Hi Jacek,
>
> Sorry, I'm not familiar with the exact details. Does a typeinfo always 
> get returned? Because MSDN states:
>
> `If a FUNCDESC or VARDESC was returned, then ppTInfo points to a 
> pointer to the type description that contains the item to which it is 
> bound.`
>
> Additionally also for DESCKIND_IMPLICITAPPOBJ in the Remarks. But not 
> for other returns.
>
> Should I move the release of the typeinfo outside the switch, and 
> check it for NULL instead before releasing, or is that NULL check not 
> needed? 


I misinterpreted it, sorry. This part looks good.


Thanks,

Jacek