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