From: Jeremy Harris Date: Wed, 8 Jan 2020 10:49:31 +0000 (+0000) Subject: Fix error logging for dynamically-loaded modules. Bug 2507 X-Git-Url: https://git.exim.org/users/jgh/exim.git/commitdiff_plain/b1c673ddfac7f322a62786cd4aae8b5b30ba69e8?ds=sidebyside Fix error logging for dynamically-loaded modules. Bug 2507 --- diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index ce225e949..2b5b592c5 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -81,7 +81,11 @@ WB/01 SPF: DNS lookups for the obsolete SPF RR type done by the libspf2 library are now specifically given a NO_DATA response without hitting the system resolver. The library goes on to do the now-standard TXT lookup. Use of dnsdb lookups is not affected. - + +JH/19 Bug 2507: Modules: on handling a dynamic-module (lookups) open failure, + only retrieve the errormessage once. Previously two calls to dlerror() + were used, and the second one (for mainlog/paniclog) retrieved null + information. Exim version 4.93 diff --git a/src/src/drtables.c b/src/src/drtables.c index 578ddf370..558359032 100644 --- a/src/src/drtables.c +++ b/src/src/drtables.c @@ -753,9 +753,10 @@ else if (!(dl = dlopen(CS big_buffer, RTLD_NOW))) { - fprintf(stderr, "Error loading %s: %s\n", name, dlerror()); + errormessage = dlerror(); + fprintf(stderr, "Error loading %s: %s\n", name, errormessage); moduleerrors++; - log_write(0, LOG_MAIN|LOG_PANIC, "Error loading lookup module %s: %s\n", name, dlerror()); + log_write(0, LOG_MAIN|LOG_PANIC, "Error loading lookup module %s: %s\n", name, errormessage); continue; } diff --git a/src/src/exim.c b/src/src/exim.c index af4b52559..92f5623d2 100644 --- a/src/src/exim.c +++ b/src/src/exim.c @@ -1265,9 +1265,9 @@ void *dlhandle; void *dlhandle_curses = dlopen("libcurses." DYNLIB_FN_EXT, RTLD_GLOBAL|RTLD_LAZY); dlhandle = dlopen("libreadline." DYNLIB_FN_EXT, RTLD_GLOBAL|RTLD_NOW); -if (dlhandle_curses != NULL) dlclose(dlhandle_curses); +if (dlhandle_curses) dlclose(dlhandle_curses); -if (dlhandle != NULL) +if (dlhandle) { /* Checked manual pages; at least in GNU Readline 6.1, the prototypes are: * char * readline (const char *prompt); @@ -1277,9 +1277,7 @@ if (dlhandle != NULL) *fn_addhist_ptr = (void(*)(const char*))dlsym(dlhandle, "add_history"); } else - { DEBUG(D_any) debug_printf("failed to load readline: %s\n", dlerror()); - } return dlhandle; }