some const fixes and cleanup
[openssl.git] / crypto / objects / o_names.c
index dca988230e8c69ac4a382d8c2331ee23895bc5f2..adb5731f7659f52e3c0de19af53a26e4fee4bb4e 100644 (file)
@@ -2,9 +2,22 @@
 #include <stdlib.h>
 #include <string.h>
 
+#include <openssl/err.h>
 #include <openssl/lhash.h>
 #include <openssl/objects.h>
 #include <openssl/safestack.h>
+#include <openssl/e_os2.h>
+
+/* Later versions of DEC C has started to add lnkage information to certain
+ * functions, which makes it tricky to use them as values to regular function
+ * pointers.  One way is to define a macro that takes care of casting them
+ * correctly.
+ */
+#ifdef OPENSSL_SYS_VMS_DECC
+# define OPENSSL_strcmp (int (*)(const char *,const char *))strcmp
+#else
+# define OPENSSL_strcmp strcmp
+#endif
 
 /* I use the ex_data stuff to manage the identifiers for the obj_name_types
  * that applications may define.  I only really use the free function field.
@@ -14,9 +27,9 @@ static int names_type_num=OBJ_NAME_TYPE_NUM;
 
 typedef struct name_funcs_st
        {
-       unsigned long (*hash_func)();
-       int (*cmp_func)();
-       void (*free_func)();
+       unsigned long (*hash_func)(const char *name);
+       int (*cmp_func)(const char *a,const char *b);
+       void (*free_func)(const char *, int, const char *);
        } NAME_FUNCS;
 
 DECLARE_STACK_OF(NAME_FUNCS)
@@ -24,20 +37,26 @@ IMPLEMENT_STACK_OF(NAME_FUNCS)
 
 static STACK_OF(NAME_FUNCS) *name_funcs_stack;
 
-static unsigned long obj_name_hash(OBJ_NAME *a);
-static int obj_name_cmp(OBJ_NAME *a,OBJ_NAME *b);
+/* The LHASH callbacks now use the raw "void *" prototypes and do per-variable
+ * casting in the functions. This prevents function pointer casting without the
+ * need for macro-generated wrapper functions. */
+
+/* static unsigned long obj_name_hash(OBJ_NAME *a); */
+static unsigned long obj_name_hash(const void *a_void);
+/* static int obj_name_cmp(OBJ_NAME *a,OBJ_NAME *b); */
+static int obj_name_cmp(const void *a_void,const void *b_void);
 
 int OBJ_NAME_init(void)
        {
        if (names_lh != NULL) return(1);
        MemCheck_off();
-       names_lh=lh_new(obj_name_hash,obj_name_cmp);
+       names_lh=lh_new(obj_name_hash, obj_name_cmp);
        MemCheck_on();
        return(names_lh != NULL);
        }
 
 int OBJ_NAME_new_index(unsigned long (*hash_func)(const char *),
-       int (*cmp_func)(const void *, const void *),
+       int (*cmp_func)(const char *, const char *),
        void (*free_func)(const char *, int, const char *))
        {
        int ret;
@@ -61,13 +80,20 @@ int OBJ_NAME_new_index(unsigned long (*hash_func)(const char *),
                {
                MemCheck_off();
                name_funcs = OPENSSL_malloc(sizeof(NAME_FUNCS));
+               MemCheck_on();
+               if (!name_funcs)
+                       {
+                       OBJerr(OBJ_F_OBJ_NAME_NEW_INDEX,ERR_R_MALLOC_FAILURE);
+                       return(0);
+                       }
                name_funcs->hash_func = lh_strhash;
-               name_funcs->cmp_func = (int (*)())strcmp;
+               name_funcs->cmp_func = OPENSSL_strcmp;
                name_funcs->free_func = 0; /* NULL is often declared to
-                                           * ((void *)0), which according
-                                           * to Compaq C is not really
-                                           * compatible with a function
-                                           * pointer.  -- Richard Levitte*/
+                                               * ((void *)0), which according
+                                               * to Compaq C is not really
+                                               * compatible with a function
+                                               * pointer.      -- Richard Levitte*/
+               MemCheck_off();
                sk_NAME_FUNCS_push(name_funcs_stack,name_funcs);
                MemCheck_on();
                }
@@ -81,9 +107,12 @@ int OBJ_NAME_new_index(unsigned long (*hash_func)(const char *),
        return(ret);
        }
 
-static int obj_name_cmp(OBJ_NAME *a, OBJ_NAME *b)
+/* static int obj_name_cmp(OBJ_NAME *a, OBJ_NAME *b) */
+static int obj_name_cmp(const void *a_void, const void *b_void)
        {
        int ret;
+       const OBJ_NAME *a = (const OBJ_NAME *)a_void;
+       const OBJ_NAME *b = (const OBJ_NAME *)b_void;
 
        ret=a->type-b->type;
        if (ret == 0)
@@ -91,8 +120,8 @@ static int obj_name_cmp(OBJ_NAME *a, OBJ_NAME *b)
                if ((name_funcs_stack != NULL)
                        && (sk_NAME_FUNCS_num(name_funcs_stack) > a->type))
                        {
-                       ret=sk_NAME_FUNCS_value(name_funcs_stack,a->type)
-                               ->cmp_func(a->name,b->name);
+                       ret=sk_NAME_FUNCS_value(name_funcs_stack,
+                               a->type)->cmp_func(a->name,b->name);
                        }
                else
                        ret=strcmp(a->name,b->name);
@@ -100,14 +129,16 @@ static int obj_name_cmp(OBJ_NAME *a, OBJ_NAME *b)
        return(ret);
        }
 
-static unsigned long obj_name_hash(OBJ_NAME *a)
+/* static unsigned long obj_name_hash(OBJ_NAME *a) */
+static unsigned long obj_name_hash(const void *a_void)
        {
        unsigned long ret;
+       const OBJ_NAME *a = (const OBJ_NAME *)a_void;
 
        if ((name_funcs_stack != NULL) && (sk_NAME_FUNCS_num(name_funcs_stack) > a->type))
                {
-               ret=sk_NAME_FUNCS_value(name_funcs_stack,a->type)
-                       ->hash_func(a->name);
+               ret=sk_NAME_FUNCS_value(name_funcs_stack,
+                       a->type)->hash_func(a->name);
                }
        else
                {
@@ -132,7 +163,7 @@ const char *OBJ_NAME_get(const char *name, int type)
        on.type=type;
 
        for (;;)
-               {
+       {
                ret=(OBJ_NAME *)lh_retrieve(names_lh,&on);
                if (ret == NULL) return(NULL);
                if ((ret->alias) && !alias)
@@ -179,8 +210,8 @@ int OBJ_NAME_add(const char *name, int type, const char *data)
                         * function should get three arguments...
                         * -- Richard Levitte
                         */
-                       sk_NAME_FUNCS_value(name_funcs_stack,ret->type)
-                               ->free_func(ret->name,ret->type,ret->data);
+                       sk_NAME_FUNCS_value(name_funcs_stack,
+                               ret->type)->free_func(ret->name,ret->type,ret->data);
                        }
                OPENSSL_free(ret);
                }
@@ -214,8 +245,8 @@ int OBJ_NAME_remove(const char *name, int type)
                         * function should get three arguments...
                         * -- Richard Levitte
                         */
-                       sk_NAME_FUNCS_value(name_funcs_stack,ret->type)
-                               ->free_func(ret->name,ret->type,ret->data);
+                       sk_NAME_FUNCS_value(name_funcs_stack,
+                               ret->type)->free_func(ret->name,ret->type,ret->data);
                        }
                OPENSSL_free(ret);
                return(1);
@@ -224,12 +255,82 @@ int OBJ_NAME_remove(const char *name, int type)
                return(0);
        }
 
+struct doall
+       {
+       int type;
+       void (*fn)(const OBJ_NAME *,void *arg);
+       void *arg;
+       };
+
+static void do_all_fn(const OBJ_NAME *name,struct doall *d)
+       {
+       if(name->type == d->type)
+               d->fn(name,d->arg);
+       }
+
+static IMPLEMENT_LHASH_DOALL_ARG_FN(do_all_fn, const OBJ_NAME *, struct doall *)
+
+void OBJ_NAME_do_all(int type,void (*fn)(const OBJ_NAME *,void *arg),void *arg)
+       {
+       struct doall d;
+
+       d.type=type;
+       d.fn=fn;
+       d.arg=arg;
+
+       lh_doall_arg(names_lh,LHASH_DOALL_ARG_FN(do_all_fn),&d);
+       }
+
+struct doall_sorted
+       {
+       int type;
+       int n;
+       const OBJ_NAME **names;
+       };
+
+static void do_all_sorted_fn(const OBJ_NAME *name,void *d_)
+       {
+       struct doall_sorted *d=d_;
+
+       if(name->type != d->type)
+               return;
+
+       d->names[d->n++]=name;
+       }
+
+static int do_all_sorted_cmp(const void *n1_,const void *n2_)
+       {
+       const OBJ_NAME * const *n1=n1_;
+       const OBJ_NAME * const *n2=n2_;
+
+       return strcmp((*n1)->name,(*n2)->name);
+       }
+
+void OBJ_NAME_do_all_sorted(int type,void (*fn)(const OBJ_NAME *,void *arg),
+                               void *arg)
+       {
+       struct doall_sorted d;
+       int n;
+
+       d.type=type;
+       d.names=OPENSSL_malloc(lh_num_items(names_lh)*sizeof *d.names);
+       d.n=0;
+       OBJ_NAME_do_all(type,do_all_sorted_fn,&d);
+
+       qsort((void *)d.names,d.n,sizeof *d.names,do_all_sorted_cmp);
+
+       for(n=0 ; n < d.n ; ++n)
+               fn(d.names[n],arg);
+
+       OPENSSL_free((void *)d.names);
+       }
+
 static int free_type;
 
-static void names_lh_free(OBJ_NAME *onp, int type)
+static void names_lh_free(OBJ_NAME *onp)
 {
        if(onp == NULL)
-           return;
+               return;
 
        if ((free_type < 0) || (free_type == onp->type))
                {
@@ -237,6 +338,8 @@ static void names_lh_free(OBJ_NAME *onp, int type)
                }
        }
 
+static IMPLEMENT_LHASH_DOALL_FN(names_lh_free, OBJ_NAME *)
+
 static void name_funcs_free(NAME_FUNCS *ptr)
        {
        OPENSSL_free(ptr);
@@ -252,7 +355,7 @@ void OBJ_NAME_cleanup(int type)
        down_load=names_lh->down_load;
        names_lh->down_load=0;
 
-       lh_doall(names_lh,names_lh_free);
+       lh_doall(names_lh,LHASH_DOALL_FN(names_lh_free));
        if (type < 0)
                {
                lh_free(names_lh);