Merge pull request #408 from Agoric/bugFix

BUGFIX: deliverOneMessage() accidentally skipped insist() msgs.
Agoric · Jan 14, 2020 · 0bc5ca1 · 0bc5ca1
2 parents 755ebbd + d1cb43f
commit 0bc5ca1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/packages/SwingSet/src/kernel/vatManager.js b/packages/SwingSet/src/kernel/vatManager.js
@@ -291,9 +291,10 @@ export default function makeVatManager(
   async function deliverOneMessage(target, msg) {
     insistMessage(msg);
     const targetSlot = mapKernelSlotToVatSlot(target);
-    if (targetSlot.type === 'object') {
+    const { type } = parseVatSlot(targetSlot);
+    if (type === 'object') {
       insist(parseVatSlot(targetSlot).allocatedByVat, `deliver() to wrong vat`);
-    } else if (targetSlot.type === 'promise') {
+    } else if (type === 'promise') {
       const p = kernelKeeper.getKernelPromise(target);
       insist(p.decider === vatID, `wrong decider`);
     }