1
0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00

cov: some stack tracing corrections

Checking some returned values and adding/correcting
some debug tracing.
This commit is contained in:
Zdenek Kabelac 2024-01-05 15:13:42 +01:00
parent a882878eba
commit 18d3283157
3 changed files with 9 additions and 7 deletions

View File

@ -409,13 +409,13 @@ int get_sysfs_value(const char *path, char *buf, size_t buf_size, int error_if_n
if (!(fp = fopen(path, "r"))) { if (!(fp = fopen(path, "r"))) {
if (error_if_no_value) if (error_if_no_value)
log_sys_error("fopen", path); log_sys_debug("fopen", path);
return 0; return 0;
} }
if (!fgets(buf, buf_size, fp)) { if (!fgets(buf, buf_size, fp)) {
if (error_if_no_value) if (error_if_no_value)
log_sys_error("fgets", path); log_sys_debug("fgets", path);
goto out; goto out;
} }

View File

@ -764,7 +764,9 @@ const char *dev_mpath_component_wwid(struct cmd_context *cmd, struct device *dev
continue; continue;
} }
get_sysfs_value(wwid_path, sysbuf, sizeof(sysbuf), 0); if (!get_sysfs_value(wwid_path, sysbuf, sizeof(sysbuf), 0))
stack;
if (!sysbuf[0]) if (!sysbuf[0])
continue; continue;
@ -784,5 +786,3 @@ const char *dev_mpath_component_wwid(struct cmd_context *cmd, struct device *dev
return wwid; return wwid;
} }

View File

@ -215,7 +215,9 @@ static int _searched_devnames_exists(struct cmd_context *cmd,
if (pvids_ok && devs_ok) if (pvids_ok && devs_ok)
ret = 1; ret = 1;
out: out:
fclose(fp); if (fclose(fp))
log_sys_debug("fclose", _searched_file);
log_debug("searched_devnames %s file pvids %d %u devs %d %u search pvids %d %u devs %d %u", log_debug("searched_devnames %s file pvids %d %u devs %d %u search pvids %d %u devs %d %u",
ret ? "match" : "differ", ret ? "match" : "differ",
pvids_count_file, pvids_hash_file, devs_count_file, devs_hash_file, pvids_count_file, pvids_hash_file, devs_count_file, devs_hash_file,