Skip to content

Commit 4a05545

Browse files
authored
Fix[BMQ]: Fix ntcchannelfactory IT with Memory Sanitizer (#869)
Signed-off-by: Dmitrii Petukhov <[email protected]>
1 parent 81a0356 commit 4a05545

File tree

3 files changed

+4
-2
lines changed

3 files changed

+4
-2
lines changed

docker/sanitizers/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Usually sanitizers check is done on CI, but using Docker it is possible to run s
77
- Prerequisites: docker should be installed;
88
- Run docker build from the BlazingMQ root folder:
99
```
10-
docker build -f docker/sanitizers/Dockerfile --no-cache --build-arg SANITIZER_NAME=<sanitizer-name> -t sanitizer-<sanitizer-name> .
10+
docker build -f docker/sanitizers/Dockerfile --no-cache --build-arg SANITIZER_NAME=<sanitizer-name> --build-arg FUZZER=off -t sanitizer-<sanitizer-name> .
1111
```
1212
NOTE: `sanitizer-name` is `asan`, `msan`, `tsan` or `ubsan`.
1313

docker/sanitizers/build_sanitizer.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -294,6 +294,8 @@ cmake --build "${DIR_BUILD_BMQ}" -j${PARALLELISM} \
294294
--target ${TARGETS} -v --clean-first
295295

296296
if [ "${FUZZER}" == "on" ]; then
297+
# In fuzzers case we only need to build the tests
298+
# So we can exit here
297299
exit 0
298300
fi
299301

src/groups/bmq/bmqio/bmqio_ntcchannelfactory.t.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1164,7 +1164,7 @@ static void test7_checkMultithreadListen()
11641164

11651165
// Wait for all threads to finish
11661166
const int rc = doneLatch.timedWait(bsls::SystemTime::nowRealtimeClock() +
1167-
bsls::TimeInterval(2.0));
1167+
bsls::TimeInterval(5.0));
11681168
BMQTST_ASSERT_EQ_D(L_, rc, 0);
11691169

11701170
for (size_t i = 0; i < k_NUM_THREADS; i++) {

0 commit comments

Comments
 (0)