HI Vu,

ACK

-AVM


On 8/16/2016 3:39 PM, Vu Minh Nguyen wrote:
>   osaf/services/saf/logsv/lgs/lgs_evt.cc |  4 +++-
>   1 files changed, 3 insertions(+), 1 deletions(-)
>
>
> Assertion failed in log_stream_close() because the caller did not check NULL
> before calling the function.
>
> This patch adds the protection.
>
> diff --git a/osaf/services/saf/logsv/lgs/lgs_evt.cc 
> b/osaf/services/saf/logsv/lgs/lgs_evt.cc
> --- a/osaf/services/saf/logsv/lgs/lgs_evt.cc
> +++ b/osaf/services/saf/logsv/lgs/lgs_evt.cc
> @@ -166,7 +166,9 @@ int lgs_client_delete(uint32_t client_id
>               lgs_stream_list_t *tmp_rec;
>               log_stream_t *stream = log_stream_get_by_id(cur_rec->stream_id);
>               TRACE_4("client_id: %u, REMOVE stream id: %u", 
> client->client_id, cur_rec->stream_id);
> -             log_stream_close(&stream, &closetime);
> +             if (stream != NULL) {
> +                     log_stream_close(&stream, &closetime);
> +             }
>               tmp_rec = cur_rec->next;
>               free(cur_rec);
>               cur_rec = tmp_rec;


------------------------------------------------------------------------------
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to