Division and modulo operations are susceptible to divide-by-zero errors.
C identifies two conditions under which division and modulo operations result in undefined behavior:
UB | Description |
The value of the second operand of the | |
n/a | If the quotient a/b is not representable, the behavior of both a/b and a%b is undefined (6.5.5). |
Division
The result of the /
operator is the quotient from the division of the first arithmetic operand by the second arithmetic operand. Division operations are susceptible to divide-by-zero errors. Overflow can also occur during two's complement signed integer division when the dividend is equal to the minimum (negative) value for the signed integer type and the divisor is equal to −1. (See INT32-C. Ensure that operations on signed integers do not result in overflow.)
Noncompliant Code Example
This noncompliant code example can result in a divide-by-zero error during the division of the signed operands sl1
and sl2
:
signed long sl1; signed long sl2; signed long result; /* Initialize sl1 and sl2 */ result = sl1 / sl2;
Compliant Solution
This compliant solution tests the suspect division operation to guarantee there is no possibility of divide-by-zero errors or signed overflow:
signed long sl1; signed long sl2; signed long result; /* Initialize sl1 and sl2 */ if ( (sl2 == 0) || ( (sl1 == LONG_MIN) && (sl2 == -1) ) ) { /* Handle error condition */ } else { result = sl1 / sl2; }
Modulo
The modulo operator provides the remainder when two operands of integer type are divided.
Noncompliant Code Example
This noncompliant code example can result in a divide-by-zero error during the modulo operation on the signed operands sl1
and sl2
:
signed long sl1; signed long sl2; signed long result; /* Initialize sl1 and sl2 */ result = sl1 % sl2;
Compliant Solution
This compliant solution tests the suspect modulo operation to guarantee there is no possibility of a divide-by-zero error or an overflow error:
signed long sl1; signed long sl2; signed long result; /* Initialize sl1 and sl2 */ if ( (sl2 == 0 ) || ( (sl1 == LONG_MIN) && (sl2 == -1) ) ) { /* Handle error condition */ } else { result = sl1 % sl2; }
Risk Assessment
A divide by zero can result in abnormal program termination and denial of service.
Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
---|---|---|---|---|---|
INT33-C | low | likely | medium | P6 | L2 |
Automated Detection
Tool | Version | Checker | Description |
---|---|---|---|
Compass/ROSE | Can detect some violations of this rule. In particular, it ensures that all operations involving division or modulo are preceded by a check ensuring that the second operand is nonzero | ||
Coverity | 6.5 | DIVIDE_BY_ZERO | Fully Implemented |
Fortify SCA | 5.0 | Can detect violations of this rule with CERT C Rule Pack | |
9.7.1 | 43 D | Partially implemented | |
PRQA QA-C | Unable to render {include} The included page could not be found. | 2830 (C) | Fully implemented |
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this rule on the CERT website.
Related Guidelines
Bibliography
[Seacord 2013] | Chapter 5, "Integer Security" |
[Warren 2002] | Chapter 2, "Basics" |