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.

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

Merge develop into master (B2CA-1730) #88

Merged
merged 66 commits into from
Jul 29, 2024
Merged

Merge develop into master (B2CA-1730) #88

merged 66 commits into from
Jul 29, 2024

Conversation

tdejoigny-ledger
Copy link
Contributor

No description provided.

- Reconstruct the app based on the latest guidelines.
- Make the code easier to test.
- Make it easier to add new features.
overcat and others added 27 commits July 10, 2024 15:08
Comment on lines +1420 to +1424
// for (uint8_t i = 0; i < 16; i++) {
// PRINTF("sub_invocation_positions[%d]=%d\n",
// i,
// envelope->soroban_authorization.sub_invocation_positions[i]);
// }

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +1282 to +1291
// static bool parse_fee_bump_transaction_ext(buffer_t *buffer)
// {
// uint32_t ext;
// PARSER_CHECK(parse_uint32(buffer, &ext))
// if (ext != 0)
// {
// return false;
// }
// return true;
// }

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +1089 to +1091
// for (uint8_t i = 0; i < 16; i++) {
// PRINTF("sub_invocation_positions[%d]=%d\n", i, op->sub_invocation_positions[i]);
// }

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +31 to +33
// #ifdef HAVE_NBGL
// #include "nbgl_use_case.h"
// #endif

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +159 to +162
// TODO: SDK bug, waiting for fix
// if (G_context.unverified_contracts) {
// op_type |= BLIND_OPERATION;
// }

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +635 to +642
switch (claimant->type) {
case CLAIMANT_TYPE_V0:
PARSER_CHECK(parse_account_id(buffer, &claimant->v0.destination))
PARSER_CHECK(parse_claimant_predicate(buffer))
return true;
default:
return false;
}

Check notice

Code scanning / CodeQL

No trivial switch statements Note

This switch statement should either handle more cases, or be rewritten as an if statement.
Comment on lines +369 to +378
switch (liquidity_pool_type) {
case LIQUIDITY_POOL_CONSTANT_PRODUCT: {
PARSER_CHECK(parse_asset(buffer, &liquidity_pool_parameters->constant_product.asset_a))
PARSER_CHECK(parse_asset(buffer, &liquidity_pool_parameters->constant_product.asset_b))
PARSER_CHECK(parse_int32(buffer, &liquidity_pool_parameters->constant_product.fee))
return true;
}
default:
return false;
}

Check notice

Code scanning / CodeQL

No trivial switch statements Note

This switch statement should either handle more cases, or be rewritten as an if statement.
#include "apdu/dispatcher.h"

// Global variables
swap_or_ui_t G;

Check notice

Code scanning / CodeQL

Short global name Note

Poor global variable name 'G'. Prefer longer, descriptive names for globals (eg. kMyGlobalConstant, not foo).
return true;
}

static bool parse_operation(buffer_t *buffer, operation_t *operation) {

Check warning

Code scanning / CodeQL

Poorly documented large function Warning

Poorly documented function: fewer than 2% comments for a function of 101 lines.
return STELLAR_PLUGIN_RESULT_UNAVAILABLE;
}

stellar_plugin_result_t token_plugin_query_data_pair(const uint8_t *contract_address,

Check warning

Code scanning / CodeQL

Poorly documented large function Warning

Poorly documented function: fewer than 2% comments for a function of 150 lines.
@ledger-wiz-cspm-secret-detection

Wiz Scan Summary

IaC Misconfigurations 0C 0H 2M 0L 2I
Sensitive Data 0C 0H 0M 0L 0I
Total 0C 0H 2M 0L 2I
Secrets 0🔑

@tdejoigny-ledger tdejoigny-ledger merged commit 1d98a4f into master Jul 29, 2024
275 of 277 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants