Comment side-effect only calls of X509_check_purpose
[openssl.git] / crypto / ts / ts_rsp_sign.c
index df09e17834c29794cf97a935bb3a6f9981e274b4..0ad6f100d6db569d4d6039480191d7ee7fdb8e19 100644 (file)
@@ -1,6 +1,6 @@
-/* crypto/ts/ts_resp_sign.c */
-/* Written by Zoltan Glozik (zglozik@stones.com) for the OpenSSL
- * project 2002.
+/*
+ * Written by Zoltan Glozik (zglozik@stones.com) for the OpenSSL project
+ * 2002.
  */
 /* ====================================================================
  * Copyright (c) 2006 The OpenSSL Project.  All rights reserved.
@@ -10,7 +10,7 @@
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer. 
+ *    notice, this list of conditions and the following disclaimer.
  *
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in
  *
  */
 
-#include "cryptlib.h"
+#include "internal/cryptlib.h"
 
 #if defined(OPENSSL_SYS_UNIX)
-#include <sys/time.h>
+# include <sys/time.h>
 #endif
 
 #include <openssl/objects.h>
 #include <openssl/ts.h>
 #include <openssl/pkcs7.h>
-
-/* Private function declarations. */
+#include "ts_lcl.h"
 
 static ASN1_INTEGER *def_serial_cb(struct TS_resp_ctx *, void *);
 static int def_time_cb(struct TS_resp_ctx *, void *, long *sec, long *usec);
 static int def_extension_cb(struct TS_resp_ctx *, X509_EXTENSION *, void *);
 
-static void TS_RESP_CTX_init(TS_RESP_CTX *ctx);
-static void TS_RESP_CTX_cleanup(TS_RESP_CTX *ctx);
-static int TS_RESP_check_request(TS_RESP_CTX *ctx);
-static ASN1_OBJECT *TS_RESP_get_policy(TS_RESP_CTX *ctx);
-static TS_TST_INFO *TS_RESP_create_tst_info(TS_RESP_CTX *ctx, 
-                                           ASN1_OBJECT *policy);
-static int TS_RESP_process_extensions(TS_RESP_CTX *ctx);
-static int TS_RESP_sign(TS_RESP_CTX *ctx);
-
-static ESS_SIGNING_CERT *ESS_SIGNING_CERT_new_init(X509 *signcert, 
-                                                  STACK_OF(X509) *certs);
-static ESS_CERT_ID *ESS_CERT_ID_new_init(X509 *cert, int issuer_needed);
-static int TS_TST_INFO_content_new(PKCS7 *p7);
+static void ts_RESP_CTX_init(TS_RESP_CTX *ctx);
+static void ts_RESP_CTX_cleanup(TS_RESP_CTX *ctx);
+static int ts_RESP_check_request(TS_RESP_CTX *ctx);
+static ASN1_OBJECT *ts_RESP_get_policy(TS_RESP_CTX *ctx);
+static TS_TST_INFO *ts_RESP_create_tst_info(TS_RESP_CTX *ctx,
+                                            ASN1_OBJECT *policy);
+static int ts_RESP_process_extensions(TS_RESP_CTX *ctx);
+static int ts_RESP_sign(TS_RESP_CTX *ctx);
+
+static ESS_SIGNING_CERT *ess_SIGNING_CERT_new_init(X509 *signcert,
+                                                   STACK_OF(X509) *certs);
+static ESS_CERT_ID *ess_CERT_ID_new_init(X509 *cert, int issuer_needed);
+static int ts_TST_INFO_content_new(PKCS7 *p7);
 static int ESS_add_signing_cert(PKCS7_SIGNER_INFO *si, ESS_SIGNING_CERT *sc);
 
-static ASN1_GENERALIZEDTIME *TS_RESP_set_genTime_with_precision(
-       ASN1_GENERALIZEDTIME *, long, long, unsigned);
-
-/* Default callbacks for response generation. */
+static ASN1_GENERALIZEDTIME
+*TS_RESP_set_genTime_with_precision(ASN1_GENERALIZEDTIME *, long, long,
+                                    unsigned);
 
+/* Default callback for response generation. */
 static ASN1_INTEGER *def_serial_cb(struct TS_resp_ctx *ctx, void *data)
-       {
-       ASN1_INTEGER *serial = ASN1_INTEGER_new();
-       if (!serial) goto err;
-       if (!ASN1_INTEGER_set(serial, 1)) goto err;
-       return serial;
+{
+    ASN1_INTEGER *serial = ASN1_INTEGER_new();
+
+    if (serial == NULL)
+        goto err;
+    if (!ASN1_INTEGER_set(serial, 1))
+        goto err;
+    return serial;
+
  err:
-       TSerr(TS_F_DEF_SERIAL_CB, ERR_R_MALLOC_FAILURE);
-       TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                   "Error during serial number generation.");
-       return NULL;
-       }
+    TSerr(TS_F_DEF_SERIAL_CB, ERR_R_MALLOC_FAILURE);
+    TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                "Error during serial number generation.");
+    return NULL;
+}
 
 #if defined(OPENSSL_SYS_UNIX)
 
-/* Use the gettimeofday function call. */
-static int def_time_cb(struct TS_resp_ctx *ctx, void *data, 
-                      long *sec, long *usec)
-       {
-       struct timeval tv;
-       if (gettimeofday(&tv, NULL) != 0) 
-               {
-               TSerr(TS_F_DEF_TIME_CB, TS_R_TIME_SYSCALL_ERROR);
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Time is not available.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_TIME_NOT_AVAILABLE);
-               return 0;
-               }
-       /* Return time to caller. */
-       *sec = tv.tv_sec;
-       *usec = tv.tv_usec;
-
-       return 1;
-       }
+static int def_time_cb(struct TS_resp_ctx *ctx, void *data,
+                       long *sec, long *usec)
+{
+    struct timeval tv;
+    if (gettimeofday(&tv, NULL) != 0) {
+        TSerr(TS_F_DEF_TIME_CB, TS_R_TIME_SYSCALL_ERROR);
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Time is not available.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_TIME_NOT_AVAILABLE);
+        return 0;
+    }
+    *sec = tv.tv_sec;
+    *usec = tv.tv_usec;
+
+    return 1;
+}
 
 #else
 
-/* Use the time function call that provides only seconds precision. */
-static int def_time_cb(struct TS_resp_ctx *ctx, void *data, 
-                      long *sec, long *usec)
-       {
-       time_t t;
-       if (time(&t) == (time_t) -1)
-               {
-               TSerr(TS_F_DEF_TIME_CB, TS_R_TIME_SYSCALL_ERROR);
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Time is not available.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_TIME_NOT_AVAILABLE);
-               return 0;
-               }
-       /* Return time to caller, only second precision. */
-       *sec = (long) t;
-       *usec = 0;
-
-       return 1;
-       }
+static int def_time_cb(struct TS_resp_ctx *ctx, void *data,
+                       long *sec, long *usec)
+{
+    time_t t;
+    if (time(&t) == (time_t)-1) {
+        TSerr(TS_F_DEF_TIME_CB, TS_R_TIME_SYSCALL_ERROR);
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Time is not available.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_TIME_NOT_AVAILABLE);
+        return 0;
+    }
+    *sec = (long)t;
+    *usec = 0;
+
+    return 1;
+}
 
 #endif
 
 static int def_extension_cb(struct TS_resp_ctx *ctx, X509_EXTENSION *ext,
-                           void *data)
-       {
-       /* No extensions are processed here. */
-       TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                   "Unsupported extension.");
-       TS_RESP_CTX_add_failure_info(ctx, TS_INFO_UNACCEPTED_EXTENSION);
-       return 0;
-       }
+                            void *data)
+{
+    TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                "Unsupported extension.");
+    TS_RESP_CTX_add_failure_info(ctx, TS_INFO_UNACCEPTED_EXTENSION);
+    return 0;
+}
 
 /* TS_RESP_CTX management functions. */
 
 TS_RESP_CTX *TS_RESP_CTX_new()
-       {
-       TS_RESP_CTX *ctx;
+{
+    TS_RESP_CTX *ctx;
+
+    if ((ctx = OPENSSL_zalloc(sizeof(*ctx))) == NULL) {
+        TSerr(TS_F_TS_RESP_CTX_NEW, ERR_R_MALLOC_FAILURE);
+        return NULL;
+    }
 
-       if (!(ctx = (TS_RESP_CTX *) OPENSSL_malloc(sizeof(TS_RESP_CTX))))
-               {
-               TSerr(TS_F_TS_RESP_CTX_NEW, ERR_R_MALLOC_FAILURE);
-               return NULL;
-               }
-       memset(ctx, 0, sizeof(TS_RESP_CTX));
+    ctx->signer_md = EVP_sha256();
 
-       /* Setting default callbacks. */
-       ctx->serial_cb = def_serial_cb;
-       ctx->time_cb = def_time_cb;
-       ctx->extension_cb = def_extension_cb;
+    ctx->serial_cb = def_serial_cb;
+    ctx->time_cb = def_time_cb;
+    ctx->extension_cb = def_extension_cb;
 
-       return ctx;
-       }
+    return ctx;
+}
 
 void TS_RESP_CTX_free(TS_RESP_CTX *ctx)
