libacl: Remove unnecessary assignment
authorSimon Wilkinson <sxw@your-file-system.com>
Tue, 19 Feb 2013 15:27:58 +0000 (15:27 +0000)
committerJeffrey Altman <jaltman@your-file-system.com>
Fri, 22 Feb 2013 06:18:14 +0000 (22:18 -0800)
Don't assign i to j, when we never use j again in the function

Caught by clang-analyzer

Change-Id: I9dd2d929019ac3cc84cde4e44c4883806bd5964a
Reviewed-on: http://gerrit.openafs.org/9175
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>

src/libacl/aclprocs.c

index fd4d877..8643cae 100644 (file)
@@ -305,7 +305,6 @@ acl_Internalize_pr(int (*func)(namelist *names, idlist *ids), char *elist, struc
        }
        (*acl)->entries[i].id = lids.idlist_val[i];
     }
-    j = i;
     for (i = (*acl)->total - 1; i >= (*acl)->total - (*acl)->negative; i--) {
        if (lids.idlist_val[i] == ANONYMOUSID) {
            free(lnames.namelist_val);