Blame SOURCES/0002-Use-strdup-if-offset-are-used.patch
|
 |
9a0aa5 |
From 0b24b7ece288641f1d5d6844d1a42ff30b16511a Mon Sep 17 00:00:00 2001
|
|
|
9a0aa5 |
From: Sumit Bose <sbose@redhat.com>
|
|
|
9a0aa5 |
Date: Wed, 24 Aug 2016 16:19:36 +0200
|
|
|
9a0aa5 |
Subject: [PATCH 2/2] Use strdup() if offset are used
|
|
|
9a0aa5 |
|
|
|
9a0aa5 |
Strings with an offset to the original starting point must be copied
|
|
|
9a0aa5 |
because otherwise they cannot be properly freed later.
|
|
|
9a0aa5 |
---
|
|
|
9a0aa5 |
library/adenroll.c | 6 +++---
|
|
|
9a0aa5 |
1 file changed, 3 insertions(+), 3 deletions(-)
|
|
|
9a0aa5 |
|
|
|
9a0aa5 |
diff --git a/library/adenroll.c b/library/adenroll.c
|
|
|
9a0aa5 |
index d1020e9..05885d0 100644
|
|
|
9a0aa5 |
--- a/library/adenroll.c
|
|
|
9a0aa5 |
+++ b/library/adenroll.c
|
|
|
9a0aa5 |
@@ -1318,9 +1318,9 @@ load_keytab_entry (krb5_context k5,
|
|
|
9a0aa5 |
|
|
|
9a0aa5 |
} else if (!enroll->host_fqdn && _adcli_str_has_prefix (name, "host/") && strchr (name, '.')) {
|
|
|
9a0aa5 |
/* Skip host/ prefix */
|
|
|
9a0aa5 |
- enroll->host_fqdn = name + 5;
|
|
|
9a0aa5 |
- _adcli_info ("Found host qualified name in keytab: %s", name);
|
|
|
9a0aa5 |
- name = NULL;
|
|
|
9a0aa5 |
+ enroll->host_fqdn = strdup (name + 5);
|
|
|
9a0aa5 |
+ return_val_if_fail (enroll->host_fqdn != NULL, FALSE);
|
|
|
9a0aa5 |
+ _adcli_info ("Found host qualified name in keytab: %s", enroll->host_fqdn);
|
|
|
9a0aa5 |
}
|
|
|
9a0aa5 |
}
|
|
|
9a0aa5 |
|
|
|
9a0aa5 |
--
|
|
|
9a0aa5 |
2.7.4
|
|
|
9a0aa5 |
|