[jira] [Created] (AXIS2C-1609) memory leak in axutil_hash_first() / axutil_hash_next() in generated xml deserializer code for an xsd element with attributes

["Jürgen Keil (JIRA)" <ji...@apache.org>]

Jürgen Keil created AXIS2C-1609:
-----------------------------------

             Summary: memory leak in axutil_hash_first() / axutil_hash_next() in generated xml deserializer code for an xsd element with attributes
                 Key: AXIS2C-1609
                 URL: https://issues.apache.org/jira/browse/AXIS2C-1609
             Project: Axis2-C
          Issue Type: Bug
          Components: code generation, util
    Affects Versions: 1.7.0
         Environment: probably all (noticed on iOS with Xcode memory leak detector)
            Reporter: Jürgen Keil


In the generated adb_{type}_deserialize() code ( adb_ActiveOrHistoricCurrencyAndAmount_deserialize() 
in my case), code like this can be found:

...
                  parent_attri = NULL;
                  attrib_text = NULL;
                  if(attribute_hash)
                  {
                       axutil_hash_index_t *hi;
                       void *val;
                       const void *key;

                       for (hi = axutil_hash_first(attribute_hash, env); hi; hi = axutil_hash_next(env, hi)) 
                       {
                           axutil_hash_this(hi, &key, NULL, &val);
                           
                           
                               if(!strcmp((axis2_char_t*)key, "Ccy"))
                             
                               {
                                   parent_attri = (axiom_attribute_t*)val;
                                   break;
                               }
                       }
                  }

                  if(parent_attri)
                  {
                    attrib_text = axiom_attribute_get_value(parent_attri, env);
                  }
                  else
                  {
                    /* this is hoping that attribute is stored in "Ccy", this happnes when name is in default namespace */
                    attrib_text = axiom_element_get_attribute_value_by_name(parent_element, env, "Ccy");
                  }
...

axutil_hash_first() allocates some memory (when env != NULL), which is freed by
axutil_hash_next() when we reach the end of the hash list.

But when we do find the correct key before reaching the end of the list ("Ccy" in the
above case), we break out of the for loop and axutil_hash_next() will never reach the
end of the list so that the allocated "hi" hash iterator never gets freed.


(Partial) XSD to reproduce the issue (xml element with attribute);

        <xs:complexType name="ActiveOrHistoricCurrencyAndAmount">
                <xs:simpleContent>
                        <xs:extension base="ActiveOrHistoricCurrencyAndAmount_SimpleType">
                                <xs:attribute name="Ccy" type="ActiveOrHistoricCurrencyCode" use="required"/>
                        </xs:extension>
                </xs:simpleContent>
        </xs:complexType>

        <xs:simpleType name="ActiveOrHistoricCurrencyAndAmount_SimpleType">
                <xs:restriction base="xs:decimal">
                        <xs:minInclusive value="0"/>
                        <xs:fractionDigits value="5"/>
                        <xs:totalDigits value="18"/>
                </xs:restriction>
        </xs:simpleType>

        <xs:simpleType name="ActiveOrHistoricCurrencyCode">
                <xs:restriction base="xs:string">
                        <xs:pattern value="[A-Z]{3,3}"/>
                </xs:restriction>
        </xs:simpleType>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscribe@axis.apache.org
For additional commands, e-mail: c-dev-help@axis.apache.org