V687. Size of an array calculated by the sizeof() operator was added to a pointer. It is possible that the number of elements should be calculated by sizeof(A)/sizeof(A[0]).


The analyzer has detected an issue when an array size is added to a pointer, which is strange. Perhaps it is an error, and it is actually the number of the array items instead of its size that should be added to the pointer.

Note. It is safe to work with arrays consisting of bytes (char/unsigned char).

An example of the error:

int A[10];
...
std::sort(A, A + sizeof(A));

The function's first argument is a random-access iterator addressing the position of the first element in the range to be sorted.

The function's second argument is a random-access iterator addressing the position one past the final element in the range to be sorted.

The function call is incorrect: by mistake, the array size is added to the pointer which results in the function trying to sort more elements than necessary.

To fix the bug, the code should be rewritten so that the pointer is summed with the number of array items:

int A[10];
...
std::sort(A, A + sizeof(A) / sizeof(A[0]));

According to Common Weakness Enumeration, potential errors found by using this diagnostic are classified as CWE-682.


Bugs Found

Checked Projects
363
Collected Errors
13 495