diff options
Diffstat (limited to 'meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch')
| -rw-r--r-- | meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch | 37 |
1 files changed, 0 insertions, 37 deletions
diff --git a/meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch b/meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch deleted file mode 100644 index c92b58609c..0000000000 --- a/meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch +++ /dev/null @@ -1,37 +0,0 @@ -From ea8c7b3efce4c1762411e073893e948de5d552d6 Mon Sep 17 00:00:00 2001 -From: Ross Burton <ross.burton@intel.com> -Date: Tue, 17 Jul 2012 16:04:12 +0100 -Subject: [PATCH] storage: check that the string isn't empty before splitting - -If the string was non-NULL but empty (str="\0"), the following \0 assignment -would write to str[-1] and thus cause memory corruption. - -On PPC and MIPS, this was causing crashes in glibc. - -Signed-off-by: Ross Burton <ross.burton@intel.com> -Upstream-Status: Submitted - ---- - src/storage.c | 6 +++++- - 1 file changed, 5 insertions(+), 1 deletion(-) - -diff --git a/src/storage.c b/src/storage.c -index 47bd0cb..20766a3 100644 ---- a/src/storage.c -+++ b/src/storage.c -@@ -212,7 +212,11 @@ gchar **connman_storage_get_services() - closedir(dir); - - str = g_string_free(result, FALSE); -- if (str) { -+ if (str && str[0] != '\0') { -+ /* -+ * Remove the trailing separator so that services doesn't end up -+ * with an empty element. -+ */ - str[strlen(str) - 1] = '\0'; - services = g_strsplit(str, "/", -1); - } --- -1.7.10.4 - |