summaryrefslogtreecommitdiffstats
path: root/lib
diff options
context:
space:
mode:
authorNathan Kinder <nkinder@redhat.com>2010-09-09 15:04:53 -0700
committerNathan Kinder <nkinder@redhat.com>2010-09-09 15:04:53 -0700
commitb2799df7aeae0f034846ca647e71134b23fa1ab5 (patch)
tree8f9e2580e8726a849b7d4ce04ea5bd6e88d2e919 /lib
parent245d17832879a160eb85a7a9d784aa2ac4f2081d (diff)
downloadds-b2799df7aeae0f034846ca647e71134b23fa1ab5.tar.gz
ds-b2799df7aeae0f034846ca647e71134b23fa1ab5.tar.xz
ds-b2799df7aeae0f034846ca647e71134b23fa1ab5.zip
Bug 690090 - (cov#11974) Remove additional unused ACL functions
There are a few more unused ACL functions to remove. One of these unused functions is causing coverity to report an error about memory corruption.
Diffstat (limited to 'lib')
-rw-r--r--lib/libaccess/acltools.cpp131
1 files changed, 0 insertions, 131 deletions
diff --git a/lib/libaccess/acltools.cpp b/lib/libaccess/acltools.cpp
index 28df5999..4e1274e2 100644
--- a/lib/libaccess/acltools.cpp
+++ b/lib/libaccess/acltools.cpp
@@ -1429,137 +1429,6 @@ char *errmsg;
}
-/*
- * LOCAL FUNCTION
- *
- * Convert sub-expression to string.
- */
-
-static int
-acl_expr_string( ACLExprOp_t logical, ACLExprStack_t *expr_stack )
-{
-char **expr_text;
-char **prev_expr_text;
-char *tmp;
-
- switch (logical) {
- case ACL_EXPR_OP_NOT:
- if ( expr_stack->stack_index < 1 ) {
- printf("expression stack underflow.\n");
- return(ACLERRINTERNAL);
- }
-
- expr_text = &expr_stack->expr_text[expr_stack->stack_index - 1];
- tmp = (char *) PERM_MALLOC(strlen(*expr_text) + 7);
- if ( tmp == NULL )
- return(ACLERRNOMEM);
-
- if ( expr_stack->found_subexpression ) {
- sprintf(tmp, "not (%s)", *expr_text);
- expr_stack->found_subexpression = 0;
- expr_stack->last_subexpression = expr_stack->stack_index - 1;
- } else {
- sprintf(tmp, "not %s", *expr_text);
- }
-
- PERM_FREE(*expr_text);
- *expr_text = tmp;
- return(0);
-
- case ACL_EXPR_OP_AND:
- case ACL_EXPR_OP_OR:
- if ( expr_stack->stack_index < 2 ) {
- printf("expression stack underflow.\n");
- return(ACLERRINTERNAL);
- }
-
- expr_stack->stack_index--;
- prev_expr_text = &expr_stack->expr_text[expr_stack->stack_index];
- expr_stack->stack_index--;
- expr_text = &expr_stack->expr_text[expr_stack->stack_index];
-
- tmp = (char *) PERM_MALLOC (strlen(*expr_text)
- + strlen(*prev_expr_text) + 15);
- if ( tmp == NULL )
- return(ACLERRNOMEM);
-
- if ( expr_stack->found_subexpression &&
- expr_stack->stack_index == expr_stack->last_subexpression &&
- logical == ACL_EXPR_OP_AND ) {
- sprintf(tmp, "%s and\n (%s)", *expr_text, *prev_expr_text);
- } else if ( expr_stack->found_subexpression &&
- expr_stack->stack_index == expr_stack->last_subexpression ) {
- sprintf(tmp, "%s or\n (%s)", *expr_text, *prev_expr_text);
- } else if ( logical == ACL_EXPR_OP_AND ) {
- sprintf(tmp, "%s and\n %s", *expr_text, *prev_expr_text);
- } else {
- sprintf(tmp, "%s or\n %s", *expr_text, *prev_expr_text);
- }
-
- expr_stack->found_subexpression++;
- expr_stack->stack_index++;
- PERM_FREE(*expr_text);
- PERM_FREE(*prev_expr_text);
- *expr_text = tmp;
- *prev_expr_text = NULL;
- return(0);
-
- default:
- printf("Bad boolean logic value.\n");
- return(ACLERRINTERNAL);
- }
-
-}
-
-/*
- * LOCAL FUNCTION
- *
- * Reduce all sub-expressions to a single string.
- */
-
-static int
-acl_reduce_expr_logic( ACLExprStack_t *expr_stack, ACLExprRaw_t *expr_raw )
-{
-char **expr_text;
-char **prev_expr_text;
-char *tmp;
-
- if (expr_raw->attr_name) {
- if (expr_stack->stack_index >= ACL_EXPR_STACK ) {
- printf("expression stack overflow.");
- return(ACLERRINTERNAL);
- }
-
- if ( expr_stack->found_subexpression && expr_stack->stack_index > 0 ) {
- prev_expr_text = &expr_stack->expr_text[expr_stack->stack_index-1];
- tmp = (char *) PERM_MALLOC(strlen(*prev_expr_text) + 3);
- sprintf(tmp, "(%s)", *prev_expr_text);
- PERM_FREE(*prev_expr_text);
- *prev_expr_text = tmp;
- expr_stack->found_subexpression = 0;
- expr_stack->last_subexpression = expr_stack->stack_index - 1;
- }
-
- expr_stack->expr[expr_stack->stack_index] = expr_raw;
- expr_text = &expr_stack->expr_text[expr_stack->stack_index];
- *expr_text = (char *) PERM_MALLOC(strlen(expr_raw->attr_name)
- + strlen(expr_raw->attr_pattern)
- + 7);
- if ( *expr_text == NULL )
- return(ACLERRNOMEM);
-
- sprintf(*expr_text, "%s %s \"%s\"", expr_raw->attr_name,
- acl_comp_string(expr_raw->comparator),
- expr_raw->attr_pattern);
-
- expr_stack->stack_index++;
- expr_stack->expr_text[expr_stack->stack_index] = NULL;
- } else {
- return(acl_expr_string(expr_raw->logical, expr_stack));
- }
- return(0);
-}
-
/*
* Delete a named ACL from an ACL list