aboutsummaryrefslogtreecommitdiff
path: root/tests/cfail
diff options
context:
space:
mode:
authorJorge Aparicio <japaricious@gmail.com>2017-04-13 22:57:06 -0500
committerJorge Aparicio <japaricious@gmail.com>2017-04-13 22:58:29 -0500
commit2fb00c7d85f2eee6caa0bc742afd496a6ef7f33c (patch)
tree3f43c203a1cd2b9d0c53c984f8250bdfa8b42ffc /tests/cfail
parent692ad81ad87299a14ed76dc455ceed011b9c5c45 (diff)
fix memory safety hole around `borrow`
Diffstat (limited to 'tests/cfail')
-rw-r--r--tests/cfail/borrow.rs27
-rw-r--r--tests/cfail/lock_mut.rs16
2 files changed, 18 insertions, 25 deletions
diff --git a/tests/cfail/borrow.rs b/tests/cfail/borrow.rs
index fc8638f..6d8ab2a 100644
--- a/tests/cfail/borrow.rs
+++ b/tests/cfail/borrow.rs
@@ -1,22 +1,31 @@
extern crate cortex_m_srp;
-use cortex_m_srp::{C2, C3, C4, P1, Resource};
+use cortex_m_srp::{C1, C2, C3, C4, C5, P2, Resource};
-static R1: Resource<i32, C3> = Resource::new(0);
-static R2: Resource<i32, C2> = Resource::new(0);
-static R3: Resource<i32, C3> = Resource::new(0);
-static R4: Resource<i32, C4> = Resource::new(0);
+static R1: Resource<i32, C4> = Resource::new(0);
+static R2: Resource<i32, C3> = Resource::new(0);
+static R3: Resource<i32, C4> = Resource::new(0);
+static R4: Resource<i32, C5> = Resource::new(0);
+static R5: Resource<i32, C1> = Resource::new(0);
+static R6: Resource<i32, C2> = Resource::new(0);
-fn j1(prio: P1) {
+fn j1(prio: P2) {
R1.lock(&prio, |r1, c3| {
// CAN borrow a resource with ceiling C when the system ceiling SC > C
- let r2 = R2.borrow(&c3);
+ let r2 = R2.borrow(&prio, &c3);
// CAN borrow a resource with ceiling C when the system ceiling SC == C
- let r3 = R3.borrow(&c3);
+ let r3 = R3.borrow(&prio, &c3);
// CAN'T borrow a resource with ceiling C when the system ceiling SC < C
- let r4 = R4.borrow(&c3);
+ let r4 = R4.borrow(&prio, &c3);
//~^ error
+
+ // CAN'T borrow a resource with ceiling C < P (task priority)
+ let r5 = R5.borrow(&prio, &c3);
+ //~^ error
+
+ // CAN borrow a resource with ceiling C == P (task priority)
+ let r6 = R6.borrow(&prio, &c3);
});
}
diff --git a/tests/cfail/lock_mut.rs b/tests/cfail/lock_mut.rs
deleted file mode 100644
index b5e1ae9..0000000
--- a/tests/cfail/lock_mut.rs
+++ /dev/null
@@ -1,16 +0,0 @@
-extern crate cortex_m_srp;
-
-use cortex_m_srp::{C3, C4, P2, Resource};
-
-static R1: Resource<i32, C4> = Resource::new(0);
-static R2: Resource<i32, C3> = Resource::new(0);
-
-fn j1(mut prio: P2) {
- R1.lock_mut(
- &mut prio, |r1: &mut i32, c3| {
- let r2 = R2.borrow(&c3);
- let another_r1: &i32 = R1.borrow(&c3);
- //~^ error
- }
- );
-}