-       {
-       if (!ctx) return;
-
-       X509_free(ctx->signer_cert);
-       EVP_PKEY_free(ctx->signer_key);
-       sk_X509_pop_free(ctx->certs, X509_free);
-       sk_ASN1_OBJECT_pop_free(ctx->policies, ASN1_OBJECT_free);
-       ASN1_OBJECT_free(ctx->default_policy);
-       sk_EVP_MD_free(ctx->mds);       /* No EVP_MD_free method exists. */
-       ASN1_INTEGER_free(ctx->seconds);
-       ASN1_INTEGER_free(ctx->millis);
-       ASN1_INTEGER_free(ctx->micros);
-       OPENSSL_free(ctx);
-       }
+{
+    if (!ctx)
+        return;
+
+    X509_free(ctx->signer_cert);
+    EVP_PKEY_free(ctx->signer_key);
+    sk_X509_pop_free(ctx->certs, X509_free);
+    sk_ASN1_OBJECT_pop_free(ctx->policies, ASN1_OBJECT_free);
+    ASN1_OBJECT_free(ctx->default_policy);
+    sk_EVP_MD_free(ctx->mds);   /* No EVP_MD_free method exists. */
+    ASN1_INTEGER_free(ctx->seconds);
+    ASN1_INTEGER_free(ctx->millis);
+    ASN1_INTEGER_free(ctx->micros);
+    OPENSSL_free(ctx);
+}
 
 int TS_RESP_CTX_set_signer_cert(TS_RESP_CTX *ctx, X509 *signer)
-       {
-       if (X509_check_purpose(signer, X509_PURPOSE_TIMESTAMP_SIGN, 0) != 1)
-               {
-               TSerr(TS_F_TS_RESP_CTX_SET_SIGNER_CERT, 
-                     TS_R_INVALID_SIGNER_CERTIFICATE_PURPOSE);
-               return 0;
-               }
-       if (ctx->signer_cert) X509_free(ctx->signer_cert);
-       ctx->signer_cert = signer;
-       CRYPTO_add(&ctx->signer_cert->references, +1, CRYPTO_LOCK_X509);
-       return 1;
-       }
+{
+    if (X509_check_purpose(signer, X509_PURPOSE_TIMESTAMP_SIGN, 0) != 1) {
+        TSerr(TS_F_TS_RESP_CTX_SET_SIGNER_CERT,
+              TS_R_INVALID_SIGNER_CERTIFICATE_PURPOSE);
+        return 0;
+    }
+    X509_free(ctx->signer_cert);
+    ctx->signer_cert = signer;
+    X509_up_ref(ctx->signer_cert);
+    return 1;
+}
 
 int TS_RESP_CTX_set_signer_key(TS_RESP_CTX *ctx, EVP_PKEY *key)
-       {
-       if (ctx->signer_key) EVP_PKEY_free(ctx->signer_key);
-       ctx->signer_key = key;
-       CRYPTO_add(&ctx->signer_key->references, +1, CRYPTO_LOCK_EVP_PKEY);
+{
+    EVP_PKEY_free(ctx->signer_key);
+    ctx->signer_key = key;
+    EVP_PKEY_up_ref(ctx->signer_key);
+
+    return 1;
+}
 
-       return 1;
-       }
+int TS_RESP_CTX_set_signer_digest(TS_RESP_CTX *ctx, const EVP_MD *md)
+{
+    ctx->signer_md = md;
+    return 1;
+}
 
 int TS_RESP_CTX_set_def_policy(TS_RESP_CTX *ctx, ASN1_OBJECT *def_policy)
-       {
-       if (ctx->default_policy) ASN1_OBJECT_free(ctx->default_policy);
-       if (!(ctx->default_policy = OBJ_dup(def_policy))) goto err;
-       return 1;
+{
+    ASN1_OBJECT_free(ctx->default_policy);
+    if ((ctx->default_policy = OBJ_dup(def_policy)) == NULL)
+        goto err;
+    return 1;
  err:
-       TSerr(TS_F_TS_RESP_CTX_SET_DEF_POLICY, ERR_R_MALLOC_FAILURE);
-       return 0;
-       }
+    TSerr(TS_F_TS_RESP_CTX_SET_DEF_POLICY, ERR_R_MALLOC_FAILURE);
+    return 0;
+}
 
 int TS_RESP_CTX_set_certs(TS_RESP_CTX *ctx, STACK_OF(X509) *certs)
-       {
-
-       if (ctx->certs)
-               {
-               sk_X509_pop_free(ctx->certs, X509_free);
-               ctx->certs = NULL;
-               }
-       if (!certs) return 1;
-       if (!(ctx->certs = X509_chain_up_ref(certs))) 
-               {
-               TSerr(TS_F_TS_RESP_CTX_SET_CERTS, ERR_R_MALLOC_FAILURE);
-               return 0;
-               }
-
-       return 1;
-       }
+{
 
-int TS_RESP_CTX_add_policy(TS_RESP_CTX *ctx, ASN1_OBJECT *policy)
-       {
-       ASN1_OBJECT *copy = NULL;
+    sk_X509_pop_free(ctx->certs, X509_free);
+    ctx->certs = NULL;
+    if (!certs)
+        return 1;
+    if ((ctx->certs = X509_chain_up_ref(certs)) == NULL) {
+        TSerr(TS_F_TS_RESP_CTX_SET_CERTS, ERR_R_MALLOC_FAILURE);
+        return 0;
+    }
 
-       /* Create new policy stack if necessary. */
-       if (!ctx->policies && !(ctx->policies = sk_ASN1_OBJECT_new_null())) 
-               goto err;
-       if (!(copy = OBJ_dup(policy))) goto err;
-       if (!sk_ASN1_OBJECT_push(ctx->policies, copy)) goto err;
+    return 1;
+}
 
-       return 1;
+int TS_RESP_CTX_add_policy(TS_RESP_CTX *ctx, ASN1_OBJECT *policy)
+{
+    ASN1_OBJECT *copy = NULL;
+
+    if (ctx->policies == NULL
+        && (ctx->policies = sk_ASN1_OBJECT_new_null()) == NULL)
+        goto err;
+    if ((copy = OBJ_dup(policy)) == NULL)
+        goto err;
+    if (!sk_ASN1_OBJECT_push(ctx->policies, copy))
+        goto err;
+
+    return 1;
  err:
-       TSerr(TS_F_TS_RESP_CTX_ADD_POLICY, ERR_R_MALLOC_FAILURE);
-       ASN1_OBJECT_free(copy);
-       return 0;
-       }
+    TSerr(TS_F_TS_RESP_CTX_ADD_POLICY, ERR_R_MALLOC_FAILURE);
+    ASN1_OBJECT_free(copy);
+    return 0;
+}
 
 int TS_RESP_CTX_add_md(TS_RESP_CTX *ctx, const EVP_MD *md)
-       {
-       /* Create new md stack if necessary. */
-       if (!ctx->mds && !(ctx->mds = sk_EVP_MD_new_null())) 
-               goto err;
-       /* Add the shared md, no copy needed. */
-       if (!sk_EVP_MD_push(ctx->mds, (EVP_MD *)md)) goto err;
-
-       return 1;
+{
+    if (ctx->mds == NULL
+        && (ctx->mds = sk_EVP_MD_new_null()) == NULL)
+        goto err;
+    if (!sk_EVP_MD_push(ctx->mds, (EVP_MD *)md))
+        goto err;
+
+    return 1;
  err:
-       TSerr(TS_F_TS_RESP_CTX_ADD_MD, ERR_R_MALLOC_FAILURE);
-       return 0;
-       }
-
-#define TS_RESP_CTX_accuracy_free(ctx)         \
-       ASN1_INTEGER_free(ctx->seconds);        \
-       ctx->seconds = NULL;                    \
-       ASN1_INTEGER_free(ctx->millis);         \
-       ctx->millis = NULL;                     \
-       ASN1_INTEGER_free(ctx->micros);         \
-       ctx->micros = NULL;
-
-int TS_RESP_CTX_set_accuracy(TS_RESP_CTX *ctx, 
-                            int secs, int millis, int micros)
-       {
-
-       TS_RESP_CTX_accuracy_free(ctx);
-       if (secs && (!(ctx->seconds = ASN1_INTEGER_new())
-                    || !ASN1_INTEGER_set(ctx->seconds, secs)))
-               goto err;
-       if (millis && (!(ctx->millis = ASN1_INTEGER_new())
-                      || !ASN1_INTEGER_set(ctx->millis, millis)))
-               goto err;
-       if (micros && (!(ctx->micros = ASN1_INTEGER_new())
-                      || !ASN1_INTEGER_set(ctx->micros, micros)))
-               goto err;
-
-       return 1;
+    TSerr(TS_F_TS_RESP_CTX_ADD_MD, ERR_R_MALLOC_FAILURE);
+    return 0;
+}
+
+#define TS_RESP_CTX_accuracy_free(ctx)          \
+        ASN1_INTEGER_free(ctx->seconds);        \
+        ctx->seconds = NULL;                    \
+        ASN1_INTEGER_free(ctx->millis);         \
+        ctx->millis = NULL;                     \
+        ASN1_INTEGER_free(ctx->micros);         \
+        ctx->micros = NULL;
+
+int TS_RESP_CTX_set_accuracy(TS_RESP_CTX *ctx,
+                             int secs, int millis, int micros)
+{
+
+    TS_RESP_CTX_accuracy_free(ctx);
+    if (secs
+        && ((ctx->seconds = ASN1_INTEGER_new()) == NULL
+            || !ASN1_INTEGER_set(ctx->seconds, secs)))
+        goto err;
+    if (millis
+        && ((ctx->millis = ASN1_INTEGER_new()) == NULL
+            || !ASN1_INTEGER_set(ctx->millis, millis)))
+        goto err;
+    if (micros
+        && ((ctx->micros = ASN1_INTEGER_new()) == NULL
+            || !ASN1_INTEGER_set(ctx->micros, micros)))
+        goto err;
+
+    return 1;
  err:
-       TS_RESP_CTX_accuracy_free(ctx);
-       TSerr(TS_F_TS_RESP_CTX_SET_ACCURACY, ERR_R_MALLOC_FAILURE);
-       return 0;
-       }
+    TS_RESP_CTX_accuracy_free(ctx);
+    TSerr(TS_F_TS_RESP_CTX_SET_ACCURACY, ERR_R_MALLOC_FAILURE);
+    return 0;
+}
 
 void TS_RESP_CTX_add_flags(TS_RESP_CTX *ctx, int flags)
