BlindTransaction() and VerifyAmounts() need detailed error return codes

Open

#1,199 opened on Dec 22, 2022 2 comments 0 reactions 1 assignee View on GitHub

Nobody has claimed this yet.

Assessment

This issue has not been assessed yet.

Description

enhancement

The functions BlindTransaction() and VerifyAmounts() each have several failure paths, but only use -1 and false to indicate to the caller that an operation failed. This makes it difficult to report helpful error messages to the user.

The error paths could return an enum indicating the reason for failure, and the callers could then surface error strings to the user.

BlindTransaction(): https://github.com/ElementsProject/elements/blob/master/src/blind.cpp#L286
VerifyAmounts(): https://github.com/ElementsProject/elements/blob/master/src/confidential_validation.cpp#L134

Dominant language: C++
Stars: 1.2k
Forks: 418
Avg merge: 1d 12h
Merged PRs (30d): 13

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

An automatic array with a negative run-time extent has a negative size instead of zero Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

lfortran/lfortran#13475 · Opened Sep 24, 2026
Use ament_cmake_ros_core::ament_ros_defaults instead of setting CMAKE_C_STANDARD or CMAKE_CXX_STANDARD Open

good first issue

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

ros2/message_filters#338 · Opened Sep 24, 2026
Needs documentation update: Qt6 install or build instructions are not provided Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

subsurface/subsurface#4984 · Opened Sep 24, 2026
[Bug] Desktop: DataChannel onBufferedAmountLow / onBufferedAmountChange never fires — missing C++ OnBufferedAmountChange override Open

Difficulty 2/5 1-3 hours Newbie friendliness 75/100

flutter-webrtc/flutter-webrtc#2206 · Opened Sep 24, 2026
litertlm-android AAR ships no consumer ProGuard rules → "mid == null" SIGABRT in minified apps Open

Difficulty 2/5 1-3 hours Newbie friendliness 70/100

google-ai-edge/LiteRT-LM#3739 · Opened Sep 24, 2026

More C++ issues