---

yaml
---
r: 15425
b: refs/heads/try
c: 10236f8
h: refs/heads/master
i:
  15423: 08de5e2
v: v3

Author: brson

[refs]

@@ -2,5 +2,5 @@
 refs/heads/master: 61b1875c16de39c166b0f4d54bba19f9c6777d1a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
-refs/heads/try: 59abf93b799a588e07060c7e4efd5f20cdce4293
+refs/heads/try: 10236f8cd48ad862c32573411acb4423e441f883
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105

branches/try/src/libcore/str.rs

@@ -1509,10 +1509,8 @@ let i = str::as_bytes(\"Hello World\") { |bytes| vec::len(bytes) };
 ~~~
 "]
 fn as_bytes<T>(s: str, f: fn([u8]) -> T) -> T unsafe {
-    let mut v: [u8] = ::unsafe::reinterpret_cast(s);
-    let r = f(v);
-    ::unsafe::forget(v);
-    r
+    let v: *[u8] = ::unsafe::reinterpret_cast(ptr::addr_of(s));
+    f(*v)
 }
 
 #[doc = "
@@ -2449,6 +2447,14 @@ mod tests {
         assert (c == "AAA");
     }
 
+    #[test]
+    #[ignore(cfg(target_os = "win32"))]
+    #[should_fail]
+    fn test_as_bytes_fail() {
+        // Don't double free
+        as_bytes("") {|_bytes| fail }
+    }
+
     #[test]
     fn test_as_buf() unsafe {
         let a = "Abcdefg";

branches/try/src/test/run-fail/issue-2156.rs

@@ -0,0 +1,10 @@
+// error-pattern:explicit failure
+// Don't double free the string
+use std;
+import io::{reader, reader_util};
+
+fn main() {
+    io::with_str_reader("") { |rdr|
+        alt rdr.read_char() { '=' { } _ { fail } }
+    }
+}