-       {
-       ctx->flags |= flags;
-       }
+{
+    ctx->flags |= flags;
+}
 
 void TS_RESP_CTX_set_serial_cb(TS_RESP_CTX *ctx, TS_serial_cb cb, void *data)
-       {
-       ctx->serial_cb = cb;
-       ctx->serial_cb_data = data;
-       }
+{
+    ctx->serial_cb = cb;
+    ctx->serial_cb_data = data;
+}
 
 void TS_RESP_CTX_set_time_cb(TS_RESP_CTX *ctx, TS_time_cb cb, void *data)
-       {
-       ctx->time_cb = cb;
-       ctx->time_cb_data = data;
-       }
-
-void TS_RESP_CTX_set_extension_cb(TS_RESP_CTX *ctx, 
-                                 TS_extension_cb cb, void *data)
-       {
-       ctx->extension_cb = cb;
-       ctx->extension_cb_data = data;
-       }
-
-int TS_RESP_CTX_set_status_info(TS_RESP_CTX *ctx, 
-                               int status, const char *text)
-       {
-       TS_STATUS_INFO *si = NULL;
-       ASN1_UTF8STRING *utf8_text = NULL;
-       int ret = 0;
-
-       if (!(si = TS_STATUS_INFO_new())) goto err;
-       if (!ASN1_INTEGER_set(si->status, status)) goto err;
-       if (text)
-               {
-               if (!(utf8_text = ASN1_UTF8STRING_new())
-                   || !ASN1_STRING_set(utf8_text, text, strlen(text)))
-                       goto err;
-               if (!si->text && !(si->text = sk_ASN1_UTF8STRING_new_null()))
-                       goto err;
-               if (!sk_ASN1_UTF8STRING_push(si->text, utf8_text)) goto err;
-               utf8_text = NULL;       /* Ownership is lost. */
-               }
-       if (!TS_RESP_set_status_info(ctx->response, si)) goto err;
-       ret = 1;
+{
+    ctx->time_cb = cb;
+    ctx->time_cb_data = data;
+}
+
+void TS_RESP_CTX_set_extension_cb(TS_RESP_CTX *ctx,
+                                  TS_extension_cb cb, void *data)
+{
+    ctx->extension_cb = cb;
+    ctx->extension_cb_data = data;
+}
+
+int TS_RESP_CTX_set_status_info(TS_RESP_CTX *ctx,
+                                int status, const char *text)
+{
+    TS_STATUS_INFO *si = NULL;
+    ASN1_UTF8STRING *utf8_text = NULL;
+    int ret = 0;
+
+    if ((si = TS_STATUS_INFO_new()) == NULL)
+        goto err;
+    if (!ASN1_INTEGER_set(si->status, status))
+        goto err;
+    if (text) {
+        if ((utf8_text = ASN1_UTF8STRING_new()) == NULL
+            || !ASN1_STRING_set(utf8_text, text, strlen(text)))
+            goto err;
+        if (si->text == NULL
+            && (si->text = sk_ASN1_UTF8STRING_new_null()) == NULL)
+            goto err;
+        if (!sk_ASN1_UTF8STRING_push(si->text, utf8_text))
+            goto err;
+        utf8_text = NULL;       /* Ownership is lost. */
+    }
+    if (!TS_RESP_set_status_info(ctx->response, si))
+        goto err;
+    ret = 1;
  err:
-       if (!ret)
-               TSerr(TS_F_TS_RESP_CTX_SET_STATUS_INFO, ERR_R_MALLOC_FAILURE);
-       TS_STATUS_INFO_free(si);
-       ASN1_UTF8STRING_free(utf8_text);
-       return ret;
-       }
-
-int TS_RESP_CTX_set_status_info_cond(TS_RESP_CTX *ctx, 
-                                    int status, const char *text)
-       {
-       int ret = 1;
-       TS_STATUS_INFO *si = TS_RESP_get_status_info(ctx->response);
-
-       if (ASN1_INTEGER_get(si->status) == TS_STATUS_GRANTED)
-               {
-               /* Status has not been set, set it now. */
-               ret = TS_RESP_CTX_set_status_info(ctx, status, text);
-               }
-       return ret;
-       }
+    if (!ret)
+        TSerr(TS_F_TS_RESP_CTX_SET_STATUS_INFO, ERR_R_MALLOC_FAILURE);
+    TS_STATUS_INFO_free(si);
+    ASN1_UTF8STRING_free(utf8_text);
+    return ret;
+}
+
+int TS_RESP_CTX_set_status_info_cond(TS_RESP_CTX *ctx,
+                                     int status, const char *text)
+{
+    int ret = 1;
+    TS_STATUS_INFO *si = ctx->response->status_info;
+
+    if (ASN1_INTEGER_get(si->status) == TS_STATUS_GRANTED) {
+        ret = TS_RESP_CTX_set_status_info(ctx, status, text);
+    }
+    return ret;
+}
 
 int TS_RESP_CTX_add_failure_info(TS_RESP_CTX *ctx, int failure)
-       {
-       TS_STATUS_INFO *si = TS_RESP_get_status_info(ctx->response);
-       if (!si->failure_info && !(si->failure_info = ASN1_BIT_STRING_new()))
-               goto err;
-       if (!ASN1_BIT_STRING_set_bit(si->failure_info, failure, 1))
-               goto err;
-       return 1;
+{
+    TS_STATUS_INFO *si = ctx->response->status_info;
+    if (si->failure_info == NULL
+        && (si->failure_info = ASN1_BIT_STRING_new()) == NULL)
+        goto err;
+    if (!ASN1_BIT_STRING_set_bit(si->failure_info, failure, 1))
+        goto err;
+    return 1;
  err:
-       TSerr(TS_F_TS_RESP_CTX_ADD_FAILURE_INFO, ERR_R_MALLOC_FAILURE);
-       return 0;
-       }
+    TSerr(TS_F_TS_RESP_CTX_ADD_FAILURE_INFO, ERR_R_MALLOC_FAILURE);
+    return 0;
+}
 
 TS_REQ *TS_RESP_CTX_get_request(TS_RESP_CTX *ctx)
-       {
-       return ctx->request;
-       }
+{
+    return ctx->request;
+}
 
 TS_TST_INFO *TS_RESP_CTX_get_tst_info(TS_RESP_CTX *ctx)
-       {
-       return ctx->tst_info;
-       }
-
-int TS_RESP_CTX_set_clock_precision_digits(TS_RESP_CTX *ctx, unsigned precision)
-       {
-       if (precision > TS_MAX_CLOCK_PRECISION_DIGITS)
-              return 0;
-       ctx->clock_precision_digits = precision;
-       return 1;
-       }
+{
+    return ctx->tst_info;
+}
+
+int TS_RESP_CTX_set_clock_precision_digits(TS_RESP_CTX *ctx,
+                                           unsigned precision)
+{
+    if (precision > TS_MAX_CLOCK_PRECISION_DIGITS)
+        return 0;
+    ctx->clock_precision_digits = precision;
+    return 1;
+}
 
 /* Main entry method of the response generation. */
 TS_RESP *TS_RESP_create_response(TS_RESP_CTX *ctx, BIO *req_bio)
