aboutsummaryrefslogtreecommitdiff
path: root/openssl
diff options
context:
space:
mode:
authorRob Austein <sra@hactrn.net>2006-07-25 07:38:02 +0000
committerRob Austein <sra@hactrn.net>2006-07-25 07:38:02 +0000
commit2ba1d33c2552d36defd19fa81c9e1519b8e67b6e (patch)
tree04d2455983efca8ee5e6760b3ac606a80314ff02 /openssl
parent9be4a2742cb969ec6af7e933598a08932c0c4ef4 (diff)
_new() and _free() functions explained
svn path=/openssl/README; revision=59
Diffstat (limited to 'openssl')
-rw-r--r--openssl/README18
1 files changed, 18 insertions, 0 deletions
diff --git a/openssl/README b/openssl/README
index 0f760451..7789fc58 100644
--- a/openssl/README
+++ b/openssl/README
@@ -363,3 +363,21 @@ necessary, needs reorganization. All we really need to do is:
3) If so, replace them with a merged prefix or range.
+
+
+Doh. A lot of the missing documentation is buried in ssleay.txt,
+which the other documentation says not to read because it's so old.
+But it's where Eric explains all the basic data structures and
+expected usage as of the dawn of time, so most of the stuff that's so
+old that it's undocumented is really documented there. Sigh.
+
+Anyway, it looks like the xxx_new() functions already set pointers of
+sub structures to NULL or allocate the substructures (have to check
+which for the things we care about, I guess), and the xxx_free()
+functions already clean up complex structures. So be sure to set
+unused pointers to NULL if one has been fiddling.
+
+foo->type fields may be initialized to -1 too, need to check that.
+
+So the destructor routines I wrote are unnecessary, and some of the
+memset()s may be actively harmful. Need to clean that up.