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

feat: update base cosmos applicationo to version 2.35.13 #2

Merged
merged 86 commits into from
Oct 25, 2023

Conversation

manu0466
Copy link

No description provided.


/* Error API */

typedef enum CborError {

Check notice

Code scanning / CodeQL

Irregular enum initialization Note

In an enumerator list, the = construct should not be used to explicitly initialize members other than the first, unless all items are explicitly initialized.
if (tagData->tag < tag)
continue;
if (tagData->tag > tag)
tagData = NULL;

Check notice

Code scanning / CodeQL

For loop variable changed in body Note

Loop counters should not be modified in the body of the
loop
.
if (flags & CborValidateShortestFloatingPoint && type > CborHalfFloatType) {
if (type == CborDoubleType) {
valf = (float)val;
if ((double)valf == val)

Check notice

Code scanning / CodeQL

Equality test on floating-point values Note

Equality checks on floating point values can yield unexpected results.
# ifndef CBOR_NO_HALF_FLOAT_TYPE
if (type == CborFloatType) {
valf16 = encode_half(valf);
if (valf == decode_half(valf16))

Check notice

Code scanning / CodeQL

Equality test on floating-point values Note

Equality checks on floating point values can yield unexpected results.
CborSignatureTag = 55799
} CborKnownTags;

/* #define the constants so we can check with #ifdef */

Check notice

Code scanning / CodeQL

Commented-out code Note

This comment appears to contain commented-out code.
Comment on lines +515 to +624
return err;
}
err = cbor_value_leave_container(it, &recursed);
if (err)
return err;
return CborNoError;
}

case CborIntegerType: {
uint64_t val;
err = cbor_value_get_raw_integer(it, &val);
cbor_assert(err == CborNoError); /* can't fail */

break;
}

case CborByteStringType:
case CborTextStringType: {
size_t n = 0;
const void *ptr;

err = cbor_value_begin_string_iteration(it);
if (err)
return err;

while (1) {
CborValue next;
err = _cbor_value_get_string_chunk(it, &ptr, &n, &next);
if (!err) {
err = validate_number(it, type, flags);
if (err)
return err;
}

*it = next;
if (err == CborErrorNoMoreStringChunks)
return cbor_value_finish_string_iteration(it);
if (err)
return err;

if (type == CborTextStringType && flags & CborValidateUtf8) {
err = validate_utf8_string(ptr, n);
if (err)
return err;
}
}

return CborNoError;
}

case CborTagType: {
CborTag tag;
err = cbor_value_get_tag(it, &tag);
cbor_assert(err == CborNoError); /* can't fail */

err = cbor_value_advance_fixed(it);
if (err)
return err;
err = validate_tag(it, tag, flags, recursionLeft - 1);
if (err)
return err;

return CborNoError;
}

case CborSimpleType: {
uint8_t simple_type;
err = cbor_value_get_simple_type(it, &simple_type);
cbor_assert(err == CborNoError); /* can't fail */
err = validate_simple_type(simple_type, flags);
if (err)
return err;
break;
}

case CborNullType:
case CborBooleanType:
break;

case CborUndefinedType:
if (flags & CborValidateNoUndefined)
return CborErrorExcludedType;
break;

case CborHalfFloatType:
case CborFloatType:
case CborDoubleType: {
#ifdef CBOR_NO_FLOATING_POINT
return CborErrorUnsupportedType;
#else
err = validate_floating_point(it, type, flags);
if (err)
return err;
break;
#endif /* !CBOR_NO_FLOATING_POINT */
}

case CborInvalidType:
return CborErrorUnknownType;
}

Check notice

Code scanning / CodeQL

Long switch case Note

Switch has at least one case that is too long:
CborTextStringType (32 lines)
.
};
typedef struct CborEncoder CborEncoder;

static const size_t CborIndefiniteLength = SIZE_MAX;

Check notice

Code scanning / CodeQL

Unused static variable Note

Static variable CborIndefiniteLength is never read.
@manu0466 manu0466 force-pushed the manuel/update-base-ledger-app branch 4 times, most recently from c678307 to bcf0aa8 Compare October 24, 2023 20:21
@manu0466 manu0466 force-pushed the manuel/update-base-ledger-app branch from bcf0aa8 to 7c37632 Compare October 24, 2023 20:29
@manu0466 manu0466 marked this pull request as ready for review October 25, 2023 07:52
@manu0466 manu0466 merged commit 4442f3e into main Oct 25, 2023
39 of 41 checks passed
@manu0466 manu0466 deleted the manuel/update-base-ledger-app branch October 25, 2023 07:53
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.

4 participants