|
 |
9a267a |
From 13affd82b8248b954d10eb4fd3d544796360fafd Mon Sep 17 00:00:00 2001
|
|
|
9a267a |
From: Chris Leech <cleech@redhat.com>
|
|
|
9a267a |
Date: Wed, 17 Feb 2021 09:56:55 -0800
|
|
|
9a267a |
Subject: [PATCH] libopeniscsiusr: fix error messages
|
|
|
9a267a |
|
|
|
9a267a |
The error message in iscsi_session_get [libopeniscsiusr/session.c:140]
|
|
|
9a267a |
when a session or connection path isn't found in sysfs was failing to
|
|
|
9a267a |
print the session ID, instead it printed the address of the static
|
|
|
9a267a |
string for the remainder of the message ("does not exists") due to an
|
|
|
9a267a |
extra comma.
|
|
|
9a267a |
|
|
|
9a267a |
Additionally change all occurances of "does not exists" to "does not
|
|
|
9a267a |
exist"
|
|
|
9a267a |
|
|
|
9a267a |
Signed-off-by: Chris Leech <cleech@redhat.com>
|
|
|
9a267a |
---
|
|
|
9a267a |
libopeniscsiusr/libopeniscsiusr/libopeniscsiusr.h | 2 +-
|
|
|
9a267a |
libopeniscsiusr/session.c | 6 +++---
|
|
|
9a267a |
libopeniscsiusr/sysfs.c | 8 ++++----
|
|
|
9a267a |
3 files changed, 8 insertions(+), 8 deletions(-)
|
|
|
9a267a |
|
|
|
9a267a |
diff --git a/libopeniscsiusr/libopeniscsiusr/libopeniscsiusr.h b/libopeniscsiusr/libopeniscsiusr/libopeniscsiusr.h
|
|
|
9a267a |
index 4395902..a29d5b1 100644
|
|
|
9a267a |
--- a/libopeniscsiusr/libopeniscsiusr/libopeniscsiusr.h
|
|
|
9a267a |
+++ b/libopeniscsiusr/libopeniscsiusr/libopeniscsiusr.h
|
|
|
9a267a |
@@ -288,7 +288,7 @@ __DLL_EXPORT void iscsi_sessions_free(struct iscsi_session **ses,
|
|
|
9a267a |
* Output pointer of 'struct iscsi_session' pointer. Its memory
|
|
|
9a267a |
* should be freed by iscsi_session_free().
|
|
|
9a267a |
* If this pointer is NULL, your program will be terminated by assert.
|
|
|
9a267a |
- * If specified iSCSI session does not exists, this pointer will be set to
|
|
|
9a267a |
+ * If specified iSCSI session does not exist, this pointer will be set to
|
|
|
9a267a |
* NULL with LIBISCSI_OK returned.
|
|
|
9a267a |
*
|
|
|
9a267a |
* Return:
|
|
|
9a267a |
diff --git a/libopeniscsiusr/session.c b/libopeniscsiusr/session.c
|
|
|
9a267a |
index 7ace4d6..f122fe3 100644
|
|
|
9a267a |
--- a/libopeniscsiusr/session.c
|
|
|
9a267a |
+++ b/libopeniscsiusr/session.c
|
|
|
9a267a |
@@ -127,17 +127,17 @@ int iscsi_session_get(struct iscsi_context *ctx, uint32_t sid,
|
|
|
9a267a |
_alloc_null_check(ctx, *se , rc, out);
|
|
|
9a267a |
|
|
|
9a267a |
if (! _file_exists(sysfs_se_dir_path)) {
|
|
|
9a267a |
- _info(ctx, "Sysfs path '%s' does not exists",
|
|
|
9a267a |
+ _info(ctx, "Sysfs path '%s' does not exist",
|
|
|
9a267a |
sysfs_se_dir_path);
|
|
|
9a267a |
rc = LIBISCSI_ERR_SESS_NOT_FOUND;
|
|
|
9a267a |
}
|
|
|
9a267a |
if (! _file_exists(sysfs_con_dir_path)) {
|
|
|
9a267a |
- _info(ctx, "Sysfs path '%s' does not exists",
|
|
|
9a267a |
+ _info(ctx, "Sysfs path '%s' does not exist",
|
|
|
9a267a |
sysfs_se_dir_path);
|
|
|
9a267a |
rc = LIBISCSI_ERR_SESS_NOT_FOUND;
|
|
|
9a267a |
}
|
|
|
9a267a |
if (rc == LIBISCSI_ERR_SESS_NOT_FOUND) {
|
|
|
9a267a |
- _error(ctx, "Specified SID %" PRIu32, "does not exists",
|
|
|
9a267a |
+ _error(ctx, "Specified SID %" PRIu32 " does not exist",
|
|
|
9a267a |
sid);
|
|
|
9a267a |
goto out;
|
|
|
9a267a |
}
|
|
|
9a267a |
diff --git a/libopeniscsiusr/sysfs.c b/libopeniscsiusr/sysfs.c
|
|
|
9a267a |
index 2c3f077..355ad55 100644
|
|
|
9a267a |
--- a/libopeniscsiusr/sysfs.c
|
|
|
9a267a |
+++ b/libopeniscsiusr/sysfs.c
|
|
|
9a267a |
@@ -169,11 +169,11 @@ int _sysfs_prop_get_str(struct iscsi_context *ctx, const char *dir_path,
|
|
|
9a267a |
if (default_value == NULL) {
|
|
|
9a267a |
rc = LIBISCSI_ERR_SYSFS_LOOKUP;
|
|
|
9a267a |
_error(ctx, "Failed to read '%s': "
|
|
|
9a267a |
- "file '%s' does not exists", prop_name,
|
|
|
9a267a |
+ "file '%s' does not exist", prop_name,
|
|
|
9a267a |
file_path);
|
|
|
9a267a |
} else {
|
|
|
9a267a |
_info(ctx, "Failed to read '%s': "
|
|
|
9a267a |
- "file '%s' does not exists, "
|
|
|
9a267a |
+ "file '%s' does not exist, "
|
|
|
9a267a |
"using default value %s", prop_name,
|
|
|
9a267a |
file_path, default_value);
|
|
|
9a267a |
memcpy(buff, (void *) default_value,
|
|
|
9a267a |
@@ -244,13 +244,13 @@ static int iscsi_sysfs_prop_get_ll(struct iscsi_context *ctx,
|
|
|
9a267a |
if (! ignore_error) {
|
|
|
9a267a |
rc = LIBISCSI_ERR_SYSFS_LOOKUP;
|
|
|
9a267a |
_error(ctx, "Failed to read '%s': "
|
|
|
9a267a |
- "file '%s' does not exists",
|
|
|
9a267a |
+ "file '%s' does not exist",
|
|
|
9a267a |
prop_name, file_path);
|
|
|
9a267a |
goto out;
|
|
|
9a267a |
} else {
|
|
|
9a267a |
_info(ctx,
|
|
|
9a267a |
"Failed to read '%s': "
|
|
|
9a267a |
- "File '%s' does not exists, using ",
|
|
|
9a267a |
+ "File '%s' does not exist, using ",
|
|
|
9a267a |
"default value %lld",
|
|
|
9a267a |
prop_name, file_path, default_value);
|
|
|
9a267a |
*val = default_value;
|
|
|
9a267a |
--
|
|
|
9a267a |
2.26.2
|
|
|
9a267a |
|