From 062bd810fcd8c3d80689ab404766d20c9251dca6 Mon Sep 17 00:00:00 2001 From: Drew Moseley Date: Sun, 27 Jul 2014 19:05:57 -0400 Subject: libx11: Add missing NULL check Add a check for NULL on dereferencing the xim pointer. Using mplayer with the x11 backend will trigger this and result in a segfault. (From OE-Core rev: f7d8947744d7dd1d08721ef0e707912304722d2b) Signed-off-by: Drew Moseley Signed-off-by: Saul Wold Signed-off-by: Richard Purdie --- .../libx11/libX11-Add-missing-NULL-check.patch | 72 ++++++++++++++++++++++ meta/recipes-graphics/xorg-lib/libx11_1.6.2.bb | 1 + 2 files changed, 73 insertions(+) create mode 100644 meta/recipes-graphics/xorg-lib/libx11/libX11-Add-missing-NULL-check.patch (limited to 'meta/recipes-graphics/xorg-lib') diff --git a/meta/recipes-graphics/xorg-lib/libx11/libX11-Add-missing-NULL-check.patch b/meta/recipes-graphics/xorg-lib/libx11/libX11-Add-missing-NULL-check.patch new file mode 100644 index 0000000000..30686206ab --- /dev/null +++ b/meta/recipes-graphics/xorg-lib/libx11/libX11-Add-missing-NULL-check.patch @@ -0,0 +1,72 @@ +From 4d85d5f3bd8afac893738fd88e19d8cb5cf998ff Mon Sep 17 00:00:00 2001 +From: Drew Moseley +Date: Sun, 27 Jul 2014 13:10:07 -0400 +Subject: [PATCH] libX11: Add missing NULL check on im and im->methods + +Upstream-Status: Submitted [https://bugs.freedesktop.org/show_bug.cgi?id=81822] + +Signed-off-by: Drew Moseley +--- + src/xlibi18n/ICWrap.c | 5 +++-- + 1 file changed, 3 insertions(+), 2 deletions(-) + +diff -ru libX11-1.6.2-orig/src/xlibi18n/ICWrap.c libX11-1.6.2/src/xlibi18n/ICWrap.c +--- libX11-1.6.2-orig/src/xlibi18n/ICWrap.c 2014-07-27 13:38:29.586666449 -0400 ++++ libX11-1.6.2/src/xlibi18n/ICWrap.c 2014-07-27 13:41:39.671609030 -0400 +@@ -169,7 +169,7 @@ + va_list var; + int total_count; + XIMArg *args; +- char *ret; ++ char *ret = NULL; + + /* + * so count the stuff dangling here +@@ -185,7 +185,8 @@ + _XIMVaToNestedList(var, total_count, &args); + va_end(var); + +- ret = (*im->methods->set_values) (im, args); ++ if (im && im->methods) ++ ret = (*im->methods->set_values) (im, args); + if (args) Xfree(args); + return ret; + } +@@ -196,7 +197,7 @@ + va_list var; + int total_count; + XIMArg *args; +- char *ret; ++ char *ret = NULL; + + /* + * so count the stuff dangling here +@@ -212,7 +213,8 @@ + _XIMVaToNestedList(var, total_count, &args); + va_end(var); + +- ret = (*im->methods->get_values) (im, args); ++ if (im && im->methods) ++ ret = (*im->methods->get_values) (im, args); + if (args) Xfree(args); + return ret; + } +@@ -228,7 +230,7 @@ + va_list var; + int total_count; + XIMArg *args; +- XIC ic; ++ XIC ic = NULL; + + /* + * so count the stuff dangling here +@@ -244,7 +246,8 @@ + _XIMVaToNestedList(var, total_count, &args); + va_end(var); + +- ic = (XIC) (*im->methods->create_ic) (im, args); ++ if (im && im->methods) ++ ic = (XIC) (*im->methods->create_ic) (im, args); + if (args) Xfree(args); + if (ic) { + ic->core.next = im->core.ic_chain; diff --git a/meta/recipes-graphics/xorg-lib/libx11_1.6.2.bb b/meta/recipes-graphics/xorg-lib/libx11_1.6.2.bb index 700bc3680f..a7226480c2 100644 --- a/meta/recipes-graphics/xorg-lib/libx11_1.6.2.bb +++ b/meta/recipes-graphics/xorg-lib/libx11_1.6.2.bb @@ -5,6 +5,7 @@ BBCLASSEXTEND = "native nativesdk" SRC_URI += "file://disable_tests.patch \ file://nls-always-use-XCOMM-instead-of-for-comments-in-Comp.patch \ + file://libX11-Add-missing-NULL-check.patch \ " SRC_URI[md5sum] = "c35d6ad95b06635a524579e88622fdb5" -- cgit v1.2.3-54-g00ecf