In Java, data is stored in big-endian format (also called network order). That is, all data is represented sequentially starting from the most significant bit (MSB) to the least significant. JDK versions prior to JDK 1.4 required definition of custom methods that manage reversing byte order to maintain compatibility with little-endian systems. Correct handling of byte order related order–related issues is critical when exchanging data in a networked environment that includes both big-endian and little-endian machines or when working with other languages using Java Native Interface (JNI). Failure to handle byte-ordering issues may can cause unexpected program behavior.
Noncompliant Code Example
The read methods (readByte()
, readShort()
, readInt()
, readLong()
, readFloat()
, and readDouble()
) and the corresponding write methods defined by class java.io.DataInputStream
and class java.io.DataOutputStream
operate only on big-endian data. Use of these methods while interoperating with traditional languages, such as C or and C++, is unsafe, insecure because such languages lack any guarantees about endianness. This noncompliant code example shows such a discrepancy. :
Code Block | ||
---|---|---|
| ||
try { DataInputStream dis = null; try { dis = new DataInputStream( new FileInputStream("data")); // Little-endian data might be read as big-endian int serialNumber = dis.readInt(); |
Compliant Solution (Use ByteBuffer
)
} catch (IOException x) {
// Handle error
} finally {
if (dis != null) {
try {
dis.close();
} catch (IOException e) {
// Handle error
}
}
}
}
|
Compliant Solution (ByteBuffer
)
This compliant solution uses methods provided by class ByteBuffer
[API 2014] to correctly extract an int
from the original input value. It wraps the input byte array with a ByteBuffer
, sets the byte order to little-endian, and extracts the int
. The result is stored in the integer serialNumber
. Class ByteBuffer
provides analogous get and put methods for other numeric types. This compliant solution uses methods provided by class {{ByteBuffer}} (see \[[API 2006|AA. Bibliography#API 06]\] [ByteBuffer|http://download.oracle.com/javase/6/docs/api/java/nio/ByteBuffer.html]) to correctly extract an {{int}} from the original input value. It wraps the input byte array with a {{ByteBuffer}}, sets the byte order to little-endian, and extracts the {{int}}. The result is stored in the integer {{serialNumber}}. Wiki Markup
Code Block | ||
---|---|---|
| ||
try { DataInputStream dis = null; try { dis = new DataInputStream( new FileInputStream("data")); byte[] buffer = new byte[4]; int bytesRead = dis.read(buffer); // Bytes are read into buffer if (bytesRead != 4) { throw new IOException("Unexpected End of Stream"); } int serialNumber = ByteBuffer.wrap(buffer).order(ByteOrder.LITTLE_ENDIAN).getInt(); |
...
} finally {
if (dis != null) {
try {
dis.close();
} catch (IOException x) {
// Handle error
}
}
}
} catch (IOException x) {
// Handle error
}
|
Compliant Solution (Define Special-Purpose Methods)
An alternative compliant solution is to define read and write methods that support the necessary byte-swapping while reading from or writing to the file. In this example, the readLittleEndianInteger()
method reads four bytes into a byte buffer and then pieces together the integer in the right correct order. The writeLittleEndianInteger()
method obtains bytes by repeatedly casting the integer so that the least significant byte is extracted on successive right shifts. Long
values can be handled by defining a byte buffer of size eight8.
Code Block | ||
---|---|---|
| ||
// readRead method public static int readLittleEndianInteger(InputStream ips) throws IOException { byte[] buffer = new byte[4]; int check = ips.read(buffer); if (check != 4) { throw new IOException("Unexpected End of Stream"); } int result = (buffer[3] << 24) | (buffer[2] << 16) | (buffer[1] << 8) | buffer[0]; return result; } // writeWrite method public static void writeLittleEndianInteger(int i, OutputStream ops) throws IOException { byte[] buffer = new byte[4]; buffer[0] = (byte) i; buffer[1] = (byte) (i >> 8); buffer[2] = (byte) (i >> 16); buffer[3] = (byte) (i >> 24); ops.write(buffer); } |
Compliant Solution (
...
reverseBytes()
)
When programming for JDK 1.5 +and later, use the reverseBytes()
method defined in the classes Character
, Short
, Integer
, and Long
to reverse the order of the integral value's bytes. Note that classes Float
and Double
lack such a method.
Code Block | ||
---|---|---|
| ||
public static int reverse(int i) { return Integer.reverseBytes(i); } |
Risk Assessment
Reading and writing data without considering endianness may can lead to serious misinterpretations about of both the magnitude and sign , alikeof the data.
Rule | Severity | Likelihood | Remediation Cost | Priority | Level |
---|
FIO12-J |
Low |
Unlikely |
Low | P3 | L3 |
Automated Detection
Automated detection is infeasible in the general case.
Related Vulnerabilities
Search for vulnerabilities resulting from the violation of this guideline on the CERT website.
Bibliography
Wiki Markup |
---|
\[[API 2006|AA. Bibliography#API 06]\] Class [ByteBuffer|http://download.oracle.com/javase/6/docs/api/java/nio/ByteBuffer.html]: Methods {{wrap}} and {{order}}. Class [Integer|http://download.oracle.com/javase/6/docs/api/java/lang/Integer.html]: method {{reverseBytes}}
\[[Harold 1997|AA. Bibliography#Harold 97]\] Chapter 2: "Primitive Data Types, Cross Platform issues"
\[[MITRE 2009|AA. Bibliography#MITRE 09]\] [CWE ID 198|http://cwe.mitre.org/data/definitions/198.html] "Use of Incorrect Byte Ordering" |
Tool | Version | Checker | Description | ||||||
---|---|---|---|---|---|---|---|---|---|
Parasoft Jtest |
| CERT.FIO12.PMRWLED | Provide methods to read and write little-endian data |
Related Guidelines
Bibliography
[API 2014] | Class |
"On Holy Wars and a Plea for Peace" | |
Chapter 2, "Primitive Data Types, Cross-Platform Issues" |
...
INT06-J. Avoid incorrect mixing of signed integers with bitwise operators 06. Integers (INT) INT08-J. Provide mechanisms to handle unsigned data when required