You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 42 Next »

The write() method defined in the class java.io.OutputStream takes an argument of type int intended to be between 0 and 255. Because a value of type int may be outside this range, failure to range check can result in the truncation of the higher order bits of the input.

The general contract for the write() method says that it writes one byte to the output stream. The byte to be written constitutes the eight lower order bits of the argument b, passed to the write() method. The 24 high-order bits of b are ignored. [[API 2006]]

Noncompliant Code Example

This noncompliant code example accepts a value from the user without validating it. Any value that is not in the range of 0 to 255 is truncated. For instance, write(303) prints / because the lower order bits of 303 are preserved while the top 24 order bits are lost (303 mod 256 is 47 and / has ASCII code 47). That is, the result is remainder modulo 256 of the absolute value of the input.

class ConsoleWrite {
  public static void main(String[] args) { 
    //Any input value > 255 will result in unexpected output
    System.out.write(Integer.valueOf(args[0].toString()));
    System.out.flush();
  }
}

Compliant Solution

Use alternative means to output integers such as the System.out.print* methods.

class ConsoleWrite {
  public static void main(String[] args) { 
    System.out.println(args[0]);
  }
}

Compliant Solution (2)

Alternatively, perform range checking to be compliant. While this particular compliant solution still does not display the original out-of-range integer correctly, it behaves well when the corresponding read() method is used to convert the byte value back to a value of type int. This is because it guarantees that the byte variable will contain representable data.

class FileWrite {
  public static void main(String[] args) throws NumberFormatException, IOException { 
    FileOutputStream  out = new FileOutputStream("output");   

    //Perform range checking  
    if(Integer.valueOf(args[0]) < 0 || Integer.valueOf(args[0]) > 255) {
      throw new ArithmeticException("Value is out of range");
    }
  
    out.write(Integer.valueOf(args[0].toString()));
    System.out.flush(); 
  }
}

Note that a NumberFormatException results when attempting to use the Integer.valueOf() method, if args0 contains a value that is larger than the maximum value of an int.

Compliant Solution (3)

This compliant solution uses the writeInt() method of the DataOutputStream class.

class FileWrite {
  public static void main(String[] args) throws NumberFormatException, IOException { 
    FileOutputStream out = new FileOutputStream("output");   
    DataOutputStream dos = new DataOutputStream(out);
    dos.writeInt(Integer.valueOf(args[0].toString()));
    // close out and dos 
  }     
}

Risk Assessment

Using the write() method to output integers may result in unexpected values.

Rule

Severity

Likelihood

Remediation Cost

Priority

Level

INT09- J

low

unlikely

medium

P2

L3

Automated Detection

TODO

Related Vulnerabilities

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

References

[[API 2006]] method write()
[[Harold 1999]]


INT08-J. Provide mechanisms to handle unsigned data when required      06. Integers (INT)      07. Floating Point (FLP)

  • No labels