target: don't truncate the fail intr address
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 31 Jan 2013 08:19:09 +0000 (11:19 +0300)
committerNicholas Bellinger <nab@linux-iscsi.org>
Wed, 13 Feb 2013 19:27:58 +0000 (11:27 -0800)
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>
drivers/target/iscsi/iscsi_target_stat.c

index 54a6f3834cf0c93d596087f87190626caaef53ea..464b4206a51ef0eb06b1c5b4a33f1fcde58aeeed 100644 (file)
@@ -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);