selinux: simplify duplicate_policydb_cond_list() by using kmemdup()

We can do the allocation + copying of expr.nodes in one go using
kmemdup().

Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>

diff --git a/security/selinux/ss/conditional.c b/security/selinux/ss/conditional.c
index 1ef74c085f2b0f288ca689097e38ce03be6090c7..f6dfa9c821d6ccffb375b5975478b296c89d21de 100644 (file)
--- a/security/selinux/ss/conditional.c
+++ b/security/selinux/ss/conditional.c
@@ -628,7 +628,8 @@ static int cond_dup_av_list(struct cond_av_list *new,
 static int duplicate_policydb_cond_list(struct policydb *newp,
                                        struct policydb *origp)
 {
-       int rc, i, j;
+       int rc;
+       u32 i;
 
        rc = avtab_alloc_dup(&newp->te_cond_avtab, &origp->te_cond_avtab);
        if (rc)
@@ -648,12 +649,12 @@ static int duplicate_policydb_cond_list(struct policydb *newp,
                newp->cond_list_len++;
 
                newn->cur_state = orign->cur_state;
-               newn->expr.nodes = kcalloc(orign->expr.len,
-                                       sizeof(*newn->expr.nodes), GFP_KERNEL);
+               newn->expr.nodes = kmemdup(orign->expr.nodes,
+                               orign->expr.len * sizeof(*orign->expr.nodes),
+                               GFP_KERNEL);
                if (!newn->expr.nodes)
                        goto error;
-               for (j = 0; j < orign->expr.len; j++)
-                       newn->expr.nodes[j] = orign->expr.nodes[j];
+
                newn->expr.len = orign->expr.len;
 
                rc = cond_dup_av_list(&newn->true_list, &orign->true_list,