diff options
author | Jeremy Harris <jgh146exb@wizmail.org> | 2020-01-08 10:49:31 +0000 |
---|---|---|
committer | Jeremy Harris <jgh146exb@wizmail.org> | 2020-01-08 10:52:37 +0000 |
commit | b1c673ddfac7f322a62786cd4aae8b5b30ba69e8 (patch) | |
tree | bc5324f71cd8e3c69a11cb1271e299ce92f9a9aa /src | |
parent | 2fa25efce2a183e8886d66e2f1a0ae83ac964d8e (diff) |
Fix error logging for dynamically-loaded modules. Bug 2507
Diffstat (limited to 'src')
-rw-r--r-- | src/src/drtables.c | 5 | ||||
-rw-r--r-- | src/src/exim.c | 6 |
2 files changed, 5 insertions, 6 deletions
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; } |