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

For conversion to a sudoers parse tree, ldap_entry_compare() now

Browse Source 
needs to sort in ascending order, not descending.  Bug #849
This commit is contained in:
Todd C. Miller
2018-08-30 07:48:16 -06:00
parent 58445393a7
commit ea38e9d909
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
plugins/sudoers/ldap.c
View File

@@ -1724,7 +1724,8 @@ done:
} }
/* /*
* Comparison function for ldap_entry_wrapper structures, descending order. * Comparison function for ldap_entry_wrapper structures, ascending order.
* This should match role_order_cmp() in parse_ldif.c.
*/ */
static int static int
ldap_entry_compare(const void *a, const void *b) ldap_entry_compare(const void *a, const void *b)
@@ -1733,8 +1734,8 @@ ldap_entry_compare(const void *a, const void *b)
const struct ldap_entry_wrapper *bw = b; const struct ldap_entry_wrapper *bw = b;
debug_decl(ldap_entry_compare, SUDOERS_DEBUG_LDAP) debug_decl(ldap_entry_compare, SUDOERS_DEBUG_LDAP)
debug_return_int(bw->order < aw->order ? -1 : debug_return_int(aw->order < bw->order ? -1 :
(bw->order > aw->order ? 1 : 0)); (aw->order > bw->order ? 1 : 0));
} }
/* /*