summaryrefslogtreecommitdiffstats
path: root/ipa-client/ipa-getkeytab.c
diff options
context:
space:
mode:
authorJim Meyering <jim@meyering.net>2008-05-14 21:48:34 +0200
committerSimo Sorce <ssorce@redhat.com>2008-05-15 12:59:28 -0400
commit0d2212fe2f1ea73af36ab955acf8b1518b6cbf64 (patch)
tree7416f8c6bcd6ad4f2cc5a870844c2aedd856ede0 /ipa-client/ipa-getkeytab.c
parent3256365277d86c2bb8b693d007dcfa01adae25a8 (diff)
downloadfreeipa-0d2212fe2f1ea73af36ab955acf8b1518b6cbf64.tar.gz
freeipa-0d2212fe2f1ea73af36ab955acf8b1518b6cbf64.tar.xz
freeipa-0d2212fe2f1ea73af36ab955acf8b1518b6cbf64.zip
remove useless if-before-free tests
I've been on a crusade (;-) to remove useless if-before-free tests, so ran a script that spotted some here. I think I removed the first batch (without braces) automatically, then manually removed the ones with curly braces around the free statements. You may well have doubts about the portability of removing those tests, but as long as you don't care about SunOS4 or earlier, you'll be fine. I've done similar things for e.g., coreutils, glibc, and git, and have had no problems.
Diffstat (limited to 'ipa-client/ipa-getkeytab.c')
-rw-r--r--ipa-client/ipa-getkeytab.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/ipa-client/ipa-getkeytab.c b/ipa-client/ipa-getkeytab.c
index 96426509..8a8c856b 100644
--- a/ipa-client/ipa-getkeytab.c
+++ b/ipa-client/ipa-getkeytab.c
@@ -433,7 +433,7 @@ error_out:
if (res) ldap_msgfree(res);
if (ld) ldap_unbind_ext(ld, NULL, NULL);
if (control) ber_bvfree(control);
- if (encs) free(encs);
+ free(encs);
return 0;
}