Fix resource leak. db connection handle was not released. 80/144580/1
authorHaejeong Kim <backto.kim@samsung.com>
Thu, 17 Aug 2017 07:27:54 +0000 (16:27 +0900)
committerHaejeong Kim <backto.kim@samsung.com>
Thu, 17 Aug 2017 07:27:54 +0000 (16:27 +0900)
Change-Id: Ic10ef7dfef61e963aef18213b353025f21ea0901

src/dcm_svc_internal.c

index cd9b8c2278379464996c320ddb296fbabb1e8ac2..b399347c58325da947f2970e8e27427924166896 100755 (executable)
@@ -538,8 +538,11 @@ int dcm_scan_single(const char *file_path, uid_t uid, int *face_count)
        dcm_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "Fail dcm_svc_db_connect");
 
        ret = dcm_svc_db_get_scan_image_info_by_path(file_path, &scan_item);
-       dcm_retvm_if(ret != MS_MEDIA_ERR_NONE, ret, "Fail dcm_svc_db_get_scan_image_info_by_path");
-       dcm_retvm_if(scan_item == NULL, ret, "scan_item is NULL");
+       if (ret != MS_MEDIA_ERR_NONE || scan_item == NULL) {
+               dcm_error("Failed to dcm_svc_db_get_scan_image_info_by_path err: %d", ret);
+               dcm_svc_db_disconnect();
+               return ret;
+       }
 
        ret = __dcm_scan_process(scan_item);
        if (ret != MS_MEDIA_ERR_NONE) {