d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

inetd: prefer strlcpy to strlen(3) check + strcpy(3), NFC

Browse Source 
This is again functionally equivalent but more concise.
This commit is contained in:
Kyle Evans 2019-12-31 04:53:50 +00:00
parent 1c3b9acf2e
commit a1cedb80c6
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=356218
1 changed files with 9 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
usr.sbin/inetd/inetd.c
View File

@ -1612,6 +1612,7 @@ getconfigent(void)
int v6bind;
#endif
int i;
size_t unsz;
#ifdef IPSEC
policy = NULL;
@ -1832,16 +1833,18 @@ getconfigent(void)
break;
#endif
case AF_UNIX:
if (strlen(sep->se_service) >= sizeof(sep->se_ctrladdr_un.sun_path)) {
syslog(LOG_ERR,
#define SUN_PATH_MAXSIZE sizeof(sep->se_ctrladdr_un.sun_path)
memset(&sep->se_ctrladdr, 0, sizeof(sep->se_ctrladdr));
sep->se_ctrladdr_un.sun_family = sep->se_family;
if ((unsz = strlcpy(sep->se_ctrladdr_un.sun_path,
sep->se_service, SUN_PATH_MAXSIZE) >= SUN_PATH_MAXSIZE)) {
syslog(LOG_ERR,
"domain socket pathname too long for service %s",
sep->se_service);
goto more;
}
memset(&sep->se_ctrladdr, 0, sizeof(sep->se_ctrladdr));
sep->se_ctrladdr_un.sun_family = sep->se_family;
sep->se_ctrladdr_un.sun_len = strlen(sep->se_service);
strcpy(sep->se_ctrladdr_un.sun_path, sep->se_service);
sep->se_ctrladdr_un.sun_len = unsz;
#undef SUN_PATH_MAXSIZE
sep->se_ctrladdr_size = SUN_LEN(&sep->se_ctrladdr_un);
}
arg = sskip(&cp);