From bdbefdcc4813f76b479db6220722a405db1e73d5 Mon Sep 17 00:00:00 2001 From: William King Date: Wed, 14 May 2014 19:56:32 -0700 Subject: [PATCH] CID:1087631 In the unlikely case the sprintf fails, free the memory instead of leak it --- src/switch_xml.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/switch_xml.c b/src/switch_xml.c index dad3d49503..8a5b76a98b 100644 --- a/src/switch_xml.c +++ b/src/switch_xml.c @@ -2043,6 +2043,8 @@ static void switch_xml_user_cache(const char *key, const char *user_name, const char *expires_val = malloc(1024); if (sprintf(expires_val, "%ld", (long)expires)) { switch_core_hash_insert(CACHE_EXPIRES_HASH, mega_key, expires_val); + } else { + switch_safe_free(expires_val); } } switch_core_hash_insert(CACHE_HASH, mega_key, switch_xml_dup(user));