Fix for svace 01/75901/1
authorsajal.j <sajal.jain@samsung.com>
Wed, 22 Jun 2016 06:08:32 +0000 (11:38 +0530)
committersajal.j <sajal.jain@samsung.com>
Wed, 22 Jun 2016 06:09:17 +0000 (11:39 +0530)
Signed-off-by: sajal.j <sajal.jain@samsung.com>
Change-Id: I846f0cf5cfb8755836521862187f83da8a907dee

server/fido_asm_plugin_manager.c
server/fido_uaf_policy_checker.c

index d8e2e58..7b2255b 100644 (file)
@@ -292,6 +292,9 @@ _asm_plugin_mgr_discover_all(_asm_plugin_discover_response_cb cb, void *user_dat
 
        cb_data->user_data = user_data;
 
+       if(cb_data->asm_proxy_list_iter == NULL)
+               return FIDO_ERROR_NOT_SUPPORTED;
+
        _fido_asm_proxy_t *asm_proxy = (_fido_asm_proxy_t*)(cb_data->asm_proxy_list_iter->data);
 
        return _asm_ipc_send(asm_proxy->asm_id, _GET_INFO_ASM_REQUEST_JSON, __discover_cb_internal, cb_data);
index 7d1c3f6..efd57e5 100644 (file)
@@ -50,7 +50,7 @@ _int_cmp(gconstpointer a, gconstpointer b)
 bool
 _policy_checker_is_matched(_match_criteria_t *match_criteria, fido_authenticator_s *auth_info)
 {
-       _INFO("_policy_checker_is_matched");
+       _INFO("_policy_checker_is_matched::start");
 
        /* -1 means the int value is not present, so we should ignore that. */
 
@@ -91,7 +91,7 @@ _policy_checker_is_matched(_match_criteria_t *match_criteria, fido_authenticator
                SAFE_DELETE(auth_aaid);
        }
 
-       _INFO("keyid matching");
+       _INFO("keyid matching start");
 
        /* 3. If any Key ID is mentioned in match_criteria, then atleast one Key ID should match */
        GList *key_id_list = match_criteria->key_id_list;