From 4f012e33040f1e0b206dcdd2bd3f7d888f3574eb Mon Sep 17 00:00:00 2001 From: Constantin Musca Date: Fri, 17 Aug 2012 19:10:46 +0300 Subject: [PATCH] connman: upgrade to 1.4 The removed patch is included in connman 1.4. (From OE-Core rev: a1f16a0f3a015628f3d2d02d980311537a3a2c00) Signed-off-by: Constantin Musca Signed-off-by: Saul Wold Signed-off-by: Richard Purdie --- ...ck-that-the-string-isn-t-empty-before-spl.patch | 37 ---------------------- .../connman/{connman_1.3.bb => connman_1.4.bb} | 9 +++--- 2 files changed, 4 insertions(+), 42 deletions(-) delete mode 100644 meta/recipes-connectivity/connman/connman/0001-storage-check-that-the-string-isn-t-empty-before-spl.patch rename meta/recipes-connectivity/connman/{connman_1.3.bb => connman_1.4.bb} (69%) 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 c92b586..0000000 --- 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 -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 -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 - diff --git a/meta/recipes-connectivity/connman/connman_1.3.bb b/meta/recipes-connectivity/connman/connman_1.4.bb similarity index 69% rename from meta/recipes-connectivity/connman/connman_1.3.bb rename to meta/recipes-connectivity/connman/connman_1.4.bb index 4d82794..bd8ba65 100644 --- a/meta/recipes-connectivity/connman/connman_1.3.bb +++ b/meta/recipes-connectivity/connman/connman_1.4.bb @@ -1,13 +1,12 @@ require connman.inc -# 1.3 tag -SRCREV = "3c0fa84091524c7cd6237744f2088ffee2f1d5ad" +# 1.4 tag +SRCREV = "f701bbca259f1f35e68d338f31f5373f75f3da5f" SRC_URI = "git://git.kernel.org/pub/scm/network/connman/connman.git \ file://0001-plugin.h-Change-visibility-to-default-for-debug-symb.patch \ file://add_xuser_dbus_permission.patch \ file://connman \ file://0002-storage.c-If-there-is-no-d_type-support-use-fstatat.patch \ - file://0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch \ - file://0001-storage-check-that-the-string-isn-t-empty-before-spl.patch" + file://0001-timezone.c-If-there-is-no-d_type-support-use-fstatat.patch" S = "${WORKDIR}/git" -PR = "${INC_PR}.2" +PR = "${INC_PR}.0" -- 2.7.4