From: Dewal Agarwal <d1.agarwal@samsung.com>
Date: Thu, 29 Oct 2020 08:33:03 +0000 (+0530)
Subject: [Non-ACR] Coverity Fixes in Test App
X-Git-Tag: accepted/tizen/unified/20240924.153234~3
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=100383e5b48d9dd8f1ad38dea624ed2b2c4ae554;p=platform%2Fcore%2Fapi%2Fbatterymonitor.git

[Non-ACR] Coverity Fixes in Test App
- Resource Leak
- Unused Code

Change-Id: Icd75864f32cc539482dce236f96525c5cb5b2444
---

diff --git a/test/test_cli_app/src/bmt_usage.c b/test/test_cli_app/src/bmt_usage.c
index 08f6750..ea1d8d5 100644
--- a/test/test_cli_app/src/bmt_usage.c
+++ b/test/test_cli_app/src/bmt_usage.c
@@ -211,9 +211,6 @@ static gboolean bmt_testapp_get_total_power_usage_by_app_id_for_all_resource_id_
 			case BATTERY_MONITOR_RESOURCE_ID_GPS_SENSOR:
 				testapp_print ("Resource ID : GPS App Usage Value = [%lf] \n", battery_usage);
 				break;
-			default:
-				testapp_print ("Default Case");
-				break;
 			}
 		}
 	}
@@ -389,9 +386,6 @@ static gboolean bmt_testapp_get_power_usage_for_all_resource_id()
 		case BATTERY_MONITOR_RESOURCE_ID_GPS_SENSOR:
 			testapp_print ("Resource ID : GPS Usage Value = [%lf] \n", battery_usage);
 			break;
-		default:
-			testapp_print ("Default Case \n");
-			break;
 		}
 
 	}
@@ -449,9 +443,6 @@ bool get_app_id_all_resource_cb(app_info_h app_info, void *user_data)
 			case BATTERY_MONITOR_RESOURCE_ID_GPS_SENSOR:
 				testapp_print ("Resource ID : GPS App Usage Value = [%lf] \n", battery_usage);
 				break;
-			default:
-				testapp_print ("Default Case");
-				break;
 			}
 		}
 		battery_monitor_battery_usage_data_destroy(data_handle);
@@ -515,8 +506,10 @@ bool bmt_testapp_get_power_usage_automated()
 			start_time = s_t;
 			end_time = e_t;
 		}
-		else
+		else {
+			free(user_data);
 			return FALSE;
+		}
 
 		testapp_print("\n Start Time: %ld \n", start_time);
 		testapp_print("\n End Time: %ld \n", end_time);
@@ -557,9 +550,6 @@ bool bmt_testapp_get_power_usage_automated()
 			case BATTERY_MONITOR_RESOURCE_ID_GPS_SENSOR:
 				testapp_print ("Resource ID : GPS Usage Value = [%lf] \n", battery_usage);
 				break;
-			default:
-				testapp_print ("Default Case \n");
-				break;
 			}
 		}
 		i++;