diff options
| author | John Dennis <jdennis@redhat.com> | 2013-04-09 09:46:46 -0400 |
|---|---|---|
| committer | John Dennis <jdennis@redhat.com> | 2013-04-09 09:46:46 -0400 |
| commit | 42c8a18eb027264a3f3a0e63e51d1fcac35a07b1 (patch) | |
| tree | f8b7c5da353ff7a3d648e4d3b4cef995c49d3581 | |
| parent | 088abab8d683af92520a5b88dd91050de224630c (diff) | |
| download | realmd-cim-42c8a18eb027264a3f3a0e63e51d1fcac35a07b1.tar.gz realmd-cim-42c8a18eb027264a3f3a0e63e51d1fcac35a07b1.tar.xz realmd-cim-42c8a18eb027264a3f3a0e63e51d1fcac35a07b1.zip | |
Remove unnecessary NULL test prior to callin g_free, g_strfreev
| -rw-r--r-- | LMI_RealmdKerberosRealmProvider.c | 8 | ||||
| -rw-r--r-- | LMI_RealmdServiceProvider.c | 8 |
2 files changed, 4 insertions, 12 deletions
diff --git a/LMI_RealmdKerberosRealmProvider.c b/LMI_RealmdKerberosRealmProvider.c index 0aecd12..70253d1 100644 --- a/LMI_RealmdKerberosRealmProvider.c +++ b/LMI_RealmdKerberosRealmProvider.c @@ -430,9 +430,7 @@ KEXTERN KUint32 LMI_RealmdKerberosRealm_Join( exit: - if (data) { - g_free(data); - } + g_free(data); if (credentials) { g_variant_unref(credentials); } @@ -624,9 +622,7 @@ KEXTERN KUint32 LMI_RealmdKerberosRealm_Leave( exit: - if (data) { - g_free(data); - } + g_free(data); if (credentials) { g_variant_unref(credentials); } diff --git a/LMI_RealmdServiceProvider.c b/LMI_RealmdServiceProvider.c index ba1a6d5..52d7d7d 100644 --- a/LMI_RealmdServiceProvider.c +++ b/LMI_RealmdServiceProvider.c @@ -295,9 +295,7 @@ KUint32 LMI_RealmdService_Discover( if (options) { g_variant_unref(options); } - if (paths) { - g_strfreev(paths); - } + g_strfreev(paths); return result; } @@ -520,9 +518,7 @@ KUint32 LMI_RealmdService_Join_Leave_Domain( if (options) { g_variant_unref(options); } - if (paths) { - g_strfreev(paths); - } + g_strfreev(paths); if (credentials) { g_variant_unref(credentials); } |