Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Fix: add executed to EOSIO_DISPATCH() #870

Open
wants to merge 1 commit into
base: develop-boxed
Choose a base branch
from

Conversation

b1aafulei
Copy link

Change Description

Related to Issue #868.

In the current code, EOSIO_DISPATCH_INTERNAL assumes the existence of boolean variable executed. There are two external entry points for that:

Path 1: EOSIO_ACTIONS --> EOSIO_DISPATCH_HELPER --> EOSIO_DISPATCH_INTERNAL
Path 2: EOSIO_DISPATCH --> EOSIO_DISPATCH_HELPER --> EOSIO_DISPATCH_INTERNAL

While executed is defined in EOSIO_ACTIONS, it is not defined in EOSIO_DISPATCH. This quick fix adds executed to EOSIO_DISPATCH (and does nothing else), so the assumption of the existence of executed now holds in both paths.

@jeffreyssmith2nd jeffreyssmith2nd changed the base branch from eosio-cdt-2.1-staging-b to eosio-cdt-2.1-staging-c May 12, 2020 17:44
@heifner heifner changed the base branch from eosio-cdt-2.1-staging-c to develop-boxed August 4, 2021 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant