codeql-coding-standards icon indicating copy to clipboard operation
codeql-coding-standards copied to clipboard

`ARR32-C`: Improve alert message and links

Open lcartey opened this issue 1 year ago • 0 comments

Affected rules

  • ARR32-C

Description

Address the non-critical review issues from https://github.com/github/codeql-coding-standards/pull/274#discussion_r1151171993:

  • The alert message should be changed to "Variable-length array size derives from a potentially overflowing or wrapping expression." to capture uncertainty in the overflow library (it only tells you if the expression may overflow).
  • Add links in the overflow cases to the overflowing expression and the relevant dimension expression.

lcartey avatar Mar 29 '23 14:03 lcartey