rjancewicz / python-kadmin

Python module for kerberos admin (kadm5)
MIT License
35 stars 40 forks source link

restructure kadmin.c to correctly handle errors and avoid memory leaks #32

Closed rjancewicz closed 9 years ago

rjancewicz commented 9 years ago

cleanup errors on kadmin object new error model for principal objects remove remaining instances of old error handling. restore direct error numbers (lower errno values are possible); cleanup memory leaks track internal lock state; only unlock on dealloc if lock is held. krb5_init leaks memory - same result running kadmin.local: unsupported command when resolving the client_name from a credential cache we are responsible for freeing allocated memory valgrind supp file for python leaks. restructure assignment so no memory is allocated until argumets are parsed.