Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG]: "No original smartcontract saved for trigger" #4133

Closed
Arjentix opened this issue Dec 8, 2023 · 0 comments
Closed

[BUG]: "No original smartcontract saved for trigger" #4133

Arjentix opened this issue Dec 8, 2023 · 0 comments
Assignees
@Arjentix @timofeevmd
Labels
Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST QA-confirmed This bug is reproduced and needs a fix

Comments

@Arjentix
Copy link
Contributor

Arjentix commented Dec 8, 2023 

2023-12-08 10:57:01 thread 'sumeragi thread' panicked at 'No original smartcontract saved for trigger. This is a bug.', core/src/smartcontracts/isi/triggers/set.rs:459:22

Originally found by @arndey

By a quick look at the code it seems like the corresponding field or trigger::Set is not (de-)serialized anyhow, which may introduce this bug.

Hower @Mingela pointed that it hapens in a test without loading from snapshot, so the problem needs furthes investigation.
@Arjentix Arjentix added Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST labels Dec 8, 2023
@Arjentix Arjentix self-assigned this Dec 8, 2023
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 13, 2023
@Arjentix
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
542b40a 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 13, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
ce3d9c9 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 18, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
5689acb 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 19, 2023
@Arjentix
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
27b07a4 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 19, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
db103de 
Signed-off-by: Daniil Polyakov <[email protected]>
This was referenced Dec 19, 2023
Merged
Merged
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 19, 2023
@Arjentix
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
7955db8 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 19, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
0505609 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 20, 2023
@Arjentix
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
72fc915 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 20, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
a5bd61b 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 25, 2023
@Arjentix
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
f955940 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit to Arjentix/iroha that referenced this issue Dec 25, 2023
@Arjentix
[BACKPORT] hyperledger-iroha#4133: Count identical wasm for triggers
83cc5c1 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit that referenced this issue Dec 25, 2023
@Arjentix
[BACKPORT] #4133: Count identical wasm for triggers
d67f716 
Signed-off-by: Daniil Polyakov <[email protected]>
Arjentix added a commit that referenced this issue Dec 25, 2023
@Arjentix
[fix] #4133: Count identical wasm for triggers
ba01819 
Signed-off-by: Daniil Polyakov <[email protected]>
@timofeevmd timofeevmd self-assigned this Jan 16, 2024
@timofeevmd timofeevmd added the QA-confirmed This bug is reproduced and needs a fix label Jan 16, 2024
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this issue Jan 22, 2024
@Arjentix @Asem-Abdelhady
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
882ac9b 
Signed-off-by: Daniil Polyakov <[email protected]>
Signed-off-by: Asem-Abdelhady <[email protected]>
Asem-Abdelhady pushed a commit to Asem-Abdelhady/iroha that referenced this issue Feb 9, 2024
@Arjentix @Asem-Abdelhady
[fix] hyperledger-iroha#4133: Count identical wasm for triggers
96b5125 
Signed-off-by: Daniil Polyakov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Arjentix Arjentix

@timofeevmd timofeevmd

Labels
Bug Something isn't working iroha2-dev The re-implementation of a BFT hyperledger in RUST QA-confirmed This bug is reproduced and needs a fix
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Arjentix @timofeevmd