Encountered a crash when debugging adminutil, which appears to be caused by an uninitialized pointer. I can not reproduce it anymore, the fix is quite simple and it shouldn't hurt to add it:
diff --git a/lib/libadminutil/admutil.c b/lib/libadminutil/admutil.c index 839ab2d..1ab0151 100644 --- a/lib/libadminutil/admutil.c +++ b/lib/libadminutil/admutil.c @@ -1210,7 +1210,7 @@ admldapBuildInfoOnly(char* configRoot, int *errorcode) char buf[MAX_LEN], *name= NULL, *password=NULL; int status; - LDAPURLDesc *ldapInfo; + LDAPURLDesc *ldapInfo = NULL; *errorcode = ADMUTIL_OP_OK;
attachment 0001-Ticket-47881-crash-during-debug-session-in-adminutil.patch
To ssh://git.fedorahosted.org/git/389/adminutil.git 9b3cfce..069de33 master -> master
commit 069de33671fd49732c9baeb7fb0007eb7518d97e Author: Mark Reynolds mreynolds@redhat.com Date: Wed Aug 27 10:25:30 2014 -0400
This could definitely be related to mixing mozldap and openldap in the same binary. The LDAPURLDesc size and layout is different for mozldap and openldap, and mixing the two will cause strange crashes.
Metadata Update from @mreynolds: - Issue assigned to mreynolds - Issue set to the milestone: 389-admin,console 1.1.36
389-ds-base is moving from Pagure to Github. This means that new issues and pull requests will be accepted only in 389-ds-base's github repository.
This issue has been cloned to Github and is available here: - https://github.com/389ds/389-ds-base/issues/1212
If you want to receive further updates on the issue, please navigate to the github issue and click on subscribe button.
subscribe
Thank you for understanding. We apologize for all inconvenience.
Metadata Update from @spichugi: - Issue close_status updated to: wontfix (was: Fixed)
Login to comment on this ticket.