[UTC][sensor][Non-ACR]Fix resource leak 20/263720/1
authorYunmi Ha <yunmi.ha@samsung.com>
Wed, 8 Sep 2021 08:28:29 +0000 (17:28 +0900)
committerHyotaek Shim <hyotaek.shim@samsung.com>
Thu, 9 Sep 2021 00:18:11 +0000 (00:18 +0000)
On several negative tests for sensor,
there are missing that call function for destroying variable.

Change-Id: I33df91e1d736de3d057a125ce2d6ee5b3ecc6aa0
Signed-off-by: Yunmi Ha <yunmi.ha@samsung.com>
src/utc/sensor/utc-system-sensor-basic.c

index 855f488a61219fdfae4bef06300080f3921c50ae..1999a5330be80fb329c620c42d4ebf98ece48702 100755 (executable)
@@ -741,7 +741,7 @@ int utc_system_sensor_listener_set_event_cb_n(void)
 
        assert_eq(error, SENSOR_ERROR_INVALID_PARAMETER);
 
-       sensor_listener_h listener;
+       sensor_listener_h listener, wrong_listener;
        sensor_h sensor;
 
        if (!sensor_supported[SENSOR_ACCELEROMETER])
@@ -751,12 +751,13 @@ int utc_system_sensor_listener_set_event_cb_n(void)
 
        sensor_create_listener(sensor, &listener);
 
-       *(int *)listener = WRONG_HANDLE;
-
-       error = sensor_listener_set_event_cb(listener, INTERVAL, test_callback, TEST_STRING);
+       *(int *)wrong_listener = WRONG_HANDLE;
 
+       error = sensor_listener_set_event_cb(wrong_listener, INTERVAL, test_callback, TEST_STRING);
        assert_eq(error, SENSOR_ERROR_OPERATION_FAILED);
 
+       sensor_destroy_listener(listener);
+
        return 0;
 }
 
@@ -885,7 +886,7 @@ int utc_system_sensor_listener_set_events_cb_n(void)
 
        assert_eq(error, SENSOR_ERROR_INVALID_PARAMETER);
 
-       sensor_listener_h listener;
+       sensor_listener_h listener, wrong_listener;
        sensor_h sensor;
 
        if (!sensor_supported[SENSOR_ACCELEROMETER])
@@ -895,12 +896,13 @@ int utc_system_sensor_listener_set_events_cb_n(void)
 
        sensor_create_listener(sensor, &listener);
 
-       *(int *)listener = WRONG_HANDLE;
-
-       error = sensor_listener_set_events_cb(listener, test_events_callback, TEST_STRING);
+       *(int *)wrong_listener = WRONG_HANDLE;
 
+       error = sensor_listener_set_events_cb(wrong_listener, test_events_callback, TEST_STRING);
        assert_eq(error, SENSOR_ERROR_OPERATION_FAILED);
 
+       sensor_destroy_listener(listener);
+
        return 0;
 }
 
@@ -1022,7 +1024,7 @@ int utc_system_sensor_listener_set_accuracy_cb_n(void)
 
        assert_eq(error, SENSOR_ERROR_INVALID_PARAMETER);
 
-       sensor_listener_h listener;
+       sensor_listener_h listener, wrong_listener;
        sensor_h sensor;
 
        if (!sensor_supported[SENSOR_ACCELEROMETER])
@@ -1032,12 +1034,13 @@ int utc_system_sensor_listener_set_accuracy_cb_n(void)
 
        sensor_create_listener(sensor, &listener);
 
-       *(int *)listener = WRONG_HANDLE;
-
-       error = sensor_listener_set_accuracy_cb(listener, test_callback, TEST_STRING);
+       *(int *)wrong_listener = WRONG_HANDLE;
 
+       error = sensor_listener_set_accuracy_cb(wrong_listener, test_callback, TEST_STRING);
        assert_eq(error, SENSOR_ERROR_OPERATION_FAILED);
 
+       sensor_destroy_listener(listener);
+
        return 0;
 }
 
@@ -2001,6 +2004,8 @@ int utc_system_sensor_listener_set_option_p(void)
 
                        error = sensor_listener_stop(listener);
                        assert_eq(error, SENSOR_ERROR_NONE);
+
+                       sensor_destroy_listener(listener);
                }
        }