Skip to content

Commit

Permalink
chore: Enable memory sanitizer build.
Browse files Browse the repository at this point in the history
I don't know if this will actually work, or how many of these "fixes" I
need to get msan to be happy on CI. For me locally, it all works fine.
On CI, for some reason it's not fine even though I run in the exact same
docker image as CI.
  • Loading branch information
iphydf committed Mar 19, 2022
1 parent faccbe3 commit 2b2eb41
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
3 changes: 1 addition & 2 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,7 @@ jobs:
steps:
- checkout
- run: rm -rf /src/workspace/c-toxcore/* && mv * /src/workspace/c-toxcore/
# TODO(iphydf): Remove "|| true" once this works.
- run: cd /src/workspace && bazel test //c-toxcore/auto_tests:lossless_packet_test || true
- run: cd /src/workspace && bazel test //c-toxcore/auto_tests:lossless_packet_test

infer:
working_directory: ~/work
Expand Down
2 changes: 1 addition & 1 deletion other/bootstrap_daemon/docker/tox-bootstrapd.sha256
Original file line number Diff line number Diff line change
@@ -1 +1 @@
ba9c06f1079837ac20376c25bbf1411ddb6d8f86cc5c44cf58d053969b192653 /usr/local/bin/tox-bootstrapd
73bc2fb0838ba342f92fa5e2c4a85989c61d04fd87a20db7edc300a9bb42781a /usr/local/bin/tox-bootstrapd
4 changes: 2 additions & 2 deletions toxcore/DHT.c
Original file line number Diff line number Diff line change
Expand Up @@ -351,9 +351,9 @@ int create_request(const uint8_t *send_public_key, const uint8_t *send_secret_ke

uint8_t *const nonce = packet + 1 + CRYPTO_PUBLIC_KEY_SIZE * 2;
random_nonce(nonce);
uint8_t temp[MAX_CRYPTO_REQUEST_SIZE];
memcpy(temp + 1, data, data_length);
uint8_t temp[MAX_CRYPTO_REQUEST_SIZE] = {0};
temp[0] = request_id;
memcpy(temp + 1, data, data_length);
const int len = encrypt_data(recv_public_key, send_secret_key, nonce, temp, data_length + 1,
packet + CRYPTO_SIZE);

Expand Down

0 comments on commit 2b2eb41

Please sign in to comment.