diff options
author | Lee Duncan <lduncan@suse.com> | 2022-01-26 11:44:09 -0800 |
---|---|---|
committer | Lee Duncan <lduncan@suse.com> | 2022-01-26 11:44:09 -0800 |
commit | 1cab1efc813f750f9fa68e35dc16e8e54a1ba1e8 (patch) | |
tree | dd3e52409489a4596cdf18737930dcc0ccfe34e2 /usr | |
parent | 151e701d08e16dce545d017e57c3d5ac957b7c1e (diff) | |
download | open-iscsi-1cab1efc813f750f9fa68e35dc16e8e54a1ba1e8.tar.gz |
Fix more issues discovered by gcc12
Gcc-12 caught a few more errors in the code, where we are
still checking an array address for NULL, which will never
happen.
Diffstat (limited to 'usr')
-rw-r--r-- | usr/discovery.c | 2 | ||||
-rw-r--r-- | usr/iscsi_sysfs.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/usr/discovery.c b/usr/discovery.c index 7dec696..587af6d 100644 --- a/usr/discovery.c +++ b/usr/discovery.c @@ -623,7 +623,7 @@ add_target_record(char *name, char *end, discovery_rec_t *drec, /* if no address is provided, use the default */ if (text >= end) { - if (drec->address == NULL) { + if (drec->address[0] == '\0') { log_error("no default address known for target %s", name); return 0; diff --git a/usr/iscsi_sysfs.c b/usr/iscsi_sysfs.c index 7bb834a..9a591be 100644 --- a/usr/iscsi_sysfs.c +++ b/usr/iscsi_sysfs.c @@ -1416,8 +1416,8 @@ int iscsi_sysfs_get_sessioninfo_by_id(struct session_info *info, char *session) log_debug(7, "found targetname %s address %s pers address %s port %d " "pers port %d driver %s iface name %s ipaddress %s " "netdev %s hwaddress %s iname %s", - info->targetname, info->address ? info->address : "NA", - info->persistent_address ? info->persistent_address : "NA", + info->targetname, info->address[0] ? info->address : "NA", + info->persistent_address[0] ? info->persistent_address : "NA", info->port, info->persistent_port, info->iface.transport_name, info->iface.name, info->iface.ipaddress, info->iface.netdev, info->iface.hwaddress, |