-       {
-       ASN1_OBJECT *policy;
-       TS_RESP *response;
-       int result = 0;
-
-       TS_RESP_CTX_init(ctx);
-
-       /* Creating the response object. */
-       if (!(ctx->response = TS_RESP_new())) 
-               {
-               TSerr(TS_F_TS_RESP_CREATE_RESPONSE, ERR_R_MALLOC_FAILURE);
-               goto end;
-               }
-
-       /* Parsing DER request. */
-       if (!(ctx->request = d2i_TS_REQ_bio(req_bio, NULL)))
-               {
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Bad request format or "
-                                           "system error.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_DATA_FORMAT);
-               goto end;
-               }
-
-       /* Setting default status info. */
-       if (!TS_RESP_CTX_set_status_info(ctx, TS_STATUS_GRANTED, NULL))
-               goto end;
-
-       /* Checking the request format. */
-       if (!TS_RESP_check_request(ctx)) goto end;
-
-       /* Checking acceptable policies. */
-       if (!(policy = TS_RESP_get_policy(ctx))) goto end;
-
-       /* Creating the TS_TST_INFO object. */
-       if (!(ctx->tst_info = TS_RESP_create_tst_info(ctx, policy)))
-               goto end;
-
-       /* Processing extensions. */
-       if (!TS_RESP_process_extensions(ctx)) goto end;
-
-       /* Generating the signature. */
-       if (!TS_RESP_sign(ctx)) goto end;
-
-       /* Everything was successful. */
-       result = 1;
+{
+    ASN1_OBJECT *policy;
+    TS_RESP *response;
+    int result = 0;
+
+    ts_RESP_CTX_init(ctx);
+
+    if ((ctx->response = TS_RESP_new()) == NULL) {
+        TSerr(TS_F_TS_RESP_CREATE_RESPONSE, ERR_R_MALLOC_FAILURE);
+        goto end;
+    }
+    if ((ctx->request = d2i_TS_REQ_bio(req_bio, NULL)) == NULL) {
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Bad request format or system error.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_DATA_FORMAT);
+        goto end;
+    }
+    if (!TS_RESP_CTX_set_status_info(ctx, TS_STATUS_GRANTED, NULL))
+        goto end;
+    if (!ts_RESP_check_request(ctx))
+        goto end;
+    if ((policy = ts_RESP_get_policy(ctx)) == NULL)
+        goto end;
+    if ((ctx->tst_info = ts_RESP_create_tst_info(ctx, policy)) == NULL)
+        goto end;
+    if (!ts_RESP_process_extensions(ctx))
+        goto end;
+    if (!ts_RESP_sign(ctx))
+        goto end;
+    result = 1;
+
  end:
-       if (!result)
-               {
-               TSerr(TS_F_TS_RESP_CREATE_RESPONSE, TS_R_RESPONSE_SETUP_ERROR);
-               if (ctx->response != NULL)
-                       {
-                       if (TS_RESP_CTX_set_status_info_cond(ctx,
-                               TS_STATUS_REJECTION, "Error during response "
-                               "generation.") == 0)
-                               {
-                               TS_RESP_free(ctx->response);
-                               ctx->response = NULL;
-                               }
-                       }
-               }
-       response = ctx->response;
-       ctx->response = NULL;   /* Ownership will be returned to caller. */
-       TS_RESP_CTX_cleanup(ctx);
-       return response;
-       }
+    if (!result) {
+        TSerr(TS_F_TS_RESP_CREATE_RESPONSE, TS_R_RESPONSE_SETUP_ERROR);
+        if (ctx->response != NULL) {
+            if (TS_RESP_CTX_set_status_info_cond(ctx,
+                                                 TS_STATUS_REJECTION,
+                                                 "Error during response "
+                                                 "generation.") == 0) {
+                TS_RESP_free(ctx->response);
+                ctx->response = NULL;
+            }
+        }
+    }
+    response = ctx->response;
+    ctx->response = NULL;       /* Ownership will be returned to caller. */
+    ts_RESP_CTX_cleanup(ctx);
+    return response;
+}
 
 /* Initializes the variable part of the context. */
-static void TS_RESP_CTX_init(TS_RESP_CTX *ctx)
-       {
-       ctx->request = NULL;
-       ctx->response = NULL;
-       ctx->tst_info = NULL;
-       }
+static void ts_RESP_CTX_init(TS_RESP_CTX *ctx)
+{
+    ctx->request = NULL;
+    ctx->response = NULL;
+    ctx->tst_info = NULL;
+}
 
 /* Cleans up the variable part of the context. */
-static void TS_RESP_CTX_cleanup(TS_RESP_CTX *ctx)
-       {
-       TS_REQ_free(ctx->request);
-       ctx->request = NULL;
-       TS_RESP_free(ctx->response);
-       ctx->response = NULL;
-       TS_TST_INFO_free(ctx->tst_info);
-       ctx->tst_info = NULL;
-       }
+static void ts_RESP_CTX_cleanup(TS_RESP_CTX *ctx)
+{
+    TS_REQ_free(ctx->request);
+    ctx->request = NULL;
+    TS_RESP_free(ctx->response);
+    ctx->response = NULL;
+    TS_TST_INFO_free(ctx->tst_info);
+    ctx->tst_info = NULL;
+}
 
 /* Checks the format and content of the request. */
-static int TS_RESP_check_request(TS_RESP_CTX *ctx)
-       {
-       TS_REQ *request = ctx->request;
-       TS_MSG_IMPRINT *msg_imprint;
-       X509_ALGOR *md_alg;
-       int md_alg_id;
-       const ASN1_OCTET_STRING *digest;
-       EVP_MD *md = NULL;
-       int i;
-
-       /* Checking request version. */
-       if (TS_REQ_get_version(request) != 1)
-               {
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Bad request version.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_REQUEST);
-               return 0;
-               }
-
-       /* Checking message digest algorithm. */
-       msg_imprint = TS_REQ_get_msg_imprint(request);
-       md_alg = TS_MSG_IMPRINT_get_algo(msg_imprint);
-       md_alg_id = OBJ_obj2nid(md_alg->algorithm);
-       for (i = 0; !md && i < sk_EVP_MD_num(ctx->mds); ++i)
-               {
-               EVP_MD *current_md = sk_EVP_MD_value(ctx->mds, i);
-               if (md_alg_id == EVP_MD_type(current_md))
-                       md = current_md;
-               }
-       if (!md)
-               {
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Message digest algorithm is "
-                                           "not supported.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_ALG);
-               return 0;
-               }
-
-       /* No message digest takes parameter. */
-       if (md_alg->parameter 
-           && ASN1_TYPE_get(md_alg->parameter) != V_ASN1_NULL)
-               {
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Superfluous message digest "
-                                           "parameter.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_ALG);
-               return 0;
-               }
-       /* Checking message digest size. */
-       digest = TS_MSG_IMPRINT_get_msg(msg_imprint);
-       if (digest->length != EVP_MD_size(md))
-               {
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Bad message digest.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_DATA_FORMAT);
-               return 0;
-               }
-
-       return 1;
-       }
+static int ts_RESP_check_request(TS_RESP_CTX *ctx)
+{
+    TS_REQ *request = ctx->request;
+    TS_MSG_IMPRINT *msg_imprint;
+    X509_ALGOR *md_alg;
+    int md_alg_id;
+    const ASN1_OCTET_STRING *digest;
+    EVP_MD *md = NULL;
+    int i;
+
+    if (TS_REQ_get_version(request) != 1) {
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Bad request version.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_REQUEST);
+        return 0;
+    }
+
+    msg_imprint = request->msg_imprint;
+    md_alg = msg_imprint->hash_algo;
+    md_alg_id = OBJ_obj2nid(md_alg->algorithm);
+    for (i = 0; !md && i < sk_EVP_MD_num(ctx->mds); ++i) {
+        EVP_MD *current_md = sk_EVP_MD_value(ctx->mds, i);
+        if (md_alg_id == EVP_MD_type(current_md))
+            md = current_md;
+    }
+    if (!md) {
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Message digest algorithm is "
+                                    "not supported.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_ALG);
+        return 0;
+    }
+
+    if (md_alg->parameter && ASN1_TYPE_get(md_alg->parameter) != V_ASN1_NULL) {
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Superfluous message digest "
+                                    "parameter.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_ALG);
+        return 0;
+    }
+    digest = msg_imprint->hashed_msg;
+    if (digest->length != EVP_MD_size(md)) {
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Bad message digest.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_BAD_DATA_FORMAT);
+        return 0;
+    }
+
+    return 1;
+}
 
 /* Returns the TSA policy based on the requested and acceptable policies. */
