diff options
author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-21 17:44:58 +0200 |
---|---|---|
committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-22 08:57:05 +0200 |
commit | 35f29b17699c3d52f77857c530300318b14148f8 (patch) | |
tree | a3674cfb995bbfc8d397dd2d00e5bbc8d0a45684 /contrib | |
parent | 70673115c03c37ddc64c951b53d92df9d3310762 (diff) | |
download | sssd-35f29b17699c3d52f77857c530300318b14148f8.tar.gz sssd-35f29b17699c3d52f77857c530300318b14148f8.tar.xz sssd-35f29b17699c3d52f77857c530300318b14148f8.zip |
CI: Use /bin/sh as a CONFIG SHELL
There is a bug on debian_testing in bash.
sh$ valgrind /bin/bash
==25145== Memcheck, a memory error detector
==25145== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==25145== Using Valgrind-3.12.0.SVN and LibVEX; rerun with -h for copyright info
==25145== Command: /bin/bash
==25145==
==25145== Invalid read of size 1
==25145== at 0x4B90B1: ??? (in /bin/bash)
==25145== by 0x43FE9B: initialize_shell_variables (in /bin/bash)
==25145== by 0x41E4C0: ??? (in /bin/bash)
==25145== by 0x41F722: main (in /bin/bash)
==25145== Address 0x58307f8 is 8 bytes before a block of size 31 alloc'd
==25145== at 0x4C2BBCF: malloc (vg_replace_malloc.c:299)
==25145== by 0x475D1A: xmalloc (in /bin/bash)
==25145== by 0x4B7F4A: tilde_expand (in /bin/bash)
==25145== by 0x42E63D: bash_tilde_expand (in /bin/bash)
==25145== by 0x43FE79: initialize_shell_variables (in /bin/bash)
==25145== by 0x41E4C0: ??? (in /bin/bash)
==25145== by 0x41F722: main (in /bin/bash)
==25145==
malloc: .././variables.c:570: assertion botched
free: called with unallocated block argument
last command: (null)
Aborting...==25145==
And /bin/bash was used as a default SHELL in scripts generated by
configure+libtool. It starting to fail with the latest valgrind
valgrind-3.12.0~svn20160714-1
Workaround is to use /bin/sh which is a symlink to /bin/dash
Reviewed-by: Petr Cech <pcech@redhat.com>
Diffstat (limited to 'contrib')
-rwxr-xr-x | contrib/ci/run | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/contrib/ci/run b/contrib/ci/run index 5f067bc7c..1b230f584 100755 --- a/contrib/ci/run +++ b/contrib/ci/run @@ -198,7 +198,8 @@ function build_debug() test_dir=`mktemp --directory /dev/shm/ci-test-dir.XXXXXXXX` stage configure "$BASE_DIR/configure" \ "${CONFIGURE_ARG_LIST[@]}" \ - --with-test-dir="$test_dir" + --with-test-dir="$test_dir" \ + SHELL=/bin/sh # Not building "tests" due to https://fedorahosted.org/sssd/ticket/2350 stage make-tests make -j $CPU_NUM check LOG_COMPILER=true |