From: liubo <[email protected]>

Fix the potential risk of rc value being washed out by jumping out of the loop

Signed-off-by: liubo <[email protected]>
Reported-by: Zhiqiang Liu <[email protected]>
---
 utils/fwparam_ibft/fwparam_sysfs.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/utils/fwparam_ibft/fwparam_sysfs.c 
b/utils/fwparam_ibft/fwparam_sysfs.c
index a0cd1c7..87fd6d4 100644
--- a/utils/fwparam_ibft/fwparam_sysfs.c
+++ b/utils/fwparam_ibft/fwparam_sysfs.c
@@ -115,8 +115,11 @@ static int get_iface_from_device(char *id, struct 
boot_context *context)
                                break;
                        }
 
-                       if (sscanf(dent->d_name, "net:%s", context->iface) != 1)
+                       if (sscanf(dent->d_name, "net:%s", context->iface) != 
1) {
                                rc = EINVAL;
+                               break;
+                       }
+
                        rc = 0;
                        break;
                } else {
-- 
2.21.0.windows.1

-- 
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/open-iscsi/1591273415-689835-1-git-send-email-wubo40%40huawei.com.

Reply via email to