diff options
author | Dengke Du <dengke.du@windriver.com> | 2016-12-19 13:40:14 +0800 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2016-12-22 08:50:16 +0000 |
commit | 971fc297132b9fd2938cd28a75f6689116688101 (patch) | |
tree | c444bf1677c62502c71e5c2b80dcdc1c6a20dae1 /meta | |
parent | 669edd04432f8e76e32bd7f34f4d9aa63eef4038 (diff) | |
download | poky-971fc297132b9fd2938cd28a75f6689116688101.tar.gz |
Revert "subversion: fix "svnadmin create" fail on x86"
This reverts commit cfe6f3e251240c9d9a70354be0501600357f0b87.
This is because the apr configure wrong, when the apr configure meets the
cross compiling, it pass 8 bytes to "off_t", in apr source code configure.in,
it was hardcoded:
APR_CHECK_SIZEOF_EXTENDED([#include <sys/types.h>], off_t, 8)
The macro "APR_CHECK_SIZEOF_EXTENDED" was defined in build/apr_common.m4,
it use the "AC_TRY_RUN" macro, this macro let the off_t to 8, when cross
compiling enable.
But in glibc on the x86 or multilib target the "off_t" was 4 bytes, so this
cases dismatch for softwares which use the apr.h, such as subversion, run this:
svnadmin create test
It failed because the "APR_OFF_T_FMT" was "lld" in apr.h when apr configure,
but the "apr_off_t" was 4 bytes, in the apr source code: apr_snprintf.c
i_quad = va_arg(ap, apr_int64_t);
When the function apr_vformatter meets "lld", it would use the above to parse,
but the above read 8 bytes, so the follow-up data go to wrong.
So we should configure the apr correct when cross compiling. I do this on the
following patchs.
(From OE-Core rev: fbdfb39c011676fe61a4d58b62226126e0e9ec62)
Signed-off-by: Dengke Du <dengke.du@windriver.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta')
-rw-r--r-- | meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch | 56 | ||||
-rw-r--r-- | meta/recipes-devtools/subversion/subversion_1.9.5.bb | 1 |
2 files changed, 0 insertions, 57 deletions
diff --git a/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch b/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch deleted file mode 100644 index d4405287be..0000000000 --- a/meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch +++ /dev/null | |||
@@ -1,56 +0,0 @@ | |||
1 | From 09475e0befca8d120c957177ce8568fa2209a1a9 Mon Sep 17 00:00:00 2001 | ||
2 | From: Dengke Du <dengke.du@windriver.com> | ||
3 | Date: Wed, 2 Nov 2016 11:09:44 +0800 | ||
4 | Subject: [PATCH] fix "svnadmin create" fail on x86 | ||
5 | |||
6 | When run the following command on x86: | ||
7 | |||
8 | svnadmin create /var/test_repo | ||
9 | |||
10 | It cause segmentation fault error like the following: | ||
11 | |||
12 | [16499.751837] svnadmin[21117]: segfault at 83 ip 00000000f74bf7f6 sp 00000000ffdd9b34 error 4 in libc-2.24.so[f7441000+1af000] | ||
13 | Segmentation fault (core dumped) | ||
14 | |||
15 | This is because in source code ./subversion/libsvn_fs_fs/low_level.c, | ||
16 | function svn_fs_fs__unparse_footer, when: | ||
17 | |||
18 | target arch: x86 | ||
19 | apr_off_t: 4 bytes | ||
20 | |||
21 | if the "APR_OFF_T_FMT" is "lld", it still use type "apr_off_t" to pass | ||
22 | data to apr, but in apr source code file apr_snprintf.c the function | ||
23 | apr_vformatter meet "lld", it would use the: | ||
24 | |||
25 | i_quad = va_arg(ap, apr_int64_t); | ||
26 | |||
27 | It uses the apr_int64_t to deal data, it read 8 bytes, so the follow-up | ||
28 | data may be error. | ||
29 | |||
30 | Upstream-Status: Pending | ||
31 | |||
32 | Signed-off-by: Dengke Du <dengke.du@windriver.com> | ||
33 | --- | ||
34 | subversion/libsvn_fs_fs/low_level.c | 4 ++-- | ||
35 | 1 file changed, 2 insertions(+), 2 deletions(-) | ||
36 | |||
37 | diff --git a/subversion/libsvn_fs_fs/low_level.c b/subversion/libsvn_fs_fs/low_level.c | ||
38 | index a27bbcc..6ddbe28 100644 | ||
39 | --- a/subversion/libsvn_fs_fs/low_level.c | ||
40 | +++ b/subversion/libsvn_fs_fs/low_level.c | ||
41 | @@ -250,10 +250,10 @@ svn_fs_fs__unparse_footer(apr_off_t l2p_offset, | ||
42 | { | ||
43 | return svn_stringbuf_createf(result_pool, | ||
44 | "%" APR_OFF_T_FMT " %s %" APR_OFF_T_FMT " %s", | ||
45 | - l2p_offset, | ||
46 | + (APR_OFF_T_FMT=="lld") ? (apr_int64_t)l2p_offset : l2p_offset, | ||
47 | svn_checksum_to_cstring(l2p_checksum, | ||
48 | scratch_pool), | ||
49 | - p2l_offset, | ||
50 | + (APR_OFF_T_FMT=="lld") ? (apr_int64_t)p2l_offset : p2l_offset, | ||
51 | svn_checksum_to_cstring(p2l_checksum, | ||
52 | scratch_pool)); | ||
53 | } | ||
54 | -- | ||
55 | 2.7.4 | ||
56 | |||
diff --git a/meta/recipes-devtools/subversion/subversion_1.9.5.bb b/meta/recipes-devtools/subversion/subversion_1.9.5.bb index 575bbca46e..05fba677b0 100644 --- a/meta/recipes-devtools/subversion/subversion_1.9.5.bb +++ b/meta/recipes-devtools/subversion/subversion_1.9.5.bb | |||
@@ -14,7 +14,6 @@ SRC_URI = "${APACHE_MIRROR}/${BPN}/${BPN}-${PV}.tar.bz2 \ | |||
14 | file://disable_macos.patch \ | 14 | file://disable_macos.patch \ |
15 | file://serf.m4-Regex-modified-to-allow-D-in-paths.patch \ | 15 | file://serf.m4-Regex-modified-to-allow-D-in-paths.patch \ |
16 | file://0001-Fix-libtool-name-in-configure.ac.patch \ | 16 | file://0001-Fix-libtool-name-in-configure.ac.patch \ |
17 | file://0001-fix-svnadmin-create-fail-on-x86.patch \ | ||
18 | file://serfmacro.patch \ | 17 | file://serfmacro.patch \ |
19 | " | 18 | " |
20 | 19 | ||