More error_data memory leaks
authorBodo Möller <bodo@openssl.org>
Thu, 15 Mar 2001 11:30:10 +0000 (11:30 +0000)
committerBodo Möller <bodo@openssl.org>
Thu, 15 Mar 2001 11:30:10 +0000 (11:30 +0000)
CHANGES
crypto/err/err.c

diff --git a/CHANGES b/CHANGES
index 4c8a93cb1b96578515cec04ba6f02a447c715680..dd3fc17246dca1881edd143960bb01b5736596dd 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -3,7 +3,8 @@
 
  Changes between 0.9.6 and 0.9.7  [xx XXX 2000]
 
-  *) Fix a memory leak in err.c: free err_data string if necessary.
+  *) Fix memory leaks in err.c: free err_data string if necessary;
+     don't write to the wrong index in ERR_set_error_data.
      [Bodo Moeller]
 
   *) Function EC_POINTs_mul for simultaneous scalar multiplication
index 06d1ef9c4ad867c5dd7f135e852b975d46042773..b4183a7f713cab0313426d976bf7a644804cc6a8 100644 (file)
@@ -391,20 +391,18 @@ void ERR_put_error(int lib, int func, int reason, const char *file,
 
 void ERR_clear_error(void)
        {
+       int i;
        ERR_STATE *es;
 
        es=ERR_get_state();
 
-#if 0
-       /* hmm... is this needed */
        for (i=0; i<ERR_NUM_ERRORS; i++)
                {
                es->err_buffer[i]=0;
+               err_clear_data(es,i);
                es->err_file[i]=NULL;
                es->err_line[i]= -1;
-               err_clear_data(es,i);
                }
-#endif
        es->top=es->bottom=0;
        }
 
@@ -764,8 +762,9 @@ void ERR_set_error_data(char *data, int flags)
        if (i == 0)
                i=ERR_NUM_ERRORS-1;
 
+       err_clear_data(es,i);
        es->err_data[i]=data;
-       es->err_data_flags[es->top]=flags;
+       es->err_data_flags[i]=flags;
        }
 
 void ERR_add_error_data(int num, ...)
@@ -774,7 +773,7 @@ void ERR_add_error_data(int num, ...)
        int i,n,s;
        char *str,*p,*a;
 
-       s=64;
+       s=80;
        str=OPENSSL_malloc(s+1);
        if (str == NULL) return;
        str[0]='\0';