Hi Vu,

ACK

-AVM


On 3/6/2017 5:41 PM, Vu Minh Nguyen wrote:
>   src/log/logd/lgs_dest.cc |  4 ++++
>   1 files changed, 4 insertions(+), 0 deletions(-)
>
>
> DestinationHandler::UpdateRtDestStatus() uses lgs_cfgupd_mutival_replace()
> to update logRecordDestinationStatus but not freeing buffer
> at the end of the function.
>
> This fix does freeing that buffer.
>
> diff --git a/src/log/logd/lgs_dest.cc b/src/log/logd/lgs_dest.cc
> --- a/src/log/logd/lgs_dest.cc
> +++ b/src/log/logd/lgs_dest.cc
> @@ -184,6 +184,10 @@ void DestinationHandler::UpdateRtDestSta
>     if (ret == -1) {
>       LOG_WA("%s lgs_cfg_update Fail", __func__);
>     }
> +
> +  // Free memory allocated for the config_data buffer
> +  if (config_data.ckpt_buffer_ptr != nullptr)
> +    free(config_data.ckpt_buffer_ptr);
>   }
>   
>   void DestinationHandler::FormCfgDestMsg(


------------------------------------------------------------------------------
Announcing the Oxford Dictionaries API! The API offers world-renowned
dictionary content that is easy and intuitive to access. Sign up for an
account today to start using our lexical data to power your apps and
projects. Get started today and enter our developer competition.
http://sdm.link/oxford
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to