[IPHLPAPI] - Fix a use after free in getNthInterfaceEntity (CID 716795 - partly reverts r53190). Fix a possible double free in getArpTable (CID 716796).
svn path=/trunk/; revision=57127
if (!(Ret = LoadStringW(hInst, uID[i], s, ln)))
{
LocalFree((HLOCAL)(*lpTarget));
+ return 0;
}
s += Ret;
TRACE("Index %d is entity #%d - %04x:%08x\n", index, i,
entitySet[i].tei_entity, entitySet[i].tei_instance );
- tdiFreeThingSet( entitySet );
-
if( numInterfaces == index && i < numEntities ) {
memcpy( ent, &entitySet[i], sizeof(*ent) );
+ tdiFreeThingSet( entitySet );
return STATUS_SUCCESS;
} else {
+ tdiFreeThingSet( entitySet );
return STATUS_UNSUCCESSFUL;
}
}
(PVOID *)&AdapterArpTable,
&returnSize );
- if( status == STATUS_SUCCESS ) {
- for( TmpIdx = 0; TmpIdx < returnSize; TmpIdx++, CurrIdx++ )
- IpArpTable->table[CurrIdx] = AdapterArpTable[TmpIdx];
- }
-
- if( AdapterArpTable ) tdiFreeThingSet( AdapterArpTable );
- }
+ if( status == STATUS_SUCCESS ) {
+ for( TmpIdx = 0; TmpIdx < returnSize; TmpIdx++, CurrIdx++ )
+ IpArpTable->table[CurrIdx] = AdapterArpTable[TmpIdx];
+ tdiFreeThingSet( AdapterArpTable );
+ }
+ }
}
closeTcpFile( tcpFile );