foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-22 15:06:54 -05:00
Code Issues Packages 1 Wiki Activity

Remove leaks from snapshot_creator

Browse source
This commit is contained in:
Ryan Dahl 2018-12-03 19:57:34 -05:00
parent 9e839b7e23
commit 0bec0fa594
2 changed files with 13 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
.travis.yml
View file

@ -93,18 +93,20 @@ script:
./tools/test_format.py
- |-
# LSAN build of test_cc. We are in the process of getting a
# completely clean LSAN build, but it will take some work. So for now
# we just run a single test that passes under LSAN.
# LSAN build. We are in the process of getting a completely clean LSAN build,
# but it will take some work. So for now we just run a subset of the tests.
echo is_asan=true >> target/debug/args.gn
echo is_lsan=true >> target/debug/args.gn
DENO_BUILD_MODE=debug ./tools/build.py -j2 test_cc
ASAN_OPTIONS=detect_leaks=1 ./target/debug/test_cc
# We want to detect leaks during the build process as well as when executing
# the tests. So set the ASAN_OPTIONS env var before build.py is run.
export ASAN_OPTIONS=detect_leaks=1
./tools/build.py -C target/debug -j2
./target/debug/test_cc
- |-
# Release build and test
bash -c "sleep 2100; pkill ninja; pkill cargo" &
DENO_BUILD_MODE=release ./tools/build.py -j2
./tools/build.py -C target/release -j2
DENO_BUILD_MODE=release ./tools/test.py
- |-

8
libdeno/snapshot_creator.cc
View file

@ -34,11 +34,13 @@ int main(int argc, char** argv) {
source_map_fn != nullptr ? source_map.c_str() : nullptr);
auto snapshot = deno_get_snapshot(d);
std::string snapshot_str(reinterpret_cast<char*>(snapshot.data_ptr),
snapshot.data_len);
std::ofstream file_(snapshot_out_bin, std::ios::binary);
file_ << snapshot_str;
file_.write(reinterpret_cast<char*>(snapshot.data_ptr), snapshot.data_len);
file_.close();
delete[] snapshot.data_ptr;
deno_delete(d);
return file_.bad();
}