Further simplify test KDB module authdata code

Commit 94f7c9705879500b1dc8dda8592490efce05688f simplified the
generation of authdata elements, but left behind some unnecessary
conditionalization when assembling the elements into a list, causing a
Coverity defect.  Further simplify the code.

ticket: 8868

diff --git a/src/plugins/kdb/test/kdb_test.c b/src/plugins/kdb/test/kdb_test.c
index d5b9158d06126dd31ee6c5350e9911869354e471..4f8ea786449d83b0017cc30728cddadb0984ce40 100644 (file)
--- a/src/plugins/kdb/test/kdb_test.c
+++ b/src/plugins/kdb/test/kdb_test.c
@@ -910,20 +910,22 @@ test_sign_authdata(krb5_context context, unsigned int flags,
     krb5_data **inds, d;
     int i, val;
 
+    /* Possibly create a PAC authdata element. */
     generate_pac(context, flags, client_princ, server_princ, client,
                  header_server, local_tgt, server_key, header_key,
                  local_tgt_key, authtime, ad_info, &pac_ad);
 
-    /* Add our TEST_AD_TYPE authdata */
+    /* Always create a TEST_AD_TYPE element. */
     test_ad = ealloc(sizeof(*test_ad));
     test_ad->magic = KV5M_AUTHDATA;
     test_ad->ad_type = TEST_AD_TYPE;
     test_ad->contents = (uint8_t *)estrdup("db-authdata-test");
     test_ad->length = strlen((char *)test_ad->contents);
 
+    /* Assemble the authdata into a one-element or two-element list. */
     list = ealloc(3 * sizeof(*list));
-    list[0] = (test_ad != NULL) ? test_ad : pac_ad;
-    list[1] = (test_ad != NULL) ? pac_ad : NULL;
+    list[0] = test_ad;
+    list[1] = pac_ad;
     list[2] = NULL;
     *signed_auth_data = list;