CLEANUP: map: Avoid memory leak in out-of-memory condition.

This memory leak of about 100 bytes occurs only if there is an error
condidtion during evaluation of a "map" directive in the configuration
file.  This evaluation only happens once on startup because haproxy
does not have a mechanism for re-loading the configuration file during
run-time.  The startup will be aborted anyway due to error conditions
raised.

Nevertheless fix it to silence warnings of static code analysis tools
and be safe against future revisions of the code.

Found in haproxy 1.5.14.

diff --git a/src/map.c b/src/map.c
index a28cedae926f69449c8c48d6ce13e3d001e4894c..ac1c00ab16c73a931c82aa19454955b24e3220c7 100644 (file)
--- a/src/map.c
+++ b/src/map.c
@@ -126,6 +126,7 @@ int sample_load_map(struct arg *arg, struct sample_conv *conv,
        default:
                memprintf(err, "map: internal haproxy error: no default parse case for the input type <%d>.",
                          conv->out_type);
+               free(desc);
                return 0;
        }