git: 315bca194a14 - main - Fix an early return in ctld UCL parser

Ryan Stone rstone at FreeBSD.org
Tue Aug 31 03:04:28 UTC 2021


The branch main has been updated by rstone:

URL: https://cgit.FreeBSD.org/src/commit/?id=315bca194a14f3b9ec683675bafa8d990d16bfaf

commit 315bca194a14f3b9ec683675bafa8d990d16bfaf
Author:     Ryan Stone <rstone at FreeBSD.org>
AuthorDate: 2021-07-07 20:04:10 +0000
Commit:     Ryan Stone <rstone at FreeBSD.org>
CommitDate: 2021-08-31 02:59:33 +0000

    Fix an early return in ctld UCL parser
    
    If the UCL ctld parser encountered a port that used the CTL
    ioctl device, it fell into a special case that had an erroneous
    early return.  This caused all configuration in the target
    following the port attribute to be skipped.  Fix this by replacing
    the return with a continue so that the rest of the config is
    parsed correctly.
    
    Sponsored by: Dell EMC Isilon
    MFC after: 2 weeks
    Differential Revision: https://reviews.freebsd.org/D31703
    Reviewed by: bapt
---
 usr.sbin/ctld/uclparse.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/usr.sbin/ctld/uclparse.c b/usr.sbin/ctld/uclparse.c
index b73e6b48b2f6..d02530d1b808 100644
--- a/usr.sbin/ctld/uclparse.c
+++ b/usr.sbin/ctld/uclparse.c
@@ -847,7 +847,7 @@ uclparse_target(const char *name, const ucl_object_t *top)
 					return (1);
 				}
 
-				return (0);
+				continue;
 			}
 
 			pp = pport_find(conf, value);


More information about the dev-commits-src-all mailing list