Created attachment 656789 [details]
Experimental patch

Description of problem:
I have a CMPI provider, which calls CMPIBroker->getInstance(). The call returns CMPIInstance 'i' with several properties, one of them is  uint16[] with "empty array" value, i.e. it's not Null and its size is 0. But i->ft->getPropertyAt() returns Null instead of empty array.


Version-Release number of selected component (if applicable):
tog-pegasus-2.12.0-2.fc18.x86_64

Looking at Pegasus implementation of getPropertyAt:

        SCMO_RC src = inst->getPropertyAt((Uint32)pos,
                                          &pName,
                                          type,
                                          &value,
                                          isArray,
                                          size);

After the above call: value = NULL, isArray == true, size == 0, i.e. everything is fine. Following code is then executed:

        switch(src)
        {
            case SCMO_OK:
            {
                CMPIType ct=type2CMPIType(type, isArray);
                CMPISCMOUtilities::scmoValue2CMPIData(value, ct, &data, size);
                if ((ct&~CMPI_ARRAY) == CMPI_string)
                {
                    // We always receive strings as an array of pointers
                    // with at least one element, which needs to be released
                    // after it was converted to CMPIData
                    free((void*)value);
                }
                break;

Since value == NULL, CMPISCMOUtilities::scmoValue2CMPIData() sets data->state to CMPI_nullValue, which is wrong. It should set it to empty array!

Attached patch fixes the behavior for me.