Fix SVACE issue 21/53121/4
authorJiwan Kim <ji-wan.kim@samsung.com>
Wed, 2 Dec 2015 03:04:55 +0000 (12:04 +0900)
committerJiwan Kim <ji-wan.kim@samsung.com>
Wed, 2 Dec 2015 08:50:19 +0000 (17:50 +0900)
Change-Id: Ie0344cba03b8414cec41b062c0a132bf32e11a35

13 files changed:
packaging/tel-plugin-dbus_tapi.spec
src/dtapi_call.c
src/dtapi_modem.c
src/dtapi_network.c
src/dtapi_oem.c
src/dtapi_phonebook.c
src/dtapi_sap.c
src/dtapi_sat.c
src/dtapi_sat_manager.c
src/dtapi_sim.c
src/dtapi_sms.c
src/dtapi_ss.c
src/sat_ui_support/sat_ui_support.c

index 38c34f4..5b89d0b 100644 (file)
@@ -1,6 +1,6 @@
 %define major 0
 %define minor 3
-%define patchlevel 64
+%define patchlevel 65
 
 Name:           tel-plugin-dbus_tapi
 Version:        %{major}.%{minor}.%{patchlevel}
index 330125d..144a43e 100644 (file)
@@ -39,7 +39,7 @@ static gboolean on_call_dial(TelephonyCall *call,
 {
        struct treq_call_dial req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        memset(&req, 0x0, sizeof(req));
 
@@ -69,7 +69,7 @@ static gboolean on_call_answer(TelephonyCall *call,
 {
        struct treq_call_answer req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -99,7 +99,7 @@ static gboolean on_call_end(TelephonyCall *call,
 {
        struct treq_call_end req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -128,7 +128,7 @@ static gboolean on_call_start_cont_dtmf(TelephonyCall *call,
 {
        struct treq_call_start_cont_dtmf  req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -151,7 +151,7 @@ static gboolean on_call_stop_cont_dtmf(TelephonyCall *call,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -172,7 +172,7 @@ static gboolean on_call_send_burst_dtmf(TelephonyCall *call,
 {
        struct treq_call_send_burst_dtmf req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -210,7 +210,7 @@ static gboolean on_call_active(TelephonyCall *call,
 {
        struct treq_call_active req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -233,7 +233,7 @@ static gboolean on_call_hold(TelephonyCall *call,
 {
        struct treq_call_hold req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -256,7 +256,7 @@ static gboolean on_call_swap(TelephonyCall *call,
 {
        struct treq_call_swap req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -279,7 +279,7 @@ static gboolean on_call_join(TelephonyCall *call,
 {
        struct treq_call_join req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -302,7 +302,7 @@ static gboolean on_call_split(TelephonyCall *call,
 {
        struct treq_call_split req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -325,7 +325,7 @@ static gboolean on_call_transfer(TelephonyCall *call,
 {
        struct treq_call_transfer req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -348,7 +348,7 @@ static gboolean on_call_deflect(TelephonyCall *call,
 {
        struct treq_call_deflect req = {0};
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -371,7 +371,7 @@ static gboolean on_call_get_privacy_mode(TelephonyCall *call,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "r"))
                return TRUE;
@@ -390,7 +390,7 @@ static gboolean on_call_set_privacy_mode(TelephonyCall *call,
 {
        struct treq_call_set_voice_privacy_mode req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -415,7 +415,7 @@ static gboolean on_call_get_status(TelephonyCall *call,
        TcorePlugin *plugin = 0;
        CoreObject *call_co = NULL;
        CallObject *call_obj = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        gchar call_number[MAX_CALL_NUMBER_LEN];
        gint call_type;
@@ -470,7 +470,7 @@ static gboolean on_call_get_status_all(TelephonyCall *call,
 
        GVariant *gv = NULL;
        GVariantBuilder b;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        gint call_id;
        gint handle;
@@ -585,7 +585,7 @@ static gboolean on_call_set_sound_path(TelephonyCall *call,
 {
        struct treq_call_set_sound_path req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -611,7 +611,7 @@ static gboolean on_call_get_sound_volume_level(TelephonyCall *call,
        struct treq_call_get_sound_volume_level req;
        struct custom_data *ctx = user_data;
        TReturn ret;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "r"))
                return TRUE;
@@ -650,7 +650,7 @@ static gboolean on_call_set_sound_volume_level(TelephonyCall *call,
 {
        struct treq_call_set_sound_volume_level req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -675,7 +675,7 @@ static gboolean on_call_get_sound_mute_status(TelephonyCall *call,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "r"))
                return TRUE;
@@ -694,7 +694,7 @@ static gboolean on_call_set_sound_mute_status(TelephonyCall *call,
 {
        struct treq_call_set_sound_mute_status req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -722,7 +722,7 @@ static gboolean on_call_set_sound_recording(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_set_sound_recording req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -747,7 +747,7 @@ static gboolean on_call_set_sound_equalization(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_set_sound_equalization req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -774,7 +774,7 @@ static gboolean on_call_set_sound_noise_reduction(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_set_sound_noise_reduction req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -797,7 +797,7 @@ static gboolean on_call_set_sound_clock_status(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_set_sound_clock_status req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -821,7 +821,7 @@ static gboolean on_call_set_preferred_voice_subscription(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_set_preferred_voice_subscription req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "w"))
                return TRUE;
@@ -847,7 +847,7 @@ static gboolean on_call_get_preferred_voice_subscription(TelephonyCall *call,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "r"))
                return TRUE;
@@ -866,7 +866,7 @@ static gboolean on_call_modify(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_modify req = {0};
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -894,7 +894,7 @@ static gboolean on_call_confirm_modify(TelephonyCall *call,
 {
        struct custom_data *ctx = user_data;
        struct treq_call_confirm_modify req = {0};
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_CALL, "x"))
                return TRUE;
@@ -1056,7 +1056,7 @@ gboolean dbus_plugin_call_response(struct custom_data *ctx,
        UserRequest *ur, struct dbus_request_info *dbus_info,
        enum tcore_response_command command, unsigned int data_len, const void *data)
 {
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_CALL_DIAL: {
index d4b7181..13290e7 100644 (file)
@@ -36,7 +36,7 @@ static gboolean on_modem_set_power(TelephonyModem *modem,
 {
        struct custom_data *ctx = user_data;
        enum tcore_request_command command;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "x"))
                return TRUE;
@@ -76,7 +76,7 @@ static gboolean on_modem_set_flight_mode(TelephonyModem *modem,
 {
        struct treq_modem_set_flightmode req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "w"))
                return TRUE;
@@ -95,7 +95,7 @@ static gboolean on_modem_get_flight_mode(TelephonyModem *modem,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "r"))
                return TRUE;
@@ -112,7 +112,7 @@ static gboolean on_modem_get_version(TelephonyModem *modem,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "r"))
                return TRUE;
@@ -129,7 +129,7 @@ static gboolean on_modem_get_serial_number(TelephonyModem *modem,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "r"))
                return TRUE;
@@ -146,7 +146,7 @@ static gboolean on_modem_get_imei(TelephonyModem *modem,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "r"))
                return TRUE;
@@ -165,7 +165,7 @@ static gboolean on_modem_set_dun_pin_ctrl(TelephonyModem *modem,
 {
        struct treq_modem_set_dun_pin_control req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "x"))
                return TRUE;
@@ -185,7 +185,7 @@ static gboolean on_modem_get_device_info(TelephonyModem *modem,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "r"))
                return TRUE;
@@ -256,7 +256,7 @@ gboolean dbus_plugin_modem_response(struct custom_data *ctx,
        UserRequest *ur, struct dbus_request_info *dbus_info,
        enum tcore_response_command command, unsigned int data_len, const void *data)
 {
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_MODEM_SET_FLIGHTMODE: {
index 46d0c10..c749dde 100644 (file)
@@ -462,7 +462,7 @@ static gboolean on_network_search(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "x"))
                return TRUE;
@@ -479,7 +479,7 @@ static gboolean on_network_search_cancel(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "x"))
                return TRUE;
@@ -496,7 +496,7 @@ static gboolean on_network_get_selection_mode(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -515,7 +515,7 @@ static gboolean on_network_set_selection_mode(TelephonyNetwork *network,
 {
        struct treq_network_set_plmn_selection_mode req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -548,7 +548,7 @@ static gboolean on_network_set_service_domain(TelephonyNetwork *network,
 {
        struct treq_network_set_service_domain req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -567,7 +567,7 @@ static gboolean on_network_get_service_domain(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -586,7 +586,7 @@ static gboolean on_network_set_band(TelephonyNetwork *network,
 {
        struct treq_network_set_band req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -606,7 +606,7 @@ static gboolean on_network_get_band(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -624,7 +624,7 @@ static gboolean on_network_set_mode(TelephonyNetwork *network,
 {
        struct treq_network_set_mode req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -643,7 +643,7 @@ static gboolean on_network_get_mode(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -662,7 +662,7 @@ static gboolean on_network_set_preferred_plmn(TelephonyNetwork *network,
 {
        struct treq_network_set_preferred_plmn req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -688,7 +688,7 @@ static gboolean on_network_get_preferred_plmn(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -705,7 +705,7 @@ static gboolean on_network_get_serving_network(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -722,7 +722,7 @@ static gboolean on_network_get_neighboring_cell_info(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -739,7 +739,7 @@ static gboolean on_network_set_default_data_subscription(TelephonyNetwork *netwo
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -756,7 +756,7 @@ static gboolean on_network_get_default_data_subscription(TelephonyNetwork *netwo
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -773,7 +773,7 @@ static gboolean on_network_set_default_subs(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -790,7 +790,7 @@ static gboolean on_network_get_default_subs(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -808,7 +808,7 @@ static gboolean on_network_set_emergency_callback_mode(TelephonyNetwork *network
 {
        struct treq_network_set_emergency_callback_mode req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -828,7 +828,7 @@ static gboolean on_network_set_roaming_preference(TelephonyNetwork *network,
 {
        struct treq_network_set_roaming_preference req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "w"))
                return TRUE;
@@ -847,7 +847,7 @@ static gboolean on_network_get_roaming_preference(TelephonyNetwork *network,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_NETWORK, "r"))
                return TRUE;
@@ -988,7 +988,7 @@ gboolean dbus_plugin_network_response(struct custom_data *ctx,
        UserRequest *ur, struct dbus_request_info *dbus_info,
        enum tcore_response_command command, unsigned int data_len, const void *data)
 {
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_NETWORK_SEARCH: {
index 8c93ac9..45a0440 100644 (file)
@@ -89,7 +89,7 @@ static gboolean send_oem_data(TelephonyOEM *oem,
        gint result = 1;
        guchar *decoded_data = NULL;
        gsize length;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_MODEM, "w"))
                return TRUE;
index 654d688..08eb7f4 100644 (file)
@@ -87,7 +87,7 @@ static gboolean on_phonebook_get_init_status(TelephonyPhonebook *phonebook,
        gboolean pb_status = FALSE;
        struct tel_phonebook_support_list *list = NULL;
        CoreObject *co_pb = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "r"))
                return TRUE;
@@ -144,7 +144,7 @@ static gboolean on_phonebook_get_count(TelephonyPhonebook *phonebook,
        struct treq_phonebook_get_count req;
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "r"))
                return TRUE;
@@ -181,7 +181,7 @@ static gboolean on_phonebook_get_info(TelephonyPhonebook *phonebook,
        struct treq_phonebook_get_info req;
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "r"))
                return TRUE;
@@ -219,7 +219,7 @@ static gboolean on_phonebook_get_usim_info(TelephonyPhonebook *phonebook,
        struct custom_data *ctx = user_data;
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "r"))
                return TRUE;
@@ -259,7 +259,7 @@ static gboolean on_phonebook_read_record(TelephonyPhonebook *phonebook,
        struct treq_phonebook_read_record req;
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "r"))
                return TRUE;
@@ -312,7 +312,7 @@ static gboolean on_phonebook_update_record(TelephonyPhonebook *phonebook,
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
        int temp_len = 0;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "x"))
                return TRUE;
@@ -423,7 +423,7 @@ static gboolean on_phonebook_delete_record(TelephonyPhonebook *phonebook,
        struct treq_phonebook_delete_record req;
        CoreObject *co_pb = NULL;
        gboolean pb_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_PHONEBOOK, "x"))
                return TRUE;
@@ -511,7 +511,7 @@ gboolean dbus_plugin_phonebook_response(struct custom_data *ctx,
        const struct tresp_phonebook_read_record *resp_pbread = data;
        const struct tresp_phonebook_update_record *resp_pbupdate = data;
        const struct tresp_phonebook_delete_record *resp_pbdelete = data;
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_PHONEBOOK_GETCOUNT:
index ab4f2e4..4f83d62 100644 (file)
@@ -33,7 +33,7 @@ static gboolean on_sap_connect(TelephonySap *sap,
 {
        struct custom_data *ctx = user_data;
        struct treq_sap_req_connect req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "x"))
                return TRUE;
@@ -54,7 +54,7 @@ static gboolean on_sap_disconnect(TelephonySap *sap,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "x"))
                return TRUE;
@@ -71,7 +71,7 @@ static gboolean on_sap_get_status(TelephonySap *sap,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "r"))
                return TRUE;
@@ -88,7 +88,7 @@ static gboolean on_sap_get_atr(TelephonySap *sap,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "r"))
                return TRUE;
@@ -110,7 +110,7 @@ static gboolean on_sap_transfer_apdu(TelephonySap *sap,
        GVariant *inner_gv = NULL;
        guchar rt_i;
        int i = 0;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        dbg("Func Entrance");
 
@@ -148,7 +148,7 @@ static gboolean on_sap_set_protocol(TelephonySap *sap,
 {
        struct custom_data *ctx = user_data;
        struct treq_sap_set_protocol req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "w"))
                return TRUE;
@@ -170,7 +170,7 @@ static gboolean on_sap_set_power(TelephonySap *sap,
 {
        struct custom_data *ctx = user_data;
        struct treq_sap_set_power req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "w"))
                return TRUE;
@@ -191,7 +191,7 @@ static gboolean on_sap_get_card_reader_status(TelephonySap *sap,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAP, "r"))
                return TRUE;
index cfa1032..2ccba8a 100644 (file)
@@ -106,7 +106,7 @@ static gboolean on_sat_get_main_menu_info(TelephonySAT *sat,
        GVariant *icon_id = NULL;
        GVariant *icon_list = NULL;
 #endif
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAT, "r"))
                return TRUE;
@@ -149,7 +149,7 @@ static gboolean on_sat_send_display_status(TelephonySAT *sat,
        struct custom_data *ctx = user_data;
        gboolean result = FALSE;
        gint out_param = 0;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAT, "x"))
                return TRUE;
@@ -185,7 +185,7 @@ static gboolean on_sat_send_user_confirm(TelephonySAT *sat,
        TcorePlugin *plg = NULL;
        char *cp_name;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        gboolean result = FALSE;
        gint out_param = 0;
@@ -228,7 +228,7 @@ static gboolean on_sat_send_app_exec_result(TelephonySAT *sat,
        TcorePlugin *plg = NULL;
        char *cp_name;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        gboolean result = FALSE;
        gint out_param = 0;
@@ -266,7 +266,7 @@ static gboolean on_sat_select_menu(TelephonySAT *sat,
 {
        struct custom_data *ctx = user_data;
        struct treq_sat_envelop_cmd_data req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAT, "x"))
                return TRUE;
@@ -296,7 +296,7 @@ static gboolean on_sat_download_event(TelephonySAT *sat,
        struct custom_data *ctx = user_data;
 
        struct treq_sat_envelop_cmd_data req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SAT, "x"))
                return TRUE;
index a7fae83..8e2e264 100644 (file)
@@ -53,7 +53,7 @@
 #define SAT_TIME_OUT 30000
 #define TIZEN_SAT_DELAY_TO_CLEAN_MSG 15000
 #define SAT_USC2_INPUT_LEN_MAX 70
-#define SAT_EVENT_DOWNLOAD_MAX 9
+#define SAT_EVENT_DOWNLOAD_MAX 17
 
 #define LANGUAGE_XML_PATH "/opt/usr/apps/org.tizen.setting/data/langlist.xml"
 
@@ -64,6 +64,8 @@ static gboolean _sat_manager_handle_open_channel_confirm(struct custom_data *ctx
 
 static struct sat_manager_queue_data *sat_queue[SAT_DEF_CMD_Q_MAX] = {NULL, };
 gboolean g_evt_list[SAT_EVENT_DOWNLOAD_MAX] = {0};
+/* Allocate large size structure on non-stack area (data area) */
+static struct sat_manager_queue_data q_data = {0};
 
 static unsigned char _convert_decimal_to_bcd(int dec)
 {
@@ -300,6 +302,7 @@ static gboolean _pop_nth_data(struct custom_data *ctx, struct sat_manager_queue_
        memcpy((void*)cmd_obj, (void*)item, sizeof(struct sat_manager_queue_data));
        dbg("pop data from queue at index[%d],[%p].\n", local_index, item);
        sat_queue[local_index] = NULL;
+       g_free(item->cp_name);
        g_free(item);
        return TRUE;
 }
@@ -507,7 +510,6 @@ GVariant* sat_manager_caching_setup_menu_info(struct custom_data *ctx, const cha
 {
        TcorePlugin *plg = NULL;
        GVariant *setup_menu_info = NULL;
-       struct sat_manager_queue_data q_data;
 
        gushort title_len = 0;
        gint command_id = 0, menu_cnt = 0;
@@ -680,7 +682,6 @@ GVariant* sat_manager_display_text_noti(struct custom_data *ctx, const char *cp_
 {
        TcorePlugin *plg = NULL;
        GVariant *display_text = NULL;
-       struct sat_manager_queue_data q_data;
 
        gushort text_len = 0;
        gint command_id = 0, duration = 0, tmp_duration = 0;
@@ -803,7 +804,6 @@ GVariant* sat_manager_select_item_noti(struct custom_data *ctx, const char *cp_n
 {
        TcorePlugin *plg = NULL;
        GVariant *select_item = NULL;
-       struct sat_manager_queue_data q_data;
 
        gushort text_len = 0;
        gint command_id = 0, default_item_id = 0, menu_cnt = 0;
@@ -939,7 +939,6 @@ GVariant* sat_manager_get_inkey_noti(struct custom_data *ctx, const char *cp_nam
 {
        TcorePlugin *plg = NULL;
        GVariant *get_inkey = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, key_type = 0, input_character_mode = 0;
        gushort text_len = 0;
@@ -1043,7 +1042,6 @@ GVariant* sat_manager_get_input_noti(struct custom_data *ctx, const char *cp_nam
 {
        TcorePlugin *plg = NULL;
        GVariant *get_input = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, input_character_mode = 0;
        gushort text_len = 0, def_text_len = 0;
@@ -1171,7 +1169,6 @@ GVariant* sat_manager_play_tone_noti(struct custom_data *ctx, const char *cp_nam
 {
        TcorePlugin *plg = NULL;
        GVariant *play_tone = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, tone_type = 0, duration = 0, tmp_duration = 0;
        gushort text_len = 0;
@@ -1236,7 +1233,6 @@ GVariant* sat_manager_send_sms_noti(struct custom_data *ctx, const char *cp_name
 {
        TcorePlugin *plg = NULL;
        GVariant *send_sms = NULL;
-       struct sat_manager_queue_data q_data;
 
        int local_index = 0;
        gint command_id = 0, ton = 0, npi = 0, tpdu_type = 0;
@@ -1319,7 +1315,6 @@ GVariant* sat_manager_send_ss_noti(struct custom_data *ctx, const char *cp_name,
 {
        TcorePlugin *plg = NULL;
        GVariant *send_ss = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, ton = 0, npi = 0;
        gushort text_len = 0;
@@ -1418,7 +1413,6 @@ GVariant* sat_manager_send_ussd_noti(struct custom_data *ctx, const char *cp_nam
 {
        TcorePlugin *plg = NULL;
        GVariant *send_ussd = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0;
        guchar dcs = 0;
@@ -1519,7 +1513,6 @@ GVariant* sat_manager_setup_call_noti(struct custom_data *ctx, const char *cp_na
 {
        TcorePlugin *plg = NULL;
        GVariant *setup_call = NULL;
-       struct sat_manager_queue_data q_data;
        struct treq_sat_terminal_rsp_data tr;
 
        gushort text_len = 0, confirm_text_len = 0;
@@ -1703,7 +1696,6 @@ GVariant* sat_manager_setup_idle_mode_text_noti(struct custom_data *ctx, const c
 {
        TcorePlugin *plg = NULL;
        GVariant *idle_mode = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0;
        gushort text_len = 0;
@@ -1785,7 +1777,6 @@ GVariant* sat_manager_open_channel_noti(struct custom_data *ctx, const char *cp_
        enum telephony_network_access_technology result = 0;
 
        GVariant *open_channel = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, bearer_type = 0, protocol_type = 0, dest_addr_type = 0;
        gboolean immediate_link = FALSE, auto_reconnection = FALSE, bg_mode = FALSE;
@@ -2089,7 +2080,6 @@ GVariant* sat_manager_close_channel_noti(struct custom_data *ctx, const char *cp
 {
        TcorePlugin *plg = NULL;
        GVariant *close_channel = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, channel_id = 0;
        gushort text_len = 0;
@@ -2147,7 +2137,6 @@ GVariant* sat_manager_receive_data_noti(struct custom_data *ctx, const char *cp_
 {
        TcorePlugin *plg = NULL;
        GVariant *receive_data = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0, channel_id = 0;
        gushort text_len = 0;
@@ -2208,7 +2197,6 @@ GVariant* sat_manager_send_data_noti(struct custom_data *ctx, const char *cp_nam
 {
        TcorePlugin *plg = NULL;
        GVariant *send_data = NULL;
-       struct sat_manager_queue_data q_data;
 
        int local_index = 0;
        gint command_id = 0, channel_id = 0, data_len = 0;
@@ -2282,7 +2270,6 @@ GVariant* sat_manager_get_channel_status_noti(struct custom_data *ctx, const cha
 {
        TcorePlugin *plg = NULL;
        GVariant *get_channel_status = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0;
 
@@ -2312,9 +2299,6 @@ GVariant* sat_manager_refresh_noti(struct custom_data *ctx, const char *cp_name,
 {
        TcorePlugin *plg = NULL;
        GVariant *refresh = NULL;
-#if !defined(TIZEN_SUPPORT_STK_HIDE_ALPHA_ID)
-       struct sat_manager_queue_data q_data;
-#endif
 
        gint command_id = 0;
        gint refresh_type = 0;
@@ -2397,7 +2381,6 @@ GVariant* sat_manager_send_dtmf_noti(struct custom_data *ctx, const char *cp_nam
        GSList* call_active_list = NULL;
 
        GVariant *send_dtmf = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0;
        gushort text_len = 0;
@@ -2497,7 +2480,6 @@ GVariant* sat_manager_launch_browser_noti(struct custom_data *ctx, const char *c
 {
        TcorePlugin *plg = NULL;
        GVariant *launch_browser = NULL;
-       struct sat_manager_queue_data q_data;
 
 #if GCF_SAT_BROWSER_WITH_SINGLE_SESSION
        gboolean b_app_running = FALSE;
@@ -2773,7 +2755,6 @@ GVariant* sat_manager_language_notification_noti(struct custom_data *ctx, const
 {
        TcorePlugin *plg = NULL;
        GVariant *language_noti = NULL;
-       struct sat_manager_queue_data q_data;
 
        gint command_id = 0;
        gint language = 0;
@@ -2946,7 +2927,6 @@ static gboolean _sat_manager_handle_setup_menu_result(struct custom_data *ctx, T
 
        gint resp;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3028,7 +3008,6 @@ static gboolean _sat_manager_handle_display_text_result(struct custom_data *ctx,
 
        gint resp, me_problem;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3104,7 +3083,6 @@ static gboolean _sat_manager_handle_play_tone_result(struct custom_data *ctx, Tc
 
        gint resp;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3184,7 +3162,6 @@ static gboolean _sat_manager_handle_send_sms_result(struct custom_data *ctx, Tco
 
        gint resp;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3293,7 +3270,6 @@ static gboolean _sat_manager_handle_send_ss_result(struct custom_data *ctx, Tcor
        gint resp, me_problem, ss_cause, call_ctrl_problem, ss_str_len;
        GVariant *ss_str = NULL;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
        /* call ctrl action, result data object, text */
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
@@ -3435,7 +3411,6 @@ static gboolean _sat_manager_handle_send_ussd_result(struct custom_data *ctx, Tc
        gint resp, me_problem, ss_cause, ussd_str_len;
        GVariant *ussd_str = NULL;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
        /* call ctrl action, result data object, text, result2, text2 */
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
@@ -3660,7 +3635,6 @@ static gboolean _sat_manager_handle_setup_call_result(struct custom_data *ctx, T
 
        gint resp, me_problem, cc_problem, call_cause;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3761,7 +3735,6 @@ static gboolean _sat_manager_handle_setup_idle_mode_text_result(struct custom_da
 
        gint resp;
        struct treq_sat_terminal_rsp_data tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
        memset(&tr, 0, sizeof(struct treq_sat_terminal_rsp_data));
@@ -3823,7 +3796,6 @@ static gboolean sat_manager_handle_open_channel_result(struct custom_data *ctx,
        GVariant *desc_tmp, *bearer_desc;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -3960,7 +3932,6 @@ static gboolean sat_manager_handle_close_channel_result(struct custom_data *ctx,
        gint resp, me_problem, bip_problem;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4036,7 +4007,6 @@ static gboolean sat_manager_handle_receive_data_result(struct custom_data *ctx,
        GVariant *received_data;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4130,7 +4100,6 @@ static gboolean sat_manager_handle_send_data_result(struct custom_data *ctx, Tco
        gint data_len;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4206,7 +4175,6 @@ static gboolean sat_manager_handle_get_channel_status_result(struct custom_data
        gint channel_id, channel_status, channel_status_info;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4283,7 +4251,6 @@ static gboolean sat_manager_handle_send_dtmf_result(struct custom_data *ctx, Tco
 
        gint resp;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        tr = (struct treq_sat_terminal_rsp_data *)calloc(1, sizeof(struct treq_sat_terminal_rsp_data));
        if (!tr)
@@ -4354,7 +4321,6 @@ static gboolean sat_manager_handle_launch_browser_result(struct custom_data *ctx
 
        gint resp, browser_problem;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4510,7 +4476,6 @@ static gboolean _sat_manager_handle_menu_select_confirm(struct custom_data *ctx,
 
        gint item_id = 0;
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4611,7 +4576,6 @@ static gboolean _sat_manager_handle_display_text_confirm(struct custom_data *ctx
        gboolean result = FALSE;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -4688,7 +4652,6 @@ static gboolean _sat_manager_handle_get_inkey_confirm(struct custom_data *ctx, T
        gint inkey_data_len = 0;
        gchar inkey_data[SAT_TEXT_STRING_LEN_MAX];
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
        memset(inkey_data, 0, SAT_TEXT_STRING_LEN_MAX);
@@ -4842,7 +4805,6 @@ static gboolean _sat_manager_handle_get_input_confirm(struct custom_data *ctx, T
        gint input_data_len = 0;
        gchar input_data[SAT_TEXT_STRING_LEN_MAX];
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
        memset(input_data, 0, SAT_TEXT_STRING_LEN_MAX);
@@ -4982,7 +4944,6 @@ static gboolean _sat_manager_handle_setup_call_confirm(struct custom_data *ctx,
        gint input_data_len = 0;
        gchar input_data[SAT_TEXT_STRING_LEN_MAX];
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
        memset(input_data, 0, SAT_TEXT_STRING_LEN_MAX);
@@ -5123,7 +5084,6 @@ static gboolean _sat_manager_handle_send_dtmf_confirm(struct custom_data *ctx, T
        gboolean result = FALSE;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -5177,7 +5137,6 @@ static gboolean _sat_manager_handle_launch_browser_confirm(struct custom_data *c
        gboolean result = FALSE;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -5296,7 +5255,6 @@ static gboolean _sat_manager_handle_open_channel_confirm(struct custom_data *ctx
        gboolean result = FALSE;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -5423,7 +5381,6 @@ static gboolean _sat_manager_handle_open_channel_confirm(struct custom_data *ctx
        gboolean result = FALSE;
 
        struct treq_sat_user_confirmation_data *conf_data;
-       struct sat_manager_queue_data q_data;
 
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
 
@@ -5467,11 +5424,12 @@ gboolean sat_manager_handle_user_confirm(struct custom_data *ctx, TcorePlugin *p
 {
        gboolean rv = FALSE;
        gboolean result = FALSE;
-       struct sat_manager_queue_data q_data;
 
        gint command_id, command_type, confirm_type;
        GVariant *additional_data = NULL;
 
+       memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
+
        dbg("user_confirm_data type_format(%s)", g_variant_get_type_string(user_confirm_data));
        g_variant_get(user_confirm_data, "(iiv)", &command_id, &confirm_type, &additional_data);
 
@@ -6079,7 +6037,6 @@ static gboolean _sat_manager_handle_open_channel_ui_display_status(struct custom
 gboolean sat_manager_handle_ui_display_status(struct custom_data *ctx, TcorePlugin *plg, gint command_id, gboolean display_status)
 {
        gboolean result = FALSE;
-       struct sat_manager_queue_data q_data;
 
        dbg("[SAT] ui display status : command id(%d) display status(%d)", command_id, display_status);
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
@@ -6163,7 +6120,6 @@ static gboolean _sat_manager_handle_open_channel_ui_display_status(struct custom
 gboolean sat_manager_handle_ui_display_status(struct custom_data *ctx, TcorePlugin *plg, gint command_id, gboolean display_status)
 {
        gboolean result = FALSE;
-       struct sat_manager_queue_data q_data;
 
        dbg("[SAT] ui display status : command id(%d) display status(%d)", command_id, display_status);
        memset(&q_data, 0, sizeof(struct sat_manager_queue_data));
@@ -6244,6 +6200,10 @@ gboolean sat_manager_handle_event_download_envelop(int event_type,  int src_dev,
        dbg("download data type_format(%s)", g_variant_get_type_string(download_data));
        g_variant_get(download_data, "v", &data);
 
+       if (event_type >= SAT_EVENT_DOWNLOAD_MAX) {
+               err("(%d) event number exceeds max count", event_type);
+               return FALSE;
+       }
 
        if (g_evt_list[event_type] == TRUE) {
                dbg("event (%d) shoud be passed to sim", event_type);
@@ -6327,7 +6287,6 @@ gboolean sat_manager_update_language(struct custom_data *ctx, const char *cp_nam
        gboolean b_specified;
 
        struct treq_sat_terminal_rsp_data *tr;
-       struct sat_manager_queue_data q_data;
 
        s = ctx->server;
        strg = tcore_server_find_storage(s, "vconf");
index 680e309..f099a14 100644 (file)
@@ -395,7 +395,7 @@ static gboolean on_sim_get_init_status(TelephonySim *sim,
        enum tel_sim_status sim_status = SIM_STATUS_UNKNOWN;
        gboolean sim_changed = FALSE;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -420,7 +420,7 @@ static gboolean on_sim_get_card_type(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        enum tel_sim_type sim_type = SIM_TYPE_UNKNOWN;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -449,7 +449,7 @@ static gboolean on_sim_get_imsi(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        struct tel_sim_imsi *imsi;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -479,7 +479,7 @@ static gboolean on_sim_get_ecc(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct tel_sim_ecc_list *ecc_list = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -538,7 +538,7 @@ static gboolean on_sim_get_iccid(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct tel_sim_iccid* iccid = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -571,7 +571,7 @@ static gboolean on_sim_get_language(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -594,7 +594,7 @@ static gboolean on_sim_set_language(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_set_language req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "w"))
                return TRUE;
@@ -624,7 +624,7 @@ static gboolean on_sim_get_call_forwarding(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        TReturn ret;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -658,7 +658,7 @@ static gboolean on_sim_set_call_forwarding(TelephonySim *sim,
        CoreObject *co_sim = NULL;
        struct treq_sim_set_callforwarding req;
        TReturn ret;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "w"))
                return TRUE;
@@ -713,7 +713,7 @@ static gboolean on_sim_get_message_waiting(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        TReturn ret;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -747,7 +747,7 @@ static gboolean on_sim_set_message_waiting(TelephonySim *sim,
        TReturn ret;
        CoreObject *co_sim = NULL;
        struct treq_sim_set_messagewaiting req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "w"))
                return TRUE;
@@ -804,7 +804,7 @@ static gboolean on_sim_get_mailbox(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        TReturn ret;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -836,7 +836,7 @@ static gboolean on_sim_set_mailbox(TelephonySim *sim,
        TReturn ret;
        CoreObject *co_sim = NULL;
        struct treq_sim_set_mailbox req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "w"))
                return TRUE;
@@ -890,7 +890,7 @@ static gboolean on_sim_get_cphsinfo(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -913,7 +913,7 @@ static gboolean on_sim_get_service_table(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct tel_sim_service_table *svct = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -986,7 +986,7 @@ static gboolean on_sim_get_msisdn(TelephonySim *sim,
        CoreObject *co_sim = NULL;
        struct tel_sim_msisdn_list *msisdn_list = NULL;
        gboolean read_from_modem = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1070,7 +1070,7 @@ static gboolean on_sim_get_oplmnwact(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1093,7 +1093,7 @@ static gboolean on_sim_get_spn(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct tel_sim_spn* spn = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1129,7 +1129,7 @@ static gboolean on_sim_get_cphs_netname(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct tel_sim_cphs_netname *cphs_netname = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1168,7 +1168,7 @@ static gboolean on_sim_get_gid(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1198,7 +1198,7 @@ static gboolean on_sim_authentication(TelephonySim *sim,
        TReturn ret;
        CoreObject *co_sim = NULL;
        struct treq_sim_req_authentication req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1283,7 +1283,7 @@ static gboolean on_sim_verify_sec(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_verify_pins req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1318,7 +1318,7 @@ static gboolean on_sim_verify_puk(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_verify_puks req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1355,7 +1355,7 @@ static gboolean on_sim_change_pin(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_change_pins req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1391,7 +1391,7 @@ static gboolean on_sim_disable_facility(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        struct treq_sim_disable_facility req;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1463,7 +1463,7 @@ static gboolean on_sim_enable_facility(TelephonySim *sim, GDBusMethodInvocation
        struct custom_data *ctx = user_data;
        struct treq_sim_enable_facility req;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1534,7 +1534,7 @@ static gboolean on_sim_get_facility(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        struct treq_sim_get_facility_status req;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1599,7 +1599,7 @@ static gboolean on_sim_get_lock_info(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_get_lock_info req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1667,7 +1667,7 @@ static gboolean on_sim_transfer_apdu(TelephonySim *sim,
        guchar rt_i;
        CoreObject *co_sim = NULL;
        int i = 0;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "x"))
                return TRUE;
@@ -1705,7 +1705,7 @@ static gboolean on_sim_get_atr(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1730,7 +1730,7 @@ static gboolean on_sim_get_fields(TelephonySim *sim,
        CoreObject *co_sim = NULL;
        GVariantBuilder b;
        GVariant *gv_fields = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1782,7 +1782,7 @@ static gboolean on_sim_set_power_state(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        struct treq_sim_set_powerstate req;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "w"))
                return TRUE;
@@ -1810,7 +1810,7 @@ static gboolean on_sim_get_impi(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1831,7 +1831,7 @@ static gboolean on_sim_get_impu(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1852,7 +1852,7 @@ static gboolean on_sim_get_domain(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1873,7 +1873,7 @@ static gboolean on_sim_get_pcscf(TelephonySim *sim,
 {
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1895,7 +1895,7 @@ static gboolean on_sim_get_app_list(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        CoreObject *co_sim = NULL;
        unsigned char app_list = 0;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -1917,7 +1917,7 @@ static gboolean on_sim_get_isim_service_table(TelephonySim *sim,
        struct custom_data *ctx = user_data;
        struct tel_sim_ist *ist = NULL;
        CoreObject *co_sim = NULL;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SIM, "r"))
                return TRUE;
@@ -2130,7 +2130,7 @@ gboolean dbus_plugin_sim_response(struct custom_data *ctx,
        UserRequest *ur, struct dbus_request_info *dbus_info,
        enum tcore_response_command command, unsigned int data_len, const void *data)
 {
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_SIM_GET_ECC: {
index 5746a96..dfb7fac 100644 (file)
@@ -41,7 +41,7 @@ static gboolean on_sms_send_msg(TelephonySms *sms,
        struct treq_sms_send_msg req;
        struct custom_data *ctx = user_data;
        enum tcore_request_command command;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        int i = 0;
        GVariantIter *iter = NULL;
@@ -103,7 +103,7 @@ static gboolean on_sms_read_msg(TelephonySms *sms,
 {
        struct treq_sms_read_msg req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -125,7 +125,7 @@ static gboolean on_sms_save_msg(TelephonySms *sms,
 {
        struct treq_sms_save_msg req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        int i = 0;
        GVariantIter *iter = NULL;
@@ -185,7 +185,7 @@ static gboolean on_sms_delete_msg(TelephonySms *sms,
 {
        struct treq_sms_delete_msg req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "x"))
                return TRUE;
@@ -204,7 +204,7 @@ static gboolean on_sms_get_msg_count(TelephonySms *sms,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -222,7 +222,7 @@ static gboolean on_sms_get_sca(TelephonySms *sms,
 {
        struct treq_sms_get_sca req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -244,7 +244,7 @@ static gboolean on_sms_set_sca(TelephonySms *sms,
        gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "w"))
                return TRUE;
@@ -296,7 +296,7 @@ static gboolean on_sms_get_cb_config(TelephonySms *sms,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -317,7 +317,7 @@ static gboolean on_sms_set_cb_config(TelephonySms *sms,
 {
        struct treq_sms_set_cb_config req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        GVariant *value = NULL;
        GVariant *inner_gv = NULL;
@@ -372,7 +372,7 @@ static gboolean on_sms_set_mem_status(TelephonySms *sms,
 {
        struct treq_sms_set_mem_status req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "w"))
                return TRUE;
@@ -391,7 +391,7 @@ static gboolean on_sms_get_pref_bearer(TelephonySms *sms,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -410,7 +410,7 @@ static gboolean on_sms_set_pref_bearer(TelephonySms *sms,
 {
        struct treq_sms_set_pref_bearer req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "w"))
                return TRUE;
@@ -433,7 +433,7 @@ static gboolean on_sms_set_delivery_report(TelephonySms *sms,
 {
        struct treq_sms_set_delivery_report req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        guchar *decoded_sca = NULL;
        guchar *decoded_tpdu = NULL;
@@ -512,7 +512,7 @@ static gboolean on_sms_set_msg_status(TelephonySms *sms,
 {
        struct treq_sms_set_msg_status req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "w"))
                return TRUE;
@@ -534,7 +534,7 @@ static gboolean on_sms_get_sms_params(TelephonySms *sms,
 {
        struct treq_sms_get_params req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -561,7 +561,7 @@ static gboolean on_sms_set_sms_params(TelephonySms *sms,
 {
        struct treq_sms_set_params req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        int i = 0;
        GVariantIter *iter = NULL;
@@ -633,7 +633,7 @@ static gboolean on_sms_get_sms_param_cnt(TelephonySms *sms,
        GDBusMethodInvocation *invocation, gpointer user_data)
 {
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -653,7 +653,7 @@ static gboolean on_sms_get_sms_ready_status(TelephonySms *sms,
        CoreObject *co_sms = NULL;
        TcorePlugin *plugin = NULL;
        gboolean ready_status = FALSE;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SMS, "r"))
                return TRUE;
@@ -769,7 +769,7 @@ gboolean dbus_plugin_sms_response(struct custom_data *ctx,
        UserRequest *ur, struct dbus_request_info *dbus_info,
        enum tcore_response_command command, unsigned int data_len, const void *data)
 {
-       char *cpname = dbus_info ? GET_CP_NAME(dbus_info->invocation) : "";
+       char *cpname = GET_CP_NAME(dbus_info->invocation);
 
        switch (command) {
        case TRESP_SMS_SEND_UMTS_MSG: {
index 48e7371..f1114df 100644 (file)
@@ -136,7 +136,7 @@ static gboolean on_ss_activate_barring(TelephonySs *ss,
        struct treq_ss_barring req;
        struct custom_data *ctx = user_data;
        char buf[MAX_SS_BARRING_PASSWORD_LEN + 1];
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "x"))
                return TRUE;
@@ -170,7 +170,7 @@ static gboolean on_ss_deactivate_barring(TelephonySs *ss,
        struct treq_ss_barring req;
        struct custom_data *ctx = user_data;
        char buf[MAX_SS_BARRING_PASSWORD_LEN + 1];
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "x"))
                return TRUE;
@@ -204,7 +204,7 @@ static gboolean on_ss_change_barring_password(TelephonySs *ss,
 {
        struct treq_ss_barring_change_password req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "x"))
                return TRUE;
@@ -230,7 +230,7 @@ static gboolean on_ss_get_barring_status(TelephonySs *ss,
 {
        struct treq_ss_barring req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "r"))
                return TRUE;
@@ -260,7 +260,7 @@ static gboolean on_ss_register_forwarding(TelephonySs *ss,
 {
        struct treq_ss_forwarding req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -295,7 +295,7 @@ static gboolean on_ss_deregister_forwarding(TelephonySs *ss,
 {
        struct treq_ss_forwarding req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -330,7 +330,7 @@ static gboolean on_ss_activate_forwarding(TelephonySs *ss,
 {
        struct treq_ss_forwarding req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -365,7 +365,7 @@ static gboolean on_ss_deactivate_forwarding(TelephonySs *ss,
 {
        struct treq_ss_forwarding req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -398,7 +398,7 @@ static gboolean on_ss_get_forwarding_status(TelephonySs *ss,
 {
        struct treq_ss_forwarding req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "r"))
                return TRUE;
@@ -424,7 +424,7 @@ static gboolean on_ss_activate_waiting(TelephonySs *ss,
 {
        struct treq_ss_waiting req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -449,7 +449,7 @@ static gboolean on_ss_deactivate_waiting(TelephonySs *ss,
 {
        struct treq_ss_waiting req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -474,7 +474,7 @@ static gboolean on_ss_get_waiting_status(TelephonySs *ss,
 {
        struct treq_ss_waiting req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "r"))
                return TRUE;
@@ -499,7 +499,7 @@ static gboolean on_ss_set_cli_status(TelephonySs *ss,
 {
        struct treq_ss_set_cli req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "w"))
                return TRUE;
@@ -526,7 +526,7 @@ static gboolean on_ss_get_cli_status(TelephonySs *ss,
 {
        struct treq_ss_cli req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "r"))
                return TRUE;
@@ -553,7 +553,7 @@ static gboolean on_ss_send_ussd(TelephonySs *ss,
 {
        struct treq_ss_ussd req;
        struct custom_data *ctx = user_data;
-       cynara *p_cynara = (ctx) ? ctx->p_cynara : NULL;
+       cynara *p_cynara = ctx->p_cynara;
 
        if (!check_access_control(p_cynara, invocation, AC_SS, "x"))
                return TRUE;
@@ -1032,7 +1032,7 @@ gboolean dbus_plugin_ss_notification(struct custom_data *ctx,
                return FALSE;
        }
 
-       if (!data) {
+       if (!data && command != TNOTI_SS_RELEASE_COMPLETE) {
                err("data is NULL");
                return FALSE;
        }
index 9de75f1..1376d2e 100644 (file)
@@ -350,7 +350,7 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
        gboolean rv = FALSE;
        bundle *bundle_data = 0;
        gchar *encoded_data = NULL, *cmd_type = NULL;
-       struct tel_sat_select_item_ind select_item;
+       struct tel_sat_select_item_ind *select_item = NULL;
 
        gboolean help_info ;
        gchar *selected_text = NULL;
@@ -381,7 +381,12 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
        gchar *item_str;
        gint item_id, item_len;
 #endif
-       memset(&select_item, 0, sizeof(struct tel_sat_select_item_ind));
+
+       select_item = g_try_new0(struct tel_sat_select_item_ind, 1);
+       if (select_item == NULL) {
+               err("Failed to allocate memory");
+               return FALSE;
+       }
 
 #if defined(TIZEN_SUPPORT_SAT_ICON)
        g_variant_get(data, "(ibsiii@v@v@v)", &command_id, &help_info, &selected_text,
@@ -390,23 +395,23 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
        g_variant_get(data, "(ibsiii@v)", &command_id, &help_info, &selected_text,
                &text_len, &default_item_id, &menu_cnt, &menu_items);
 #endif
-       select_item.commandId = command_id;
-       select_item.bIsHelpInfoAvailable = (help_info ? 1 : 0);
-       memcpy(select_item.text.string, selected_text, SAT_TEXT_STRING_LEN_MAX+1);
+       select_item->commandId = command_id;
+       select_item->bIsHelpInfoAvailable = (help_info ? 1 : 0);
+       memcpy(select_item->text.string, selected_text, SAT_TEXT_STRING_LEN_MAX+1);
        g_free(selected_text);
 
-       select_item.text.stringLen = text_len;
-       select_item.defaultItemIndex = default_item_id;
-       select_item.menuItemCount = menu_cnt;
+       select_item->text.stringLen = text_len;
+       select_item->defaultItemIndex = default_item_id;
+       select_item->menuItemCount = menu_cnt;
        if (menu_items && menu_cnt > 0) {
                unbox = g_variant_get_variant(menu_items);
                dbg("items(%p) items type_format(%s)", menu_items, g_variant_get_type_string(unbox));
 
                g_variant_get(unbox, "a(iis)", &iter);
                while (g_variant_iter_loop(iter, "(iis)", &item_id, &item_len, &item_str)) {
-                       select_item.menuItem[local_index].itemId = item_id;
-                       select_item.menuItem[local_index].textLen = item_len;
-                       memcpy(select_item.menuItem[local_index].text, item_str, SAT_ITEM_TEXT_LEN_MAX + 1);
+                       select_item->menuItem[local_index].itemId = item_id;
+                       select_item->menuItem[local_index].textLen = item_len;
+                       memcpy(select_item->menuItem[local_index].text, item_str, SAT_ITEM_TEXT_LEN_MAX + 1);
                        local_index++;
                }
                g_variant_iter_free(iter);
@@ -420,18 +425,19 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
                while (g_variant_iter_loop(iter, "(biiiiiis)", &is_exist, &icon_quali, &icon_identifier, &width, &height, &ics, &icon_data_len, &icon_data)) {
                        if (!is_exist)
                                break;
-                       select_item.iconId.bIsPresent = is_exist;
-                       select_item.iconId.iconQualifier = icon_quali;
-                       select_item.iconId.iconIdentifier = icon_identifier;
-                       select_item.iconId.iconInfo.width = width;
-                       select_item.iconId.iconInfo.height = height;
-                       select_item.iconId.iconInfo.ics = ics;
+                       select_item->iconId.bIsPresent = is_exist;
+                       select_item->iconId.iconQualifier = icon_quali;
+                       select_item->iconId.iconIdentifier = icon_identifier;
+                       select_item->iconId.iconInfo.width = width;
+                       select_item->iconId.iconInfo.height = height;
+                       select_item->iconId.iconInfo.ics = ics;
                        if (icon_data_len > 0) {
-                               select_item.iconId.iconInfo.iconDataLen = icon_data_len;
-                               memcpy(select_item.iconId.iconInfo.iconFile, icon_data, icon_data_len);
+                               select_item->iconId.iconInfo.iconDataLen = icon_data_len;
+                               memcpy(select_item->iconId.iconInfo.iconFile, icon_data, icon_data_len);
                        }
-                       dbg("icon exist(%d), icon_quali: (%d), icon_id: (%d), width: (%d), height: (%d), ics: (%d), icon_data_len: (%d)", select_item.iconId.bIsPresent, select_item.iconId.iconQualifier, select_item.iconId.iconIdentifier, select_item.iconId.iconInfo.width,
-                                       select_item.iconId.iconInfo.height, select_item.iconId.iconInfo.ics, select_item.iconId.iconInfo.iconDataLen);
+                       dbg("icon exist(%d), icon_quali: (%d), icon_id: (%d), width: (%d), height: (%d), ics: (%d), icon_data_len: (%d)",
+                                       select_item->iconId.bIsPresent, select_item->iconId.iconQualifier, select_item->iconId.iconIdentifier, select_item->iconId.iconInfo.width,
+                                       select_item->iconId.iconInfo.height, select_item->iconId.iconInfo.ics, select_item->iconId.iconInfo.iconDataLen);
                }
                g_variant_iter_free(iter);
        }
@@ -443,21 +449,21 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
                while (g_variant_iter_loop(iter, "(biiv)", &is_list_exist, &icon_list_quali, &list_cnt, &icon_list_info)) {
                        if (!is_list_exist)
                                break;
-                       select_item.iconIdList.bIsPresent = is_list_exist;
-                       select_item.iconIdList.iconListQualifier = icon_list_quali;
-                       select_item.iconIdList.iconCount = list_cnt;
+                       select_item->iconIdList.bIsPresent = is_list_exist;
+                       select_item->iconIdList.iconListQualifier = icon_list_quali;
+                       select_item->iconIdList.iconCount = list_cnt;
 
                        unbox_list_info = g_variant_get_variant(icon_list_info);
                        g_variant_get(unbox_list_info, "a(iiiiis)", &iter2);
 
                        while (g_variant_iter_loop(iter2, "(iiiiis)", &icon_list_identifier, &list_width, &list_height, &list_ics, &icon_list_data_len, &icon_list_data)) {
-                               select_item.iconIdList.iconIdentifierList[icon_index] = icon_identifier;
-                               select_item.iconIdList.iconInfo[icon_index].width = list_width;
-                               select_item.iconIdList.iconInfo[icon_index].height = list_height;
-                               select_item.iconIdList.iconInfo[icon_index].ics = list_ics;
+                               select_item->iconIdList.iconIdentifierList[icon_index] = icon_identifier;
+                               select_item->iconIdList.iconInfo[icon_index].width = list_width;
+                               select_item->iconIdList.iconInfo[icon_index].height = list_height;
+                               select_item->iconIdList.iconInfo[icon_index].ics = list_ics;
                                if (icon_list_data_len > 0) {
-                                       select_item.iconIdList.iconInfo[icon_index].iconDataLen = icon_list_data_len;
-                                       memcpy(select_item.iconIdList.iconInfo[icon_index].iconFile, icon_list_data, icon_list_data_len);
+                                       select_item->iconIdList.iconInfo[icon_index].iconDataLen = icon_list_data_len;
+                                       memcpy(select_item->iconIdList.iconInfo[icon_index].iconFile, icon_list_data, icon_list_data_len);
                                }
                                icon_index++;
                        }
@@ -467,7 +473,7 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
        }
 #endif
        cmd_type = g_strdup_printf("%d", SAT_PROATV_CMD_SELECT_ITEM);
-       encoded_data = g_base64_encode((const guchar*)&select_item, sizeof(struct tel_sat_select_item_ind));
+       encoded_data = g_base64_encode((const guchar*)select_item, sizeof(struct tel_sat_select_item_ind));
 
        bundle_data = bundle_create();
        bundle_add(bundle_data, "KEY_EXEC_TYPE", "1");
@@ -479,6 +485,7 @@ static gboolean _sat_ui_support_processing_select_item_ind(GVariant *data, char
 
        g_free(encoded_data);
        g_free(cmd_type);
+       g_free(select_item);
 
        return rv;
 }