The variable `cladatum` is otherwise always assigned before used, so these two assignments without a follow up usages are not needed. Found by clang-analyzer. Signed-off-by: Christian Göttsche <cgzones@xxxxxxxxxxxxxx> --- checkpolicy/checkpolicy.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/checkpolicy/checkpolicy.c b/checkpolicy/checkpolicy.c index 9627275f..c88ca542 100644 --- a/checkpolicy/checkpolicy.c +++ b/checkpolicy/checkpolicy.c @@ -1176,8 +1176,6 @@ int main(int argc, char **argv) printf("\nNo such class.\n"); break; } - cladatum = - policydb.class_val_to_struct[tclass - 1]; } else { ans[strlen(ans) - 1] = 0; cladatum = @@ -1229,8 +1227,6 @@ int main(int argc, char **argv) printf("\nNo such class.\n"); break; } - cladatum = - policydb.class_val_to_struct[tclass - 1]; } else { ans[strlen(ans) - 1] = 0; cladatum = -- 2.32.0