free NULL cleanup 11
[openssl.git] / crypto / objects / obj_xref.c
index 963177c..b752a2c 100644 (file)
@@ -1,6 +1,7 @@
 /* crypto/objects/obj_xref.c */
-/* Written by Dr Stephen N Henson (shenson@bigfoot.com) for the OpenSSL
- * project 2006.
+/*
+ * Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL project
+ * 2006.
  */
 /* ====================================================================
  * Copyright (c) 2006 The OpenSSL Project.  All rights reserved.
@@ -10,7 +11,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 <openssl/objects.h>
 #include "obj_xref.h"
 
-STACK *sig_app, *sigx_app;
+DECLARE_STACK_OF(nid_triple)
+STACK_OF(nid_triple) *sig_app, *sigx_app;
 
-static int cmp_sig(const nid_triple *a, const nid_triple *b)
-       {
-       return **a - **b;
-       }
+static int sig_cmp(const nid_triple *a, const nid_triple *b)
+{
+    return a->sign_id - b->sign_id;
+}
 
-static int cmp_sig_sk(const nid_triple **a, const nid_triple **b)
-       {
-       return ***a - ***b;
-       }
+DECLARE_OBJ_BSEARCH_CMP_FN(nid_triple, nid_triple, sig);
+IMPLEMENT_OBJ_BSEARCH_CMP_FN(nid_triple, nid_triple, sig);
 
-static int cmp_sigx(const nid_triple **a, const nid_triple **b)
-       {
-       int ret;
-       ret = (**a)[1] - (**b)[1];
-       if (ret)
-               return ret;
-       return (**a)[2] - (**b)[2];
-       }
+static int sig_sk_cmp(const nid_triple *const *a, const nid_triple *const *b)
+{
+    return (*a)->sign_id - (*b)->sign_id;
+}
 
+DECLARE_OBJ_BSEARCH_CMP_FN(const nid_triple *, const nid_triple *, sigx);
 
-int OBJ_find_sigid_algs(int signid, int *pdig_nid, int *ppkey_nid)
-       {
-       nid_triple tmp, *rv = NULL;
-       tmp[0] = signid;
+static int sigx_cmp(const nid_triple *const *a, const nid_triple *const *b)
+{
+    int ret;
+    ret = (*a)->hash_id - (*b)->hash_id;
+    if (ret)
+        return ret;
+    return (*a)->pkey_id - (*b)->pkey_id;
+}
 
-       if (sig_app)
-               {
-               int idx = sk_find(sig_app, (char *)&tmp);
-               if (idx >= 0)
-                       rv = (nid_triple *)sk_value(sig_app, idx);
-               }
+IMPLEMENT_OBJ_BSEARCH_CMP_FN(const nid_triple *, const nid_triple *, sigx);
 
+int OBJ_find_sigid_algs(int signid, int *pdig_nid, int *ppkey_nid)
+{
+    nid_triple tmp;
+    const nid_triple *rv = NULL;
+    tmp.sign_id = signid;
+
+    if (sig_app) {
+        int idx = sk_nid_triple_find(sig_app, &tmp);
+        if (idx >= 0)
+            rv = sk_nid_triple_value(sig_app, idx);
+    }
 #ifndef OBJ_XREF_TEST2
-       if (rv == NULL)
-               {
-               rv = (nid_triple *)OBJ_bsearch((char *)&tmp,
-                               (char *)sigoid_srt,
-                               sizeof(sigoid_srt) / sizeof(nid_triple),
-                               sizeof(nid_triple),
-                               (int (*)(const void *, const void *))cmp_sig);
-               }
+    if (rv == NULL) {
+        rv = OBJ_bsearch_sig(&tmp, sigoid_srt,
+                             sizeof(sigoid_srt) / sizeof(nid_triple));
+    }
 #endif
-       if (rv == NULL)
-               return 0;
-       *pdig_nid = (*rv)[1];
-       *ppkey_nid = (*rv)[2];
-       return 1;
-       }
+    if (rv == NULL)
+        return 0;
+    if (pdig_nid)
+        *pdig_nid = rv->hash_id;
+    if (ppkey_nid)
+        *ppkey_nid = rv->pkey_id;
+    return 1;
+}
 
 int OBJ_find_sigid_by_algs(int *psignid, int dig_nid, int pkey_nid)
-       {
-       nid_triple tmp, *t=&tmp, **rv = NULL;
-       tmp[1] = dig_nid;
-       tmp[2] = pkey_nid;
-
-       if (sigx_app)
-               {
-               int idx = sk_find(sigx_app, (char *)&tmp);
-               if (idx >= 0)
-                       {
-                       t = (nid_triple *)sk_value(sigx_app, idx);
-                       rv = &t;
-                       }
-               }
-
+{
+    nid_triple tmp;
+    const nid_triple *t = &tmp;
+    const nid_triple **rv = NULL;
+
+    tmp.hash_id = dig_nid;
+    tmp.pkey_id = pkey_nid;
+
+    if (sigx_app) {
+        int idx = sk_nid_triple_find(sigx_app, &tmp);
+        if (idx >= 0) {
+            t = sk_nid_triple_value(sigx_app, idx);
+            rv = &t;
+        }
+    }
 #ifndef OBJ_XREF_TEST2
-       if (rv == NULL)
-               {
-               rv = (nid_triple **)OBJ_bsearch((char *)&t,
-                               (char *)sigoid_srt_xref,
-                               sizeof(sigoid_srt_xref) / sizeof(nid_triple *),
-                               sizeof(nid_triple *),
-                               (int (*)(const void *, const void *))cmp_sigx);
-               }
+    if (rv == NULL) {
+        rv = OBJ_bsearch_sigx(&t, sigoid_srt_xref,
+                              sizeof(sigoid_srt_xref) / sizeof(nid_triple *)
+            );
+    }
 #endif
-       if (rv == NULL)
-               return 0;
-       *psignid = (**rv)[0];
-       return 1;
-       }
-
-typedef int sk_cmp_fn_type(const char * const *a, const char * const *b);
+    if (rv == NULL)
+        return 0;
+    if (psignid)
+        *psignid = (*rv)->sign_id;
+    return 1;
+}
 
 int OBJ_add_sigid(int signid, int dig_id, int pkey_id)
-       {
-       nid_triple *ntr;
-       if (!sig_app)
-               sig_app = sk_new((sk_cmp_fn_type *)cmp_sig_sk);
-       if (!sig_app)
-               return 0;
-       if (!sigx_app)
-               sigx_app = sk_new((sk_cmp_fn_type *)cmp_sigx);
-       if (!sigx_app)
-               return 0;
-       ntr = OPENSSL_malloc(sizeof(int) * 3);
-       if (!ntr)
-               return 0;
-       (*ntr)[0] = signid;
-       (*ntr)[1] = dig_id;
-       (*ntr)[2] = pkey_id;
-
-       if (!sk_push(sig_app, (char *)ntr))
-               {
-               OPENSSL_free(ntr);
-               return 0;
-               }
-
-       if (!sk_push(sigx_app, (char *)ntr))
-               return 0;
-
-       sk_sort(sig_app);
-       sk_sort(sigx_app);
-
-       return 1;
-       }
-
-static void sid_free(void *x)
-       {
-       nid_triple *tt = (nid_triple *)x;
-       OPENSSL_free(tt);
-       }
+{
+    nid_triple *ntr;
+    if (!sig_app)
+        sig_app = sk_nid_triple_new(sig_sk_cmp);
+    if (!sig_app)
+        return 0;
+    if (!sigx_app)
+        sigx_app = sk_nid_triple_new(sigx_cmp);
+    if (!sigx_app)
+        return 0;
+    ntr = OPENSSL_malloc(sizeof(int) * 3);
+    if (!ntr)
+        return 0;
+    ntr->sign_id = signid;
+    ntr->hash_id = dig_id;
+    ntr->pkey_id = pkey_id;
+
+    if (!sk_nid_triple_push(sig_app, ntr)) {
+        OPENSSL_free(ntr);
+        return 0;
+    }
+
+    if (!sk_nid_triple_push(sigx_app, ntr))
+        return 0;
+
+    sk_nid_triple_sort(sig_app);
+    sk_nid_triple_sort(sigx_app);
+
+    return 1;
+}
+
+static void sid_free(nid_triple *tt)
+{
+    OPENSSL_free(tt);
+}
 
 void OBJ_sigid_free(void)
-       {
-       if (sig_app)
-               {
-               sk_pop_free(sig_app, sid_free);
-               sig_app = NULL;
-               }
-       if (sigx_app)
-               {
-               sk_free(sigx_app);
-               sigx_app = NULL;
-               }
-       }
-               
+{
+    sk_nid_triple_pop_free(sig_app, sid_free);
+    sig_app = NULL;
+    sk_nid_triple_free(sigx_app);
+    sigx_app = NULL;
+}
+
 #ifdef OBJ_XREF_TEST
 
 main()
-       {
-       int n1, n2, n3;
-
-       int i, rv;
-#ifdef OBJ_XREF_TEST2
-       for (i = 0; i < sizeof(sigoid_srt) / sizeof(nid_triple); i++)
-               {
-               OBJ_add_sigid(sigoid_srt[i][0], sigoid_srt[i][1],
-                               sigoid_srt[i][2]);
-               }
-#endif
+{
+    int n1, n2, n3;
+
+    int i, rv;
+# ifdef OBJ_XREF_TEST2
+    for (i = 0; i < sizeof(sigoid_srt) / sizeof(nid_triple); i++) {
+        OBJ_add_sigid(sigoid_srt[i][0], sigoid_srt[i][1], sigoid_srt[i][2]);
+    }
+# endif
+
+    for (i = 0; i < sizeof(sigoid_srt) / sizeof(nid_triple); i++) {
+        n1 = sigoid_srt[i][0];
+        rv = OBJ_find_sigid_algs(n1, &n2, &n3);
+        printf("Forward: %d, %s %s %s\n", rv,
+               OBJ_nid2ln(n1), OBJ_nid2ln(n2), OBJ_nid2ln(n3));
+        n1 = 0;
+        rv = OBJ_find_sigid_by_algs(&n1, n2, n3);
+        printf("Reverse: %d, %s %s %s\n", rv,
+               OBJ_nid2ln(n1), OBJ_nid2ln(n2), OBJ_nid2ln(n3));
+    }
+}
 
-       for (i = 0; i < sizeof(sigoid_srt) / sizeof(nid_triple); i++)
-               {
-               n1 = sigoid_srt[i][0];
-               rv = OBJ_find_sigid_algs(n1, &n2, &n3);
-               printf("Forward: %d, %s %s %s\n", rv,
-                       OBJ_nid2ln(n1), OBJ_nid2ln(n2), OBJ_nid2ln(n3));
-               n1=0;
-               rv = OBJ_find_sigid_by_algs(&n1, n2, n3);
-               printf("Reverse: %d, %s %s %s\n", rv,
-                       OBJ_nid2ln(n1), OBJ_nid2ln(n2), OBJ_nid2ln(n3));
-               }
-       }
-       
 #endif