mirror of
https://gitlab.com/apparmor/apparmor
synced 2025-08-28 21:07:56 +00:00
Remove unused and broken dup_value_list function
This function was broken all this time: instead of duplicating each entry in the list, it would duplicate the first entry n times. Since this function is currently not used anywhere, delete it instead of fixing it. Signed-off-by: Ryan Lee <ryan.lee@canonical.com>
This commit is contained in:
parent
f24fc4841f
commit
af5cd5002b
@ -436,7 +436,6 @@ extern void free_var_string(struct var_string *var);
|
|||||||
extern void warn_uppercase(void);
|
extern void warn_uppercase(void);
|
||||||
extern int is_blacklisted(const char *name, const char *path);
|
extern int is_blacklisted(const char *name, const char *path);
|
||||||
extern struct value_list *new_value_list(char *value);
|
extern struct value_list *new_value_list(char *value);
|
||||||
extern struct value_list *dup_value_list(struct value_list *list);
|
|
||||||
extern void free_value_list(struct value_list *list);
|
extern void free_value_list(struct value_list *list);
|
||||||
extern void print_value_list(struct value_list *list);
|
extern void print_value_list(struct value_list *list);
|
||||||
extern struct cond_entry *new_cond_entry(char *name, int eq, struct value_list *list);
|
extern struct cond_entry *new_cond_entry(char *name, int eq, struct value_list *list);
|
||||||
|
@ -1196,37 +1196,6 @@ void free_value_list(struct value_list *list)
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
struct value_list *dup_value_list(struct value_list *list)
|
|
||||||
{
|
|
||||||
struct value_list *entry, *dup, *head = NULL;
|
|
||||||
char *value;
|
|
||||||
|
|
||||||
list_for_each(list, entry) {
|
|
||||||
value = NULL;
|
|
||||||
if (list->value) {
|
|
||||||
value = strdup(list->value);
|
|
||||||
if (!value)
|
|
||||||
goto fail2;
|
|
||||||
}
|
|
||||||
dup = new_value_list(value);
|
|
||||||
if (!dup)
|
|
||||||
goto fail;
|
|
||||||
if (head)
|
|
||||||
list_append(head, dup);
|
|
||||||
else
|
|
||||||
head = dup;
|
|
||||||
}
|
|
||||||
|
|
||||||
return head;
|
|
||||||
|
|
||||||
fail:
|
|
||||||
free(value);
|
|
||||||
fail2:
|
|
||||||
free_value_list(head);
|
|
||||||
|
|
||||||
return NULL;
|
|
||||||
}
|
|
||||||
|
|
||||||
void print_value_list(struct value_list *list)
|
void print_value_list(struct value_list *list)
|
||||||
{
|
{
|
||||||
struct value_list *entry;
|
struct value_list *entry;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user