[OE-core] [PATCH 2/2] icu 3.6: do_install failed: Segmentation fault
Robert Yang
liezhi.yang at windriver.com
Fri Jun 29 01:54:33 UTC 2012
On 06/27/2012 04:19 PM, Robert Yang wrote:
> There is a "Segmentation fault" error when the tmpdir is longer than 470
> (or less), this is because it used "char cmd[1024]" which is not enough
> for the command line. Allocate a larger memory size should fix this problem.
>
> [YOCTO #2664]
>
> Signed-off-by: Robert Yang<liezhi.yang at windriver.com>
> ---
> .../icu/files/larger-cmd-size.patch | 27 ++++++++++++++++++++
> meta/recipes-support/icu/icu-3.6.inc | 1 +
> meta/recipes-support/icu/icu_3.6.bb | 2 +-
> 3 files changed, 29 insertions(+), 1 deletions(-)
> create mode 100644 meta/recipes-support/icu/files/larger-cmd-size.patch
>
> diff --git a/meta/recipes-support/icu/files/larger-cmd-size.patch b/meta/recipes-support/icu/files/larger-cmd-size.patch
> new file mode 100644
> index 0000000..4b5f73d
> --- /dev/null
> +++ b/meta/recipes-support/icu/files/larger-cmd-size.patch
> @@ -0,0 +1,27 @@
> +Allocate a larger memory size for cmd
> +
> +The length of the command line can be longer than 1024 sometimes,
> +which will cause a "Segmentation fault" error.
> +
> +Signed-off-by: Robert Yang<liezhi.yang at windriver.com>
> +
> +Upstream-Status: Pending
> +---
> + tools/pkgdata/pkgdata.c | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/tools/pkgdata/pkgdata.c b/tools/pkgdata/pkgdata.c
> +--- a/tools/pkgdata/pkgdata.c
> ++++ b/tools/pkgdata/pkgdata.c
> +@@ -439,7 +439,7 @@ main(int argc, char* argv[]) {
> + /* POSIX - execute makefile */
> + static int executeMakefile(const UPKGOptions *o)
> + {
> +- char cmd[1024];
> ++ char cmd[BUFSIZ];
I've changed the cmd[BUFSIZ] to cmd[2048], which should be enough for us, and
pushed it to the git repo again:
git://git.pokylinux.org/poky-contrib robert/icu
http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=robert/icu
// Robert
> + /*char pwd[1024];*/
> + const char *make;
> + int rc;
> +--
> +1.7.10.2
> +
> diff --git a/meta/recipes-support/icu/icu-3.6.inc b/meta/recipes-support/icu/icu-3.6.inc
> index d3391fe..d969caa 100644
> --- a/meta/recipes-support/icu/icu-3.6.inc
> +++ b/meta/recipes-support/icu/icu-3.6.inc
> @@ -6,6 +6,7 @@ BASE_SRC_URI = "ftp://ftp.software.ibm.com/software/globalization/icu/3.6/icu4c-
> file://gccfix.patch \
> file://fix-parallel-build.patch \
> file://use-g++-for-linking.patch \
> + file://larger-cmd-size.patch \
> "
> SRC_URI = "${BASE_SRC_URI} \
> file://noldlibpath.patch \
> diff --git a/meta/recipes-support/icu/icu_3.6.bb b/meta/recipes-support/icu/icu_3.6.bb
> index d7be522..f941acf 100644
> --- a/meta/recipes-support/icu/icu_3.6.bb
> +++ b/meta/recipes-support/icu/icu_3.6.bb
> @@ -1,6 +1,6 @@
> require icu-3.6.inc
>
> -PR = "r8"
> +PR = "r9"
>
> SRC_URI[md5sum] = "6243f7a19e03e05403ce84e597510d4c"
> SRC_URI[sha256sum] = "5135e8d69d6206d320515df7aeee7027711ab1aef9d8dbf29571a97a9746b041"
More information about the Openembedded-core
mailing list