comerr: Don't leak CFStringRef

If str is NULL on entry to _intlize, than it will leak the
CFStringRef that it constructed a few lines earlier. Fix this.

Caught with clang-analyzer

Change-Id: I6d62e7792918da95e4fed1c676d14ad9124d93fa
Reviewed-on: http://gerrit.openafs.org/2137
Reviewed-by: Russ Allbery <rra@stanford.edu>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>

diff --git a/src/comerr/error_msg.c b/src/comerr/error_msg.c
index 21d46fab979a7d90666e5d8ccff67d88880c4d8e..eae3204f58298fc36d8b8de394c5c39dbda4ebcf 100644 (file)
--- a/src/comerr/error_msg.c
+++ b/src/comerr/error_msg.c
@@ -126,15 +126,25 @@ _intlize(const char *msg, int base, char *str, size_t len)
     CFStringRef cfdomain;
     CFBundleRef OpenAFSBundle = CFBundleGetBundleWithIdentifier(CFSTR("org.openafs.filesystems.afs"));
 
-    if (!str)
+    if (!str) {
+        CFRelease(cfstring);
        return msg;
+    }
+
     snprintf(domain, sizeof(domain), "heim_com_err%d", base);
     cfdomain = CFStringCreateWithCString(kCFAllocatorSystemDefault, domain,
                                         kCFStringEncodingUTF8);
-    if (OpenAFSBundle != NULL)
-       cfstring = CFBundleCopyLocalizedString(OpenAFSBundle, cfstring,
-                                              cfstring, cfdomain);
-    CFStringGetCString(cfstring, str, len, kCFStringEncodingUTF8);
+    if (OpenAFSBundle != NULL) {
+       CFStringRef cflocal;
+
+       cflocal = CFBundleCopyLocalizedString(OpenAFSBundle, cfstring,
+                                             cfstring, cfdomain);
+        CFStringGetCString(cflocal, str, len, kCFStringEncodingUTF8);
+       CFRelease(cflocal);
+    } else {
+        CFStringGetCString(cfstring, str, len, kCFStringEncodingUTF8);
+    }
+
     CFRelease(cfstring);
     CFRelease(cfdomain);
     return str;