fix svace issue 79/140579/3 accepted/tizen/3.0/common/20170731.131537 accepted/tizen/3.0/ivi/20170731.012646 accepted/tizen/3.0/wearable/20170731.012642 submit/tizen_3.0/20170726.045520
authorjkjo92 <jkjo92@samsung.com>
Tue, 25 Jul 2017 11:39:36 +0000 (20:39 +0900)
committerjkjo92 <jkjo92@samsung.com>
Wed, 26 Jul 2017 04:50:23 +0000 (13:50 +0900)
Change-Id: Ice2f99521cf960bd46953166473a57940172f13e
Signed-off-by: jkjo92 <jkjo92@samsung.com>
common/fido_tlv_util.c
server/fido_server.c

index ced34e0..b4c9e25 100755 (executable)
@@ -140,11 +140,12 @@ _tlv_util_decode_reg_assertion(char *tlv_enc)
                        }
 
                        _INFO("Found TAG_KEYID");
+                       SAFE_DELETE(tlv_dec);
                        return assrt_tlv;
                }
        }
 
-
+       SAFE_DELETE(tlv_dec);
        return NULL;
 }
 
index 51b6ae8..0bd206f 100755 (executable)
@@ -726,12 +726,15 @@ _ui_response_callback(int error_code, _ui_auth_data_t *selected_auth_data, void
        match_data->auth_index = selected_auth_data->auth_index;
        match_data->asm_id = strdup(selected_auth_data->asm_id);
 
-       if ((cb_data != NULL) && (cb_data->type == _PROCESS_TYPE_REG))
+       if (cb_data->type == _PROCESS_TYPE_REG)
                __handle_reg(cb_data, match_data);
-
-       if ((cb_data != NULL) && (cb_data->type == _PROCESS_TYPE_AUTH))
+       else if (cb_data->type == _PROCESS_TYPE_AUTH)
                __handle_auth(cb_data, match_data);
-
+       else {
+               _send_process_response(cb_data, FIDO_ERROR_UNKNOWN, NULL);
+               _free_matched_auth_data(match_data);
+               return;
+       }
        _free_matched_auth_data(match_data);
 
 }
@@ -1199,11 +1202,16 @@ _discover_response_cb_for_process(int tz_error_code, int error_code, GList *avai
                if (allowed_auth_list_iter != NULL) {
                        _matched_auth_data_t *match_data = (_matched_auth_data_t *)(allowed_auth_list_iter->data);
 
-                       if ((cb_data != NULL) && (cb_data->type == _PROCESS_TYPE_REG))
+                       if (cb_data->type == _PROCESS_TYPE_REG)
                                __handle_reg(cb_data, match_data);
-
-                       if ((cb_data != NULL) && (cb_data->type == _PROCESS_TYPE_AUTH))
+                       else if (cb_data->type == _PROCESS_TYPE_AUTH)
                                __handle_auth(cb_data, match_data);
+                       else {
+                               _send_process_response(cb_data, FIDO_ERROR_UNKNOWN, NULL);
+                               if (allowed_auth_list != NULL)
+                                       g_list_free_full(allowed_auth_list, _free_matched_auth_data);
+                               return;
+                       }
                }
        }
        if (allowed_auth_list != NULL)