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