From: Pawel Andruszkiewicz
Date: Mon, 7 Mar 2016 14:03:52 +0000 (+0100)
Subject: [Download] Make sure that the pointer is not null.
X-Git-Tag: submit/tizen/20160308.045434^2~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=f5e1dd32f53c9c6435156a21eb9ce1a0898d2564;p=platform%2Fcore%2Fapi%2Fwebapi-plugins.git
[Download] Make sure that the pointer is not null.
[Verification] TCT pass rate: 100% (62/62/0/0/0).
Change-Id: I6cbcb7f95d5b73d296034dbce6fcd112d30ae2a5
Signed-off-by: Pawel Andruszkiewicz
---
diff --git a/src/download/download_instance.cc b/src/download/download_instance.cc
index 964d9437..97ff27c0 100755
--- a/src/download/download_instance.cc
+++ b/src/download/download_instance.cc
@@ -67,21 +67,25 @@ DownloadInstance::~DownloadInstance() {
DownloadInfoPtr diPtr = it->second->instance->diMap[it->second->callbackId];
SLoggerD("~DownloadInstance() for callbackID %d Called", it->second->callbackId);
- ret = download_unset_state_changed_cb(diPtr->download_id);
- if (ret != DOWNLOAD_ERROR_NONE)
- LoggerE("download_unset_state_changed_cb() is failed. (%s)", get_error_message (ret));
-
- ret = download_unset_progress_cb(diPtr->download_id);
- if (ret != DOWNLOAD_ERROR_NONE)
- LoggerE("download_unset_progress_cb() is failed. (%s)", get_error_message (ret));
-
- ret = download_cancel(diPtr->download_id);
- if (ret != DOWNLOAD_ERROR_NONE)
- LoggerE("download_cancel() is failed. (%s)", get_error_message (ret));
-
- ret = download_destroy(diPtr->download_id);
- if (ret != DOWNLOAD_ERROR_NONE)
- LoggerE("download_destroy() is failed. (%s)", get_error_message (ret));
+ if (diPtr) {
+ ret = download_unset_state_changed_cb(diPtr->download_id);
+ if (ret != DOWNLOAD_ERROR_NONE)
+ LoggerE("download_unset_state_changed_cb() is failed. (%s)", get_error_message (ret));
+
+ ret = download_unset_progress_cb(diPtr->download_id);
+ if (ret != DOWNLOAD_ERROR_NONE)
+ LoggerE("download_unset_progress_cb() is failed. (%s)", get_error_message (ret));
+
+ ret = download_cancel(diPtr->download_id);
+ if (ret != DOWNLOAD_ERROR_NONE)
+ LoggerE("download_cancel() is failed. (%s)", get_error_message (ret));
+
+ ret = download_destroy(diPtr->download_id);
+ if (ret != DOWNLOAD_ERROR_NONE)
+ LoggerE("download_destroy() is failed. (%s)", get_error_message (ret));
+ } else {
+ LoggerD("diPtr is nullptr");
+ }
delete (it->second);
}