- dst_lib_init()/dst_lib_destroy() allow dst to use a predefined
mctx and free all resources on exit.
- dst_key_tofile() & dst_key/fromfile() take a directory parameter.
- dst_key_parsefile() removed, replaced by dst_key_fromnamedfile()
- more bug fixes and memory leak fixes
Mostly, several functions that take pointers as arguments, almost
always char * pointers, had those pointers qualified with "const".
Those that returned pointers to previously const-qualified arguments
had their return values qualified as const. Some structure members
were qualified as const to retain that attribute from the variables
from which they were assigned.
The macro DE_CONST is used to deal with a handful of very special
places where something is qualified as const but really needs to have
its const qualifier removed.
rdata.c now defines macros for the prototypes of the basic rdata functions,
and all of the lib/dns/rdata/**/*.c files now use them.
Some minor integer-compatibility issues. (IE, ~0x03 is a signed int,
so assigning it to an unsigned int should use a cast. The type of an
enum member is int, so there are some conversion issues there, too.)
A pointers-to-function should not be cast to a pointer-to-object.
Variables should not be named for C reserved identifiers.
One or two set-but-not-used variables removed.
Minor other ISC style cleanups.
there are no major fixes in here but couple of bug fixes that
may affect operation in some cases. I updated rename.h to
make sure all symbols with T have dst__openssl prefix.
Any bugs found in this source code should be reported to the
openssl people.
Olafur
and int variable. in three of the four cases this cast is questionable,
as suitable range had not been checked, but the situation is no worse
than without the casts.
"bsafe_link.c", line 116: remark(1692): prototyped function redeclared without
prototype
dst_s_bsafersa_init() {
and several variations of:
"bsafe_link.c", line 155: remark(1552): variable "status" was set but never
used
int status = 0;
along with style lint (notably bitwise operands used as truth values:
mode & DST_SIGMODE_INIT versus (mode & DST_SIGMODE_INIT) != 0.
it overwrote old keys if new key key-id conflicted
It was generating non null-keys with id==0
It was not able to generate DSA null keys.
I changed size of maximum RSA key to 2048.
cc: "mem.c", line 173: warning 714: Function "malloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 180: warning 714: Function "malloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 188: warning 714: Function "free_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 194: warning 714: Function "free_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 205: warning 714: Function "malloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 212: warning 714: Function "malloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 222: warning 714: Function "realloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 228: warning 714: Function "realloc_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 236: warning 714: Function "free_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 242: warning 714: Function "free_debug_func" called with no prototype or definition in scope.
cc: "mem.c", line 258: warning 714: Function "set_debug_options_func" called with no prototype or definition in scope.
cc: "mem.c", line 264: warning 714: Function "get_debug_options_func" called with no prototype or definition in scope.
cc: "stack.c", line 231: warning 714: Function called with no prototype or definition in scope.
cc: "stack.c", line 275: warning 714: Function "func" called with no prototype or definition in scope.
David Lawrence