target: don't truncate the fail intr address

The temporary buffer was only 32 characters but ->last_intr_fail_ip_addr
is a 48 character buffer.  We don't need to use a temporary buffer at
all, we can just print directly to "page".

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>

diff --git a/drivers/target/iscsi/iscsi_target_stat.c b/drivers/target/iscsi/iscsi_target_stat.c
index 54a6f3834cf0c93d596087f87190626caaef53ea..464b4206a51ef0eb06b1c5b4a33f1fcde58aeeed 100644 (file)
--- a/drivers/target/iscsi/iscsi_target_stat.c
+++ b/drivers/target/iscsi/iscsi_target_stat.c
@@ -429,16 +429,19 @@ static ssize_t iscsi_stat_tgt_attr_show_attr_fail_intr_addr(
        struct iscsi_tiqn *tiqn = container_of(igrps,
                        struct iscsi_tiqn, tiqn_stat_grps);
        struct iscsi_login_stats *lstat = &tiqn->login_stats;
-       unsigned char buf[32];
+       int ret;
 
        spin_lock(&lstat->lock);
-       if (lstat->last_intr_fail_ip_family == AF_INET6)
-               snprintf(buf, 32, "[%s]", lstat->last_intr_fail_ip_addr);
-       else
-               snprintf(buf, 32, "%s", lstat->last_intr_fail_ip_addr);
+       if (lstat->last_intr_fail_ip_family == AF_INET6) {
+               ret = snprintf(page, PAGE_SIZE, "[%s]\n",
+                              lstat->last_intr_fail_ip_addr);
+       } else {
+               ret = snprintf(page, PAGE_SIZE, "%s\n",
+                              lstat->last_intr_fail_ip_addr);
+       }
        spin_unlock(&lstat->lock);
 
-       return snprintf(page, PAGE_SIZE, "%s\n", buf);
+       return ret;
 }
 ISCSI_STAT_TGT_ATTR_RO(fail_intr_addr);