diff options
author | Michael Veksler <Mihails.Vekslers@zabbix.com> | 2022-05-12 16:55:04 +0300 |
---|---|---|
committer | Michael Veksler <Mihails.Vekslers@zabbix.com> | 2022-05-12 16:55:04 +0300 |
commit | 32929452e00673a08693911c81ce84a514f9b583 (patch) | |
tree | 663aa6a86c0808f77e2624a7e778bf3191d0e9d5 /src | |
parent | 8e6fa162029bac0f957159fd9e83f9fecf89ee72 (diff) |
.......PS. [ZBX-20600] refactoring dsname for check_vcenter_datastore_hv_list
Diffstat (limited to 'src')
-rw-r--r-- | src/zabbix_server/poller/checks_simple_vmware.c | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/zabbix_server/poller/checks_simple_vmware.c b/src/zabbix_server/poller/checks_simple_vmware.c index 2c03846109d..90ab321c00b 100644 --- a/src/zabbix_server/poller/checks_simple_vmware.c +++ b/src/zabbix_server/poller/checks_simple_vmware.c @@ -2359,19 +2359,19 @@ int check_vcenter_datastore_hv_list(AGENT_REQUEST *request, const char *username if (NULL == (service = get_vmware_service(url, username, password, result, &ret))) goto unlock; - for (i = 0; i < service->data->datastores.values_num; i++) + + if (NULL == (datastore = ds_get(&service->data->datastores, ds_name))) { - if (0 != strcmp(ds_name, service->data->datastores.values[i]->name)) - continue; + zbx_vmware_datastore_t ds_cmp = {.name = (char *)ds_name}; - datastore = service->data->datastores.values[i]; - break; - } + if (FAIL == (i = zbx_vector_vmware_datastore_search(&service->data->datastores, &ds_cmp, + vmware_ds_name_compare))) + { + SET_MSG_RESULT(result, zbx_strdup(NULL, "Unknown datastore name.")); + goto unlock; + } - if (NULL == datastore) - { - SET_MSG_RESULT(result, zbx_strdup(NULL, "Unknown datastore name.")); - goto unlock; + datastore = service->data->datastores.values[i]; } for (i=0; i < datastore->hv_uuids_access.values_num; i++) |