isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove bogus optimization that could lead to a double free of the

Browse Source 
group list.
This commit is contained in:
Todd C. Miller
2012-03-16 12:00:32 -04:00
parent ced34a6e26
commit 2ff9d0318c
1 changed files with 0 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
plugins/sudoers/set_perms.c
View File

@@ -132,8 +132,6 @@ set_perms(int perm)
goto bad; goto bad;
} }
ostate = &perm_stack[perm_stack_depth - 1]; ostate = &perm_stack[perm_stack_depth - 1];
if (memcmp(state, ostate, sizeof(*state)) == 0)
goto done;
} }
switch (perm) { switch (perm) {
@@ -449,8 +447,6 @@ set_perms(int perm)
goto bad; goto bad;
} }
ostate = &perm_stack[perm_stack_depth - 1]; ostate = &perm_stack[perm_stack_depth - 1];
if (memcmp(state, ostate, sizeof(*state)) == 0)
goto done;
} }
switch (perm) { switch (perm) {
@@ -816,8 +812,6 @@ set_perms(int perm)
goto bad; goto bad;
} }
ostate = &perm_stack[perm_stack_depth - 1]; ostate = &perm_stack[perm_stack_depth - 1];
if (memcmp(state, ostate, sizeof(*state)) == 0)
goto done;
} }
switch (perm) { switch (perm) {
@@ -1105,8 +1099,6 @@ set_perms(int perm)
goto bad; goto bad;
} }
ostate = &perm_stack[perm_stack_depth - 1]; ostate = &perm_stack[perm_stack_depth - 1];
if (memcmp(state, ostate, sizeof(*state)) == 0)
goto done;
} }
/* /*
@@ -1390,8 +1382,6 @@ set_perms(int perm)
goto bad; goto bad;
} }
ostate = &perm_stack[perm_stack_depth - 1]; ostate = &perm_stack[perm_stack_depth - 1];
if (memcmp(state, ostate, sizeof(*state)) == 0)
goto done;
} }
switch (perm) { switch (perm) {