From: tangwenji <tang.wenji@zte.com.cn>
Date: Thu, 24 Aug 2017 11:59:37 +0000 (+0800)
Subject: target:fix condition return in core_pr_dump_initiator_port()
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=24528f089d0a444070aa4f715ace537e8d6bf168;p=linux.git

target:fix condition return in core_pr_dump_initiator_port()

When is pr_reg->isid_present_at_reg is false,this function should return.

This fixes a regression originally introduced by:

  commit d2843c173ee53cf4c12e7dfedc069a5bc76f0ac5
  Author: Andy Grover <agrover@redhat.com>
  Date:   Thu May 16 10:40:55 2013 -0700

      target: Alter core_pr_dump_initiator_port for ease of use

Signed-off-by: tangwenji <tang.wenji@zte.com.cn>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
---

diff --git a/drivers/target/target_core_pr.c b/drivers/target/target_core_pr.c
index 871ae21870be9..a544907098110 100644
--- a/drivers/target/target_core_pr.c
+++ b/drivers/target/target_core_pr.c
@@ -58,8 +58,10 @@ void core_pr_dump_initiator_port(
 	char *buf,
 	u32 size)
 {
-	if (!pr_reg->isid_present_at_reg)
+	if (!pr_reg->isid_present_at_reg) {
 		buf[0] = '\0';
+		return;
+	}
 
 	snprintf(buf, size, ",i,0x%s", pr_reg->pr_reg_isid);
 }