From dc9e64a2344adefa32901b4dd6d935559fa200ea Mon Sep 17 00:00:00 2001 From: Ghislain MARY Date: Thu, 3 Aug 2017 15:49:14 +0200 Subject: [PATCH] Fix memory leak in external body url message test. --- tester/liblinphone_tester.h | 1 + tester/message_tester.c | 6 +++++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/tester/liblinphone_tester.h b/tester/liblinphone_tester.h index 3e77b518d..95c034e24 100644 --- a/tester/liblinphone_tester.h +++ b/tester/liblinphone_tester.h @@ -168,6 +168,7 @@ typedef struct _stats { int number_of_LinphoneMessageDelivered; int number_of_LinphoneMessageNotDelivered; int number_of_LinphoneMessageFileTransferDone; + int number_of_LinphoneMessageFileTransferError; int number_of_LinphoneMessageDeliveredToUser; int number_of_LinphoneMessageDisplayed; int number_of_LinphoneIsComposingActiveReceived; diff --git a/tester/message_tester.c b/tester/message_tester.c index cece23af6..c7a7156cb 100644 --- a/tester/message_tester.c +++ b/tester/message_tester.c @@ -190,6 +190,7 @@ void liblinphone_tester_chat_message_msg_state_changed(LinphoneChatMessage *msg, return; case LinphoneChatMessageStateFileTransferError: counters->number_of_LinphoneMessageNotDelivered++; + counters->number_of_LinphoneMessageFileTransferError++; return; case LinphoneChatMessageStateFileTransferDone: counters->number_of_LinphoneMessageFileTransferDone++; @@ -698,10 +699,13 @@ static void file_transfer_using_external_body_url(void) { BC_ASSERT_TRUE(wait_for(pauline->lc, marie->lc, &marie->stat.number_of_LinphoneMessageReceived, 1)); if (marie->stat.last_received_chat_message) { + cbs = linphone_chat_message_get_callbacks(marie->stat.last_received_chat_message); + linphone_chat_message_cbs_set_msg_state_changed(cbs, liblinphone_tester_chat_message_msg_state_changed); linphone_chat_message_download_file(marie->stat.last_received_chat_message); } BC_ASSERT_TRUE(wait_for(pauline->lc, marie->lc, &marie->stat.number_of_LinphoneMessageExtBodyReceived, 1)); - BC_ASSERT_TRUE(wait_for(pauline->lc, marie->lc, &pauline->stat.number_of_LinphoneMessageInProgress, 1)); + BC_ASSERT_TRUE(wait_for(pauline->lc, marie->lc, &pauline->stat.number_of_LinphoneMessageDelivered, 1)); + BC_ASSERT_TRUE(wait_for(pauline->lc, marie->lc, &marie->stat.number_of_LinphoneMessageFileTransferError, 1)); linphone_core_manager_destroy(pauline); linphone_core_manager_destroy(marie); } -- GitLab