[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[tor-commits] [tor/maint-0.4.0] rust: abort on panic in all profiles
commit fecd583c0e7e580f07d81b24b1a3522c52938c61
Author: cypherpunks <cypherpunks@xxxxxxxxxxxxxx>
Date: Sat Aug 18 19:18:48 2018 +0000
rust: abort on panic in all profiles
Until https://github.com/rust-lang/rust/issues/52652 is fixed,
unwinding on panic is potentially unsound in a mixed C/Rust codebase.
The codebase is supposed to be panic-free already, but just to be safe.
This started mattering at commit d1820c1516a31a149fc51a9e5126bf899e4c4e08.
Fixes #27199; bugfix on tor-0.3.3.1-alpha.
---
changes/bug27199 | 3 +++
src/rust/Cargo.toml | 10 ++++++++++
2 files changed, 13 insertions(+)
diff --git a/changes/bug27199 b/changes/bug27199
new file mode 100644
index 000000000..f9d2a422f
--- /dev/null
+++ b/changes/bug27199
@@ -0,0 +1,3 @@
+ o Minor bugfixes (rust):
+ - Abort on panic in all build profiles, instead of potentially unwinding
+ into C code. Fixes bug 27199; bugfix on 0.3.3.1-alpha.
diff --git a/src/rust/Cargo.toml b/src/rust/Cargo.toml
index 953c9b96b..c22eea984 100644
--- a/src/rust/Cargo.toml
+++ b/src/rust/Cargo.toml
@@ -1,7 +1,17 @@
[workspace]
members = ["tor_util", "protover", "smartlist", "external", "tor_allocate", "tor_rust"]
+# Can remove panic="abort" when this issue is fixed:
+# https://github.com/rust-lang/rust/issues/52652
+[profile.dev]
+panic = "abort"
+
[profile.release]
debug = true
panic = "abort"
+[profile.test]
+panic = "abort"
+
+[profile.bench]
+panic = "abort"
_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits