Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Note
titleDeprecated

This guideline has been deprecated by

Integer types smaller than int are promoted when an operation is performed on them. If all values of the original type can be represented as an int, the value of the smaller type is converted to an int; otherwise, it is converted to an unsigned int (see INT02-C. Understand integer conversion rules). If the conversion is to a wider type, the original value is zero-extended for unsigned values or zerosign-extended for signed types. Arithmetic operations performed on ints yield the same values as on chars and shorts (at least in the low-order bits). HoweverConsequently, bitwise operations on integer types smaller than int may have unexpected results..

Noncompliant Code Example

This noncompliant code example demonstrates how performing bitwise operations on short types can result in unexpected behaviorinteger types smaller than int may have unexpected results.

Code Block
bgColor#FFcccc
langc

uint8_t port = 0x5aU0x5a;
uint8_t result_8 = ( ~port ) >> 4;

In this example, a bitwise complement of port is negated, first computed and then shifted 4 bits to the right. If both of these operations were are performed on a an 8-bit unsigned integer, then result_8 would will have the value 0x0aU 0x0a. However, port will is first be promoted to a 32-bit integer (signed or unsigned, depending on implementation) signed int, with the following results (on a typical architecture where type int is 32 bits wide):

Expression

Type

Value

Notes

port

uint8_t

0x0000005a

0x5a


~port

int

0xffffffa5


~port >> 4

int

0x0ffffffa

Whether or not value is negative is implementation-defined.

result_8

uint8_t

0xfa


Compliant Solution

In this compliant solution, we truncate the negation back down the bitwise complement of port is converted back to 8 bits. Consequently, result_8 receives is assigned the expected value of 0x0aU.

Code Block
bgColor#ccccff
langc

uint8_t port = 0x5aU0x5a;
uint8_t result_8 = (uint8_t) (~port) >> 4;

Risk Assessment

Bitwise operations on shorts and chars can produce incorrect data.

Recommendation

Severity

Likelihood

Remediation Cost

Priority

Level

EXP14-C

low

likely

high

P3

L3

Automated Detection

Tool

Version

Checker

Description

Section

Compass/ROSE

 

 

Astrée
Include Page
Astrée_V
Astrée_V

Supported
Axivion Bauhaus Suite

Include Page
Axivion Bauhaus Suite_V
Axivion Bauhaus Suite_V

CertC-EXP14Fully implemented
CodeSonar
Include Page
CodeSonar_V
CodeSonar_V
LANG.CAST.RIPRisky integer promotion
Compass/ROSE




ECLAIR

Include Page
ECLAIR_V
ECLAIR_V

CC2.EXP14

Fully implemented

Parasoft C/C++test
Include Page
Parasoft_V
Parasoft_V

CERT_C-EXP14-a

Avoid mixing arithmetic of different precisions in the same expression
 

Related Vulnerabilities

Search for vulnerabilities resulting from the violation of this rule on the CERT website.

Related Guidelines

...

...

Bibliography


...

Image Removed      03. Expressions (EXP)      Image Added Image Added Image Modified