[prev in list] [next in list] [prev in thread] [next in thread] 

List:       axis-c-dev
Subject:    [jira] Updated: (AXIS2C-1486) ADB generated code generates memory
From:       "Mirko Wischer (JIRA)" <jira () apache ! org>
Date:       2010-07-12 9:22:50
Message-ID: 12773557.322621278926570873.JavaMail.jira () thor
[Download RAW message or body]


     [ https://issues.apache.org/jira/browse/AXIS2C-1486?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel \
]

Mirko Wischer updated AXIS2C-1486:
----------------------------------

    Priority: Minor  (was: Major)

> ADB generated code generates memory leak
> ----------------------------------------
> 
> Key: AXIS2C-1486
> URL: https://issues.apache.org/jira/browse/AXIS2C-1486
> Project: Axis2-C
> Issue Type: Bug
> Components: code generation
> Affects Versions: 1.4.1
> Environment: Win32
> Reporter: Mirko Wischer
> Priority: Minor
> Attachments: CADBBeanTemplateSource_trunk.xsl.diff
> 
> 
> The xslt used for transforming to C Code (CADBBeanTemplateSource.xsl) creates a \
> memory leak: Code created for adb_..._serialize(..) contains a memory leak for the \
>                 following types:
> - axutil_date_time_t
> - axutil_base64_binary_t
> The XSLT uses axutil_base64_binary_get_encoded_binary() and \
> axutil_date_time_serialize_date_time() for creating textual representations. After \
> writing these texts to the stream the start and end tags are freed but the the \
> generated code never releases the base64 text or data_time text: Look at the \
> follown snippet from the trunk version of the XSLT: (starting at Line: 4435)
> <xsl:when test="$nativePropertyType='axutil_date_time_t*'">
> text_value_<xsl:value-of select="$position"/> = \
> axutil_date_time_serialize_date_time(<xsl:value-of \
> select="$propertyInstanceName"/>, env); <xsl:if test="$anon or $istype">
> axutil_stream_write(stream, env, start_input_str, start_input_str_len);
> </xsl:if>
> axutil_stream_write(stream, env, text_value_<xsl:value-of select="$position"/>, \
>                 axutil_strlen(text_value_<xsl:value-of select="$position"/>));
> ---> MISSING FREE AT THIS POSITION: AXIS2_FREE(env->allocator, \
> text_value_<xsl:value-of select="$position"/>); <xsl:if test="$anon or $istype">
> axutil_stream_write(stream, env, end_input_str, end_input_str_len);
> </xsl:if>
> </xsl:when>
> And starting at line: 4446
> <xsl:when test="$propertyType='axutil_base64_binary_t*'">
> text_value_<xsl:value-of select="$position"/> \
> =axutil_base64_binary_get_encoded_binary(<xsl:value-of \
> select="$propertyInstanceName"/>, env); <xsl:if test="$anon or $istype">
> axutil_stream_write(stream, env, start_input_str, start_input_str_len);
> </xsl:if>
> axutil_stream_write(stream, env, text_value_<xsl:value-of select="$position"/>, \
> axutil_strlen(text_value_<xsl:value-of select="$position"/>)); ----> MISSING FREE \
> HERE: AXIS2_FREE(env->allocator, text_value_<xsl:value-of select="$position"/>); \
> <xsl:if test="$anon or $istype"> axutil_stream_write(stream, env, end_input_str, \
> end_input_str_len); </xsl:if>
> </xsl:when>
> Patch for trunk is included as DIFF file ( this also contains a fix for compiling \
> with Visual Studio C Compiler)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic