File: truncLongCast.md

package info (click to toggle)
cppcheck 2.20.0-2
  • links: PTS, VCS
  • area: main
  • in suites: forky, sid
  • size: 26,924 kB
  • sloc: cpp: 274,396; python: 22,429; ansic: 8,103; sh: 1,094; makefile: 1,041; xml: 987; cs: 291
file content (50 lines) | stat: -rw-r--r-- 1,282 bytes parent folder | download
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
# truncLongCastAssignment and truncLongCastReturn

**Message**: int result is assigned to long variable. If the variable is long to avoid loss of information, then you have loss of information.<br/>
**Category**: Type Safety<br/>
**Severity**: Style<br/>
**Language**: C/C++

## Description

This checker warns when a calculation has type 'int' and it could potentially overflow that and the result is implicitly or explicitly converted to a larger
integer type after the loss of information has already occured.

## Motivation

The motivation of this checker is to catch bugs. Unintentional loss of information.

## How to fix

You can fix these warnings by:
1. Add explicit cast to avoid loss of information
2. Explicitly truncate the result
3. Change type of assigned variable

Before:
```cpp
void foo(int32_t y) {
    int64_t x = y * y; // <- warning
}
```

After (explicit cast):
```cpp
void foo(int32_t y) {
    int64_t x = (int64_t)y * y; // <- 64-bit multiplication
}
```

After (explicitly truncate the result):
```cpp
void foo(int32_t y) {
    int64_t x = (int32_t)(y * y); // redundant cast makes it explicit that your intention is to truncate the result to 32-bit
}
```

After (change type of assigned variable):
```cpp
void foo(int32_t y) {
    int32_t x = y * y;
}
```