-static ASN1_OBJECT *TS_RESP_get_policy(TS_RESP_CTX *ctx)
-       {
-       ASN1_OBJECT *requested = TS_REQ_get_policy_id(ctx->request);
-       ASN1_OBJECT *policy = NULL;
-       int i;
-
-       if (ctx->default_policy == NULL)
-               {
-               TSerr(TS_F_TS_RESP_GET_POLICY, TS_R_INVALID_NULL_POINTER);
-               return NULL;
-               }
-       /* Return the default policy if none is requested or the default is
-          requested. */
-       if (!requested || !OBJ_cmp(requested, ctx->default_policy))
-               policy = ctx->default_policy;
-
-       /* Check if the policy is acceptable. */
-       for (i = 0; !policy && i < sk_ASN1_OBJECT_num(ctx->policies); ++i)
-               {
-               ASN1_OBJECT *current = sk_ASN1_OBJECT_value(ctx->policies, i);
-               if (!OBJ_cmp(requested, current))
-                       policy = current;
-               }
-       if (!policy)
-               {
-               TSerr(TS_F_TS_RESP_GET_POLICY, TS_R_UNACCEPTABLE_POLICY);
-               TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
-                                           "Requested policy is not "
-                                           "supported.");
-               TS_RESP_CTX_add_failure_info(ctx, TS_INFO_UNACCEPTED_POLICY);
-               }
-       return policy;
-       }
+static ASN1_OBJECT *ts_RESP_get_policy(TS_RESP_CTX *ctx)
+{
+    ASN1_OBJECT *requested = ctx->request->policy_id;
+    ASN1_OBJECT *policy = NULL;
+    int i;
+
+    if (ctx->default_policy == NULL) {
+        TSerr(TS_F_TS_RESP_GET_POLICY, TS_R_INVALID_NULL_POINTER);
+        return NULL;
+    }
+    if (!requested || !OBJ_cmp(requested, ctx->default_policy))
+        policy = ctx->default_policy;
+
+    /* Check if the policy is acceptable. */
+    for (i = 0; !policy && i < sk_ASN1_OBJECT_num(ctx->policies); ++i) {
+        ASN1_OBJECT *current = sk_ASN1_OBJECT_value(ctx->policies, i);
+        if (!OBJ_cmp(requested, current))
+            policy = current;
+    }
+    if (!policy) {
+        TSerr(TS_F_TS_RESP_GET_POLICY, TS_R_UNACCEPTABLE_POLICY);
+        TS_RESP_CTX_set_status_info(ctx, TS_STATUS_REJECTION,
+                                    "Requested policy is not " "supported.");
+        TS_RESP_CTX_add_failure_info(ctx, TS_INFO_UNACCEPTED_POLICY);
+    }
+    return policy;
+}
 
 /* Creates the TS_TST_INFO object based on the settings of the context. */
-static TS_TST_INFO *TS_RESP_create_tst_info(TS_RESP_CTX *ctx,
-                                           ASN1_OBJECT *policy)
-       {
-       int result = 0;
-       TS_TST_INFO *tst_info = NULL;
-       ASN1_INTEGER *serial = NULL;
-       ASN1_GENERALIZEDTIME *asn1_time = NULL;
-       long sec, usec;
-       TS_ACCURACY *accuracy = NULL;
-       const ASN1_INTEGER *nonce;
-       GENERAL_NAME *tsa_name = NULL;
-
-       if (!(tst_info = TS_TST_INFO_new())) goto end;
-       if (!TS_TST_INFO_set_version(tst_info, 1)) goto end;
-       if (!TS_TST_INFO_set_policy_id(tst_info, policy)) goto end;
-       if (!TS_TST_INFO_set_msg_imprint(tst_info, ctx->request->msg_imprint))
-               goto end;
-       if (!(serial = (*ctx->serial_cb)(ctx, ctx->serial_cb_data))
-           || !TS_TST_INFO_set_serial(tst_info, serial))
-               goto end;
-       if (!(*ctx->time_cb)(ctx, ctx->time_cb_data, &sec, &usec)
-            || !(asn1_time = TS_RESP_set_genTime_with_precision(NULL, 
-                                       sec, usec, 
-                                       ctx->clock_precision_digits))
-           || !TS_TST_INFO_set_time(tst_info, asn1_time))
-               goto end;
-
-       /* Setting accuracy if needed. */
-       if ((ctx->seconds || ctx->millis || ctx->micros) 
-           && !(accuracy = TS_ACCURACY_new()))
-               goto end;
-
-       if (ctx->seconds && !TS_ACCURACY_set_seconds(accuracy, ctx->seconds))
-               goto end;
-       if (ctx->millis && !TS_ACCURACY_set_millis(accuracy, ctx->millis))
-               goto end;
-       if (ctx->micros && !TS_ACCURACY_set_micros(accuracy, ctx->micros))
-               goto end;
-       if (accuracy && !TS_TST_INFO_set_accuracy(tst_info, accuracy)) 
-               goto end;
-
-       /* Setting ordering. */
-       if ((ctx->flags & TS_ORDERING) 
-           && !TS_TST_INFO_set_ordering(tst_info, 1))
-               goto end;
-       
-       /* Setting nonce if needed. */
-       if ((nonce = TS_REQ_get_nonce(ctx->request)) != NULL
-           && !TS_TST_INFO_set_nonce(tst_info, nonce))
-               goto end;
-
-       /* Setting TSA name to subject of signer certificate. */
-       if (ctx->flags & TS_TSA_NAME)
-               {
-               if (!(tsa_name = GENERAL_NAME_new())) goto end;
-               tsa_name->type = GEN_DIRNAME;
-               tsa_name->d.dirn = 
-                       X509_NAME_dup(ctx->signer_cert->cert_info->subject);
-               if (!tsa_name->d.dirn) goto end;
-               if (!TS_TST_INFO_set_tsa(tst_info, tsa_name)) goto end;
-               }
-
-       result = 1;
+static TS_TST_INFO *ts_RESP_create_tst_info(TS_RESP_CTX *ctx,
+                                            ASN1_OBJECT *policy)
+{
+    int result = 0;
+    TS_TST_INFO *tst_info = NULL;
+    ASN1_INTEGER *serial = NULL;
+    ASN1_GENERALIZEDTIME *asn1_time = NULL;
+    long sec, usec;
+    TS_ACCURACY *accuracy = NULL;
+    const ASN1_INTEGER *nonce;
+    GENERAL_NAME *tsa_name = NULL;
+
+    if ((tst_info = TS_TST_INFO_new()) == NULL)
+        goto end;
+    if (!TS_TST_INFO_set_version(tst_info, 1))
+        goto end;
+    if (!TS_TST_INFO_set_policy_id(tst_info, policy))
+        goto end;
+    if (!TS_TST_INFO_set_msg_imprint(tst_info, ctx->request->msg_imprint))
+        goto end;
+    if ((serial = ctx->serial_cb(ctx, ctx->serial_cb_data)) == NULL
+        || !TS_TST_INFO_set_serial(tst_info, serial))
+        goto end;
+    if (!ctx->time_cb(ctx, ctx->time_cb_data, &sec, &usec)
+        || (asn1_time =
+            TS_RESP_set_genTime_with_precision(NULL, sec, usec,
+                                        ctx->clock_precision_digits)) == NULL
+        || !TS_TST_INFO_set_time(tst_info, asn1_time))
+        goto end;
+
+    if ((ctx->seconds || ctx->millis || ctx->micros)
+        && (accuracy = TS_ACCURACY_new()) == NULL)
+        goto end;
+    if (ctx->seconds && !TS_ACCURACY_set_seconds(accuracy, ctx->seconds))
+        goto end;
+    if (ctx->millis && !TS_ACCURACY_set_millis(accuracy, ctx->millis))
+        goto end;
+    if (ctx->micros && !TS_ACCURACY_set_micros(accuracy, ctx->micros))
+        goto end;
+    if (accuracy && !TS_TST_INFO_set_accuracy(tst_info, accuracy))
+        goto end;
+
+    if ((ctx->flags & TS_ORDERING)
+        && !TS_TST_INFO_set_ordering(tst_info, 1))
+        goto end;
+
+    if ((nonce = ctx->request->nonce) != NULL
+        && !TS_TST_INFO_set_nonce(tst_info, nonce))
+        goto end;
+
+    if (ctx->flags & TS_TSA_NAME) {
+        if ((tsa_name = GENERAL_NAME_new()) == NULL)
+            goto end;
+        tsa_name->type = GEN_DIRNAME;
+        tsa_name->d.dirn =
+            X509_NAME_dup(X509_get_subject_name(ctx->signer_cert));
+        if (!tsa_name->d.dirn)
+            goto end;
+        if (!TS_TST_INFO_set_tsa(tst_info, tsa_name))
+            goto end;
+    }
+
+    result = 1;
  end:
-       if (!result)
-               {
-               TS_TST_INFO_free(tst_info);
-               tst_info = NULL;
-               TSerr(TS_F_TS_RESP_CREATE_TST_INFO, TS_R_TST_INFO_SETUP_ERROR);
-               TS_RESP_CTX_set_status_info_cond(ctx, TS_STATUS_REJECTION,
-                                                "Error during TSTInfo "
-                                                "generation.");
-               }
-       GENERAL_NAME_free(tsa_name);
-       TS_ACCURACY_free(accuracy);
-       ASN1_GENERALIZEDTIME_free(asn1_time);
-       ASN1_INTEGER_free(serial);
-       
-       return tst_info;
-       }
+    if (!result) {
+        TS_TST_INFO_free(tst_info);
+        tst_info = NULL;
+        TSerr(TS_F_TS_RESP_CREATE_TST_INFO, TS_R_TST_INFO_SETUP_ERROR);
+        TS_RESP_CTX_set_status_info_cond(ctx, TS_STATUS_REJECTION,
+                                         "Error during TSTInfo "
+                                         "generation.");
+    }
+    GENERAL_NAME_free(tsa_name);
+    TS_ACCURACY_free(accuracy);
+    ASN1_GENERALIZEDTIME_free(asn1_time);
+    ASN1_INTEGER_free(serial);
+
+    return tst_info;
+}
 
 /* Processing the extensions of the request. */
-static int TS_RESP_process_extensions(TS_RESP_CTX *ctx)
-       {
-       STACK_OF(X509_EXTENSION) *exts = TS_REQ_get_exts(ctx->request);
-       int i;
-       int ok = 1;
-
-       for (i = 0; ok && i < sk_X509_EXTENSION_num(exts); ++i)
-               {
-               X509_EXTENSION *ext = sk_X509_EXTENSION_value(exts, i);
-               /* XXXXX The last argument was previously
-                  (void *)ctx->extension_cb, but ISO C doesn't permit
-                  converting a function pointer to void *.  For lack of
-                  better information, I'm placing a NULL there instead.
-                  The callback can pick its own address out from the ctx
-                  anyway...
-               */
-               ok = (*ctx->extension_cb)(ctx, ext, NULL);
-               }
-
-       return ok;
-       }
+static int ts_RESP_process_extensions(TS_RESP_CTX *ctx)
+{
+    STACK_OF(X509_EXTENSION) *exts = ctx->request->extensions;
+    int i;
+    int ok = 1;
+
+    for (i = 0; ok && i < sk_X509_EXTENSION_num(exts); ++i) {
+        X509_EXTENSION *ext = sk_X509_EXTENSION_value(exts, i);
+        /*
+         * The last argument was previously (void *)ctx->extension_cb,
+         * but ISO C doesn't permit converting a function pointer to void *.
+         * For lack of better information, I'm placing a NULL there instead.
+         * The callback can pick its own address out from the ctx anyway...
+         */
+        ok = (*ctx->extension_cb) (ctx, ext, NULL);
+    }
+
+    return ok;
+}
 
 /* Functions for signing the TS_TST_INFO structure of the context. */
-static int TS_RESP_sign(TS_RESP_CTX *ctx)
-       {
-       int ret = 0;
-       PKCS7 *p7 = NULL;
-       PKCS7_SIGNER_INFO *si;
-       STACK_OF(X509) *certs;  /* Certificates to include in sc. */
-       ESS_SIGNING_CERT *sc = NULL;
-       ASN1_OBJECT *oid;
-       BIO *p7bio = NULL;
-       int i;
-
-       /* Check if signcert and pkey match. */
-       if (!X509_check_private_key(ctx->signer_cert, ctx->signer_key)) {
-               TSerr(TS_F_TS_RESP_SIGN, 
-                     TS_R_PRIVATE_KEY_DOES_NOT_MATCH_CERTIFICATE);
-                goto err;
-       }
-
-       /* Create a new PKCS7 signed object. */
-       if (!(p7 = PKCS7_new())) {
-               TSerr(TS_F_TS_RESP_SIGN, ERR_R_MALLOC_FAILURE);
-               goto err;
-       }
-       if (!PKCS7_set_type(p7, NID_pkcs7_signed)) goto err;
-
-       /* Force SignedData version to be 3 instead of the default 1. */
-       if (!ASN1_INTEGER_set(p7->d.sign->version, 3)) goto err;
-
-       /* Add signer certificate and optional certificate chain. */
-       if (TS_REQ_get_cert_req(ctx->request))
-               {
-               PKCS7_add_certificate(p7, ctx->signer_cert);
-               if (ctx->certs)
-                       {
-                       for(i = 0; i < sk_X509_num(ctx->certs); ++i) 
-                               {
-                               X509 *cert = sk_X509_value(ctx->certs, i);
-                               PKCS7_add_certificate(p7, cert);
-                               }
-                       }
-               }
-
-       /* Add a new signer info. */
-       if (!(si = PKCS7_add_signature(p7, ctx->signer_cert, 
-                                      ctx->signer_key, EVP_sha1())))
-               {
-               TSerr(TS_F_TS_RESP_SIGN, TS_R_PKCS7_ADD_SIGNATURE_ERROR);
-               goto err;
-               }
-
-       /* Add content type signed attribute to the signer info. */
-       oid = OBJ_nid2obj(NID_id_smime_ct_TSTInfo);
-       if (!PKCS7_add_signed_attribute(si, NID_pkcs9_contentType,
-                                       V_ASN1_OBJECT, oid))
-               {
-               TSerr(TS_F_TS_RESP_SIGN, TS_R_PKCS7_ADD_SIGNED_ATTR_ERROR);
-               goto err;
-               }
-
-       /* Create the ESS SigningCertificate attribute which contains 
-          the signer certificate id and optionally the certificate chain. */
-       certs = ctx->flags & TS_ESS_CERT_ID_CHAIN ? ctx->certs : NULL;
-       if (!(sc = ESS_SIGNING_CERT_new_init(ctx->signer_cert, certs)))
-               goto err;
-
-       /* Add SigningCertificate signed attribute to the signer info. */
-       if (!ESS_add_signing_cert(si, sc))
-               {
-               TSerr(TS_F_TS_RESP_SIGN, TS_R_ESS_ADD_SIGNING_CERT_ERROR);
-               goto err;
-               }       
-
-       /* Add a new empty NID_id_smime_ct_TSTInfo encapsulated content. */
-       if (!TS_TST_INFO_content_new(p7)) goto err;
-
-       /* Add the DER encoded tst_info to the PKCS7 structure. */
-       if (!(p7bio = PKCS7_dataInit(p7, NULL))) {
-               TSerr(TS_F_TS_RESP_SIGN, ERR_R_MALLOC_FAILURE);
-               goto err;
-       }
-
-       /* Convert tst_info to DER. */
-       if (!i2d_TS_TST_INFO_bio(p7bio, ctx->tst_info))
-               {
-               TSerr(TS_F_TS_RESP_SIGN, TS_R_TS_DATASIGN);
-               goto err;
-               }
-
-       /* Create the signature and add it to the signer info. */
-        if (!PKCS7_dataFinal(p7, p7bio))
-               {
-               TSerr(TS_F_TS_RESP_SIGN, TS_R_TS_DATASIGN);
-               goto err;
-               }
-
-       /* Set new PKCS7 and TST_INFO objects. */
-       TS_RESP_set_tst_info(ctx->response, p7, ctx->tst_info);
-       p7 = NULL;              /* Ownership is lost. */
-       ctx->tst_info = NULL;   /* Ownership is lost. */
-
-       ret = 1;
+static int ts_RESP_sign(TS_RESP_CTX *ctx)
+{
+    int ret = 0;
+    PKCS7 *p7 = NULL;
+    PKCS7_SIGNER_INFO *si;
+    STACK_OF(X509) *certs;      /* Certificates to include in sc. */
+    ESS_SIGNING_CERT *sc = NULL;
+    ASN1_OBJECT *oid;
+    BIO *p7bio = NULL;
+    int i;
+
+    if (!X509_check_private_key(ctx->signer_cert, ctx->signer_key)) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_PRIVATE_KEY_DOES_NOT_MATCH_CERTIFICATE);
+        goto err;
+    }
+
+    if ((p7 = PKCS7_new()) == NULL) {
+        TSerr(TS_F_TS_RESP_SIGN, ERR_R_MALLOC_FAILURE);
+        goto err;
+    }
+    if (!PKCS7_set_type(p7, NID_pkcs7_signed))
+        goto err;
+    if (!ASN1_INTEGER_set(p7->d.sign->version, 3))
+        goto err;
+
+    if (ctx->request->cert_req) {
+        PKCS7_add_certificate(p7, ctx->signer_cert);
+        if (ctx->certs) {
+            for (i = 0; i < sk_X509_num(ctx->certs); ++i) {
+                X509 *cert = sk_X509_value(ctx->certs, i);
+                PKCS7_add_certificate(p7, cert);
+            }
+        }
+    }
+
+    if ((si = PKCS7_add_signature(p7, ctx->signer_cert,
+                                  ctx->signer_key, ctx->signer_md)) == NULL) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_PKCS7_ADD_SIGNATURE_ERROR);
+        goto err;
+    }
+
+    oid = OBJ_nid2obj(NID_id_smime_ct_TSTInfo);
+    if (!PKCS7_add_signed_attribute(si, NID_pkcs9_contentType,
+                                    V_ASN1_OBJECT, oid)) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_PKCS7_ADD_SIGNED_ATTR_ERROR);
+        goto err;
+    }
+
+    certs = ctx->flags & TS_ESS_CERT_ID_CHAIN ? ctx->certs : NULL;
+    if ((sc = ess_SIGNING_CERT_new_init(ctx->signer_cert, certs)) == NULL)
+        goto err;
+    if (!ESS_add_signing_cert(si, sc)) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_ESS_ADD_SIGNING_CERT_ERROR);
+        goto err;
+    }
+
+    if (!ts_TST_INFO_content_new(p7))
+        goto err;
+    if ((p7bio = PKCS7_dataInit(p7, NULL)) == NULL) {
+        TSerr(TS_F_TS_RESP_SIGN, ERR_R_MALLOC_FAILURE);
+        goto err;
+    }
+    if (!i2d_TS_TST_INFO_bio(p7bio, ctx->tst_info)) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_TS_DATASIGN);
+        goto err;
+    }
+    if (!PKCS7_dataFinal(p7, p7bio)) {
+        TSerr(TS_F_TS_RESP_SIGN, TS_R_TS_DATASIGN);
+        goto err;
+    }
+    TS_RESP_set_tst_info(ctx->response, p7, ctx->tst_info);
+    p7 = NULL;                  /* Ownership is lost. */
+    ctx->tst_info = NULL;       /* Ownership is lost. */
+
+    ret = 1;
  err:
-       if (!ret)
-               TS_RESP_CTX_set_status_info_cond(ctx, TS_STATUS_REJECTION,
-                                                "Error during signature "
-                                                "generation.");
-       BIO_free_all(p7bio);
-       ESS_SIGNING_CERT_free(sc);
-       PKCS7_free(p7);
-       return ret;
-       }
-
-static ESS_SIGNING_CERT *ESS_SIGNING_CERT_new_init(X509 *signcert, 
-                                                  STACK_OF(X509) *certs)
-       {
-       ESS_CERT_ID *cid;
-       ESS_SIGNING_CERT *sc = NULL;
-       int i;
-
-       /* Creating the ESS_CERT_ID stack. */
-       if (!(sc = ESS_SIGNING_CERT_new())) goto err;
-       if (!sc->cert_ids && !(sc->cert_ids = sk_ESS_CERT_ID_new_null()))
-               goto err;
-
-       /* Adding the signing certificate id. */
-       if (!(cid = ESS_CERT_ID_new_init(signcert, 0))
-           || !sk_ESS_CERT_ID_push(sc->cert_ids, cid))
-               goto err;
-       /* Adding the certificate chain ids. */
-       for (i = 0; i < sk_X509_num(certs); ++i)
-               {
-               X509 *cert = sk_X509_value(certs, i);
-               if (!(cid = ESS_CERT_ID_new_init(cert, 1))
-                   || !sk_ESS_CERT_ID_push(sc->cert_ids, cid))
-                       goto err;
-               }
-
-       return sc;
-err:
-       ESS_SIGNING_CERT_free(sc);
-       TSerr(TS_F_ESS_SIGNING_CERT_NEW_INIT, ERR_R_MALLOC_FAILURE);
-       return NULL;
-       }
-
-static ESS_CERT_ID *ESS_CERT_ID_new_init(X509 *cert, int issuer_needed)
-       {
-       ESS_CERT_ID *cid = NULL;
-       GENERAL_NAME *name = NULL;
-       
-       /* Recompute SHA1 hash of certificate if necessary (side effect). */
-       X509_check_purpose(cert, -1, 0);
-
-       if (!(cid = ESS_CERT_ID_new())) goto err;
-       if (!ASN1_OCTET_STRING_set(cid->hash, cert->sha1_hash,
-                                  sizeof(cert->sha1_hash)))
-               goto err;
-
-       /* Setting the issuer/serial if requested. */
-       if (issuer_needed)
-               {
-               /* Creating issuer/serial structure. */
-               if (!cid->issuer_serial
-                   && !(cid->issuer_serial = ESS_ISSUER_SERIAL_new()))
-                       goto err;
-               /* Creating general name from the certificate issuer. */
-               if (!(name = GENERAL_NAME_new())) goto err;
-               name->type = GEN_DIRNAME;
-               if (!(name->d.dirn = X509_NAME_dup(cert->cert_info->issuer))) 
-                       goto err;
-               if (!sk_GENERAL_NAME_push(cid->issuer_serial->issuer, name)) 
-                       goto err;
-               name = NULL;    /* Ownership is lost. */
-               /* Setting the serial number. */
-               ASN1_INTEGER_free(cid->issuer_serial->serial);
-               if (!(cid->issuer_serial->serial = 
-                     ASN1_INTEGER_dup(cert->cert_info->serialNumber)))
-                       goto err;
-               }
-
-       return cid;
-err:
-       GENERAL_NAME_free(name);
-       ESS_CERT_ID_free(cid);
-       TSerr(TS_F_ESS_CERT_ID_NEW_INIT, ERR_R_MALLOC_FAILURE);
-       return NULL;
-       }
-
-static int TS_TST_INFO_content_new(PKCS7 *p7)
-       {
-       PKCS7 *ret = NULL;
-       ASN1_OCTET_STRING *octet_string = NULL;
-
-       /* Create new encapsulated NID_id_smime_ct_TSTInfo content. */
-       if (!(ret = PKCS7_new())) goto err;
-       if (!(ret->d.other = ASN1_TYPE_new())) goto err;
-       ret->type = OBJ_nid2obj(NID_id_smime_ct_TSTInfo);
-       if (!(octet_string = ASN1_OCTET_STRING_new())) goto err;
-       ASN1_TYPE_set(ret->d.other, V_ASN1_OCTET_STRING, octet_string);
-       octet_string = NULL;
-
-       /* Add encapsulated content to signed PKCS7 structure. */
-       if (!PKCS7_set_content(p7, ret)) goto err;
-
-       return 1;
+    if (!ret)
+        TS_RESP_CTX_set_status_info_cond(ctx, TS_STATUS_REJECTION,
+                                         "Error during signature "
+                                         "generation.");
+    BIO_free_all(p7bio);
+    ESS_SIGNING_CERT_free(sc);
+    PKCS7_free(p7);
+    return ret;
+}
+
+static ESS_SIGNING_CERT *ess_SIGNING_CERT_new_init(X509 *signcert,
+                                                   STACK_OF(X509) *certs)
+{
+    ESS_CERT_ID *cid;
+    ESS_SIGNING_CERT *sc = NULL;
+    int i;
+
+    if ((sc = ESS_SIGNING_CERT_new()) == NULL)
+        goto err;
+    if (sc->cert_ids == NULL
+        && (sc->cert_ids = sk_ESS_CERT_ID_new_null()) == NULL)
+        goto err;
+
+    if ((cid = ess_CERT_ID_new_init(signcert, 0)) == NULL
+        || !sk_ESS_CERT_ID_push(sc->cert_ids, cid))
+        goto err;
+    for (i = 0; i < sk_X509_num(certs); ++i) {
+        X509 *cert = sk_X509_value(certs, i);
+        if ((cid = ess_CERT_ID_new_init(cert, 1)) == NULL
+            || !sk_ESS_CERT_ID_push(sc->cert_ids, cid))
+            goto err;
+    }
+
+    return sc;
  err:
-       ASN1_OCTET_STRING_free(octet_string);
-       PKCS7_free(ret);
-       return 0;
-       }
+    ESS_SIGNING_CERT_free(sc);
+    TSerr(TS_F_ESS_SIGNING_CERT_NEW_INIT, ERR_R_MALLOC_FAILURE);
+    return NULL;
+}
+
+static ESS_CERT_ID *ess_CERT_ID_new_init(X509 *cert, int issuer_needed)
+{
+    ESS_CERT_ID *cid = NULL;
+    GENERAL_NAME *name = NULL;
+    unsigned char cert_sha1[SHA_DIGEST_LENGTH];
+
+    /* Call for side-effect of computing hash and caching extensions */
+    X509_check_purpose(cert, -1, 0);
+    if ((cid = ESS_CERT_ID_new()) == NULL)
+        goto err;
+    X509_digest(cert, EVP_sha1(), cert_sha1, NULL);
+    if (!ASN1_OCTET_STRING_set(cid->hash, cert_sha1, SHA_DIGEST_LENGTH))
+        goto err;
+
+    /* Setting the issuer/serial if requested. */
+    if (issuer_needed) {
+        if (cid->issuer_serial == NULL
+            && (cid->issuer_serial = ESS_ISSUER_SERIAL_new()) == NULL)
+            goto err;
+        if ((name = GENERAL_NAME_new()) == NULL)
+            goto err;
+        name->type = GEN_DIRNAME;
+        if ((name->d.dirn = X509_NAME_dup(X509_get_issuer_name(cert))) == NULL)
+            goto err;
+        if (!sk_GENERAL_NAME_push(cid->issuer_serial->issuer, name))
+            goto err;
+        name = NULL;            /* Ownership is lost. */
+        ASN1_INTEGER_free(cid->issuer_serial->serial);
+        if (!(cid->issuer_serial->serial =
+              ASN1_INTEGER_dup(X509_get_serialNumber(cert))))
+            goto err;
+    }
+
+    return cid;
+ err:
+    GENERAL_NAME_free(name);
+    ESS_CERT_ID_free(cid);
+    TSerr(TS_F_ESS_CERT_ID_NEW_INIT, ERR_R_MALLOC_FAILURE);
+    return NULL;
+}
+
+static int ts_TST_INFO_content_new(PKCS7 *p7)
+{
+    PKCS7 *ret = NULL;
+    ASN1_OCTET_STRING *octet_string = NULL;
+
+    /* Create new encapsulated NID_id_smime_ct_TSTInfo content. */
+    if ((ret = PKCS7_new()) == NULL)
+        goto err;
+    if ((ret->d.other = ASN1_TYPE_new()) == NULL)
+        goto err;
+    ret->type = OBJ_nid2obj(NID_id_smime_ct_TSTInfo);
+    if ((octet_string = ASN1_OCTET_STRING_new()) == NULL)
+        goto err;
+    ASN1_TYPE_set(ret->d.other, V_ASN1_OCTET_STRING, octet_string);
+    octet_string = NULL;
+
+    /* Add encapsulated content to signed PKCS7 structure. */
+    if (!PKCS7_set_content(p7, ret))
+        goto err;
+
+    return 1;
+ err:
+    ASN1_OCTET_STRING_free(octet_string);
+    PKCS7_free(ret);
+    return 0;
+}
 
 static int ESS_add_signing_cert(PKCS7_SIGNER_INFO *si, ESS_SIGNING_CERT *sc)
-       {
-       ASN1_STRING *seq = NULL;
-       unsigned char *p, *pp = NULL;
-       int len;
-
-       len = i2d_ESS_SIGNING_CERT(sc, NULL);
-       if (!(pp = (unsigned char *) OPENSSL_malloc(len)))
-               {
-               TSerr(TS_F_ESS_ADD_SIGNING_CERT, ERR_R_MALLOC_FAILURE);
-               goto err;
-               }
-       p = pp;
-       i2d_ESS_SIGNING_CERT(sc, &p);
-       if (!(seq = ASN1_STRING_new()) || !ASN1_STRING_set(seq, pp, len))
-               {
-               TSerr(TS_F_ESS_ADD_SIGNING_CERT, ERR_R_MALLOC_FAILURE);
-               goto err;
-               }
-       OPENSSL_free(pp); pp = NULL;
-       return PKCS7_add_signed_attribute(si, 
-                                         NID_id_smime_aa_signingCertificate,
-                                         V_ASN1_SEQUENCE, seq);
+{
+    ASN1_STRING *seq = NULL;
+    unsigned char *p, *pp = NULL;
+    int len;
+
+    len = i2d_ESS_SIGNING_CERT(sc, NULL);
+    if ((pp = OPENSSL_malloc(len)) == NULL) {
+        TSerr(TS_F_ESS_ADD_SIGNING_CERT, ERR_R_MALLOC_FAILURE);
+        goto err;
+    }
+    p = pp;
+    i2d_ESS_SIGNING_CERT(sc, &p);
+    if ((seq = ASN1_STRING_new()) == NULL || !ASN1_STRING_set(seq, pp, len)) {
+        TSerr(TS_F_ESS_ADD_SIGNING_CERT, ERR_R_MALLOC_FAILURE);
+        goto err;
+    }
+    OPENSSL_free(pp);
+    pp = NULL;
+    return PKCS7_add_signed_attribute(si,
+                                      NID_id_smime_aa_signingCertificate,
+                                      V_ASN1_SEQUENCE, seq);
  err:
-       ASN1_STRING_free(seq);
-       OPENSSL_free(pp);
-
-       return 0;
-       }
-
-
-static ASN1_GENERALIZEDTIME *
-TS_RESP_set_genTime_with_precision(ASN1_GENERALIZEDTIME *asn1_time, 
-                                  long sec, long usec, unsigned precision)
-       {
-       time_t time_sec = (time_t) sec;
-       struct tm *tm = NULL;   
-       char genTime_str[17 + TS_MAX_CLOCK_PRECISION_DIGITS];
-       char *p = genTime_str;
-       char *p_end = genTime_str + sizeof(genTime_str);
-
-       if (precision > TS_MAX_CLOCK_PRECISION_DIGITS)
-               goto err;
-
-       
-       if (!(tm = gmtime(&time_sec)))
-               goto err;
-
-       /* 
-        * Put "genTime_str" in GeneralizedTime format.  We work around the 
-        * restrictions imposed by rfc3280 (i.e. "GeneralizedTime values MUST 
-        * NOT include fractional seconds") and OpenSSL related functions to 
-        * meet the rfc3161 requirement: "GeneralizedTime syntax can include 
-        * fraction-of-second details". 
-        */                   
-       p += BIO_snprintf(p, p_end - p,
-                         "%04d%02d%02d%02d%02d%02d",
-                         tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday, 
-                         tm->tm_hour, tm->tm_min, tm->tm_sec);
-       if (precision > 0)
-       {
-               /* Add fraction of seconds (leave space for dot and null). */
-               BIO_snprintf(p, 2 + precision, ".%ld", usec);
-               /* We cannot use the snprintf return value, 
-                  because it might have been truncated. */
-               p += strlen(p);
-
-               /* To make things a bit harder, X.690 | ISO/IEC 8825-1 provides
-                  the following restrictions for a DER-encoding, which OpenSSL
-                  (specifically ASN1_GENERALIZEDTIME_check() function) doesn't 
-                  support:
-                  "The encoding MUST terminate with a "Z" (which means "Zulu" 
-                  time). The decimal point element, if present, MUST be the 
-                  point option ".". The fractional-seconds elements, 
-                  if present, MUST omit all trailing 0's; 
-                  if the elements correspond to 0, they MUST be wholly
-                  omitted, and the decimal point element also MUST be
-                  omitted." */
-               /* Remove trailing zeros. The dot guarantees the exit
-                  condition of this loop even if all the digits are zero. */
-               while (*--p == '0')
-                       /* empty */;
-               /* p points to either the dot or the last non-zero digit. */
-               if (*p != '.') ++p;
-               }
-       /* Add the trailing Z and the terminating null. */
-       *p++ = 'Z';
-       *p++ = '\0';
-
-       /* Now call OpenSSL to check and set our genTime value */
-       if (!asn1_time && !(asn1_time = M_ASN1_GENERALIZEDTIME_new()))
-               goto err;
-       if (!ASN1_GENERALIZEDTIME_set_string(asn1_time, genTime_str))
-               {
-               ASN1_GENERALIZEDTIME_free(asn1_time);
-               goto err;
-               }
-
-       return asn1_time;
+    ASN1_STRING_free(seq);
+    OPENSSL_free(pp);
+
+    return 0;
+}
+
+static ASN1_GENERALIZEDTIME
+*TS_RESP_set_genTime_with_precision(ASN1_GENERALIZEDTIME *asn1_time,
+                                    long sec, long usec, unsigned precision)
+{
+    time_t time_sec = (time_t)sec;
+    struct tm *tm = NULL;
+    char genTime_str[17 + TS_MAX_CLOCK_PRECISION_DIGITS];
+    char *p = genTime_str;
+    char *p_end = genTime_str + sizeof(genTime_str);
+
+    if (precision > TS_MAX_CLOCK_PRECISION_DIGITS)
+        goto err;
+
+    if ((tm = gmtime(&time_sec)) == NULL)
+        goto err;
+
+    /*
+     * Put "genTime_str" in GeneralizedTime format.  We work around the
+     * restrictions imposed by rfc3280 (i.e. "GeneralizedTime values MUST
+     * NOT include fractional seconds") and OpenSSL related functions to
+     * meet the rfc3161 requirement: "GeneralizedTime syntax can include
+     * fraction-of-second details".
+     */
+    p += BIO_snprintf(p, p_end - p,
+                      "%04d%02d%02d%02d%02d%02d",
+                      tm->tm_year + 1900, tm->tm_mon + 1, tm->tm_mday,
+                      tm->tm_hour, tm->tm_min, tm->tm_sec);
+    if (precision > 0) {
+        BIO_snprintf(p, 2 + precision, ".%06ld", usec);
+        p += strlen(p);
+
+        /*
+         * To make things a bit harder, X.690 | ISO/IEC 8825-1 provides the
+         * following restrictions for a DER-encoding, which OpenSSL
+         * (specifically ASN1_GENERALIZEDTIME_check() function) doesn't
+         * support: "The encoding MUST terminate with a "Z" (which means
+         * "Zulu" time). The decimal point element, if present, MUST be the
+         * point option ".". The fractional-seconds elements, if present,
+         * MUST omit all trailing 0's; if the elements correspond to 0, they
+         * MUST be wholly omitted, and the decimal point element also MUST be
+         * omitted."
+         */
+        /*
+         * Remove trailing zeros. The dot guarantees the exit condition of
+         * this loop even if all the digits are zero.
+         */
+        while (*--p == '0')
+             continue;
+        if (*p != '.')
+            ++p;
+    }
+    *p++ = 'Z';
+    *p++ = '\0';
+
+    if (asn1_time == NULL
+        && (asn1_time = ASN1_GENERALIZEDTIME_new()) == NULL)
+        goto err;
+    if (!ASN1_GENERALIZEDTIME_set_string(asn1_time, genTime_str)) {
+        ASN1_GENERALIZEDTIME_free(asn1_time);
+        goto err;
+    }
+    return asn1_time;
+
  err:
-       TSerr(TS_F_TS_RESP_SET_GENTIME_WITH_PRECISION, TS_R_COULD_NOT_SET_TIME);
-       return NULL;
-       }
+    TSerr(TS_F_TS_RESP_SET_GENTIME_WITH_PRECISION, TS_R_COULD_NOT_SET_TIME);
+    return NULL;
+}