Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1526)

Unified Diff: runtime/vm/isolate_reload.cc

Issue 2255553002: Fix for heap corruption issue when patch https://codereview.chromium.org/2225243003/ is applied. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/isolate_reload.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/isolate_reload.cc
diff --git a/runtime/vm/isolate_reload.cc b/runtime/vm/isolate_reload.cc
index a00f05e75700ae3baa0892b051a398c71ea9d759..b79d39a76c9f58ff42c04f08d3d95c486047a856 100644
--- a/runtime/vm/isolate_reload.cc
+++ b/runtime/vm/isolate_reload.cc
@@ -206,6 +206,15 @@ void ReasonForCancelling::AppendTo(JSONArray* array) {
}
+ClassReasonForCancelling::ClassReasonForCancelling(Zone* zone,
+ const Class& from,
+ const Class& to)
+ : ReasonForCancelling(zone),
+ from_(Class::ZoneHandle(zone, from.raw())),
+ to_(Class::ZoneHandle(zone, to.raw())) {
+}
+
+
void ClassReasonForCancelling::AppendTo(JSONArray* array) {
JSONObject jsobj(array);
jsobj.AddProperty("type", "ReasonForCancelling");
@@ -413,8 +422,10 @@ void IsolateReloadContext::ReportSuccess() {
class Aborted : public ReasonForCancelling {
public:
- explicit Aborted(Zone* zone, const Error& error)
- : ReasonForCancelling(zone), error_(error) { }
+ Aborted(Zone* zone, const Error& error)
+ : ReasonForCancelling(zone),
+ error_(Error::ZoneHandle(zone, error.raw())) {
+ }
private:
const Error& error_;
« no previous file with comments | « runtime/vm/isolate_reload.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698