Fixed #4062: Setting atomic reference to nil crashes the compiler

Ary Borenszweig · Ary Borenszweig · commit 53c8b59c3cd5 · 2017-03-02T18:25:46.000-03:00
diff --git a/spec/std/atomic_spec.cr b/spec/std/atomic_spec.cr
@@ -148,6 +148,16 @@ describe Atomic do
     atomic.get.should eq(2)
   end
 
+  it "#set with nil (#4062)" do
+    atomic = Atomic(String?).new(nil)
+
+    atomic.set("foo")
+    atomic.get.should eq("foo")
+
+    atomic.set(nil)
+    atomic.get.should eq(nil)
+  end
+
   it "#lazy_set" do
     atomic = Atomic.new(1)
     atomic.lazy_set(2).should eq(2)
diff --git a/src/atomic.cr b/src/atomic.cr
@@ -168,7 +168,7 @@ struct Atomic(T)
   # atomic.get     # => 10
   # ```
   def set(value : T)
-    Ops.store(pointerof(@value), value, :sequentially_consistent, true)
+    Ops.store(pointerof(@value), value.as(T), :sequentially_consistent, true)
     value
   end
 
