do not canonicalize errors for ap_strerror(). use sizeof() rather than

    a bare constant.

Submitted by: Brian Havard
Reviewed by: Greg Stein

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@85224 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/modules/mappers/mod_so.c b/modules/mappers/mod_so.c
index f6cf9bf0f4fc86ce4710344299e11ce04d555ab7..07d9061f813c3e9f5f02617659233360ad1b060f 100644 (file)
--- a/modules/mappers/mod_so.c
+++ b/modules/mappers/mod_so.c
@@ -256,7 +256,7 @@ static const char *load_module(cmd_parms *cmd, void *dummy,
      */
     if ((status = ap_dso_load(&modhandle, szModuleFile, cmd->pool )) != APR_SUCCESS) {
         char my_error[256];
-        ap_strerror(ap_canonical_error(status), my_error, 256);
+        ap_strerror(status, my_error, sizeof(my_error));
         return ap_pstrcat (cmd->pool, "Cannot load ", szModuleFile,
                            " into server: ", 
                            my_error ? my_error : "(reason unknown)",
@@ -274,7 +274,7 @@ static const char *load_module(cmd_parms *cmd, void *dummy,
         char my_err[256];
        return ap_pstrcat(cmd->pool, "Can't locate API module structure `", modname,
                       "' in file ", szModuleFile, ": ", 
-                       ap_strerror(ap_canonical_error(status), my_err, 256), NULL);
+                       ap_strerror(status, my_err, sizeof(my_error)), NULL);
     }
     modp = (module*) modsym;
     modp->dynamic_load_handle = (ap_dso_handle_t *)modhandle;