mir/sudo
2
0
Fork 0
mirror of https://github.com/sudo-project/sudo.git synced 2025-08-31 06:15:37 +00:00
Code Issues Releases Wiki Activity

Use sudo_strsplit() instead of doing the equivalent manually.

Browse Source
This commit is contained in:
Todd C. Miller
2016-08-26 11:07:19 -06:00
parent c50b835255
commit 7918f7e7eb
1 changed files with 7 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
plugins/sudoers/defaults.c
View File

@@ -833,7 +833,6 @@ store_str(const char *val, struct sudo_defs_types *def)
static bool
store_list(const char *str, struct sudo_defs_types *def, int op)
{
const char *start, *end;
debug_decl(store_list, SUDOERS_DEBUG_DEFAULTS)
/* Remove all old members. */
@@ -842,20 +841,15 @@ store_list(const char *str, struct sudo_defs_types *def, int op)
/* Split str into multiple space-separated words and act on each one. */
if (str != NULL) {
end = str;
do {
/* Remove leading blanks, if nothing but blanks we are done. */
for (start = end; isblank((unsigned char)*start); start++)
;
if (*start == '\0')
break;
const char *cp, *ep;
const char *end = str + strlen(str);
const enum list_ops lop = op == '-' ? delete : add;
/* Find end position and perform operation. */
for (end = start; *end && !isblank((unsigned char)*end); end++)
;
if (!list_op(start, end - start, def, op == '-' ? delete : add))
for (cp = sudo_strsplit(str, end, " \t", &ep); cp != NULL;
cp = sudo_strsplit(NULL, end, " \t", &ep)) {
if (!list_op(cp, ep - cp, def, lop))
debug_return_bool(false);
} while (*end++ != '\0');
}
}
debug_return_bool(true);
}