Versions Compared

Key

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

...

  • Leading dashes: Leading dashes can cause problems when programs are called with the file name as a parameter because the first character or characters of the file name might be interpreted as an option switch.
  • Control characters, such as newlines, carriage returns, and escape: Control characters in a file name can cause unexpected results from shell scripts and in logging.
  • Spaces: Spaces can cause problems with scripts and when double quotes aren't used to surround the file name.
  • Invalid character encodings: Character encodings can make it difficult to perform proper validation of file and path names. (See rule IDS11-J. Eliminate non-character code points before validation.)
  • Name-space separation characters: Including name-space separation characters in a file or path name can cause unexpected and potentially insecure behavior.
  • Command interpreters, scripts, and parsers: Some characters have special meaning when processed by a command interpreter, shell, or parser and should consequently be avoided.

...

This is an instance of rule IDS00-J. Sanitize untrusted data passed across a trust boundary.

Noncompliant Code Example

...

Code Block
bgColor#ffcccc
File f = new File("A\uD8AB");
OutputStream out = new FileOutputStream(f);

A Each platform is free to define its own mapping of the non-"safe" characters. For example, when tested on an Ubuntu Linux distribution, this noncompliant code example resulted in the following file name:

...

This noncompliant code example creates a file with input from the user without sanitizing itthe input.

Code Block
bgColor#FFCCCC
public static void main(String[] args) throws Exception {
  if (args.length < 1) {
    // handle error
  }
  File f = new File(args[0]);
  OutputStream out = new FileOutputStream(f);
  // ...
}

No checks are performed on the file name to prevent troublesome characters. If an attacker knew this code was in a program used to create or rename files that would later be used in a script or automated process of some sort, they the attacker could choose particular characters in the output file name to confuse the later process for malicious purposes.

...

Code Block
bgColor#ccccFF
public static void main(String[] args) throws Exception {
  if (args.length < 1) {
    // handle error
  }
  String filename = args[0];

  Pattern pattern = 
    Pattern.compile("[^AIDS05-J. Use a subset of ASCII for file"
                    + " and path names^A-Za-z0-9%&+,.:=_]");
  Matcher matcher = pattern.matcher(filename);
  if (matcher.find()) {
    // filename contains bad chars, handle error
  }
  File f = new File(filename);
  OutputStream out = new FileOutputStream(f);
  // ...
}

...

Risk Assessment

Failing to use only the a "safe" subset of ASCII that is guaranteed to work can result in misinterpreted data.

...

CERT C Secure Coding Standard

MSC09-C. Character Encoding - Use Subset of ASCII for Safety

CERT C++ Secure Coding Standard

MSC09-CPP. Character Encoding - Use Subset of ASCII for Safety

<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="7c676710babefc17-f8684f49-45b44007-aa67a6c9-16d533b0373f032e1b07860a"><ac:plain-text-body><![CDATA[

[ISO/IEC TR 24772:2010

http://www.aitcnet.org/isai/]

" Choice of Filenames filenames and other External Identifiers external identifiers [AJN] "

]]></ac:plain-text-body></ac:structured-macro>

MITRE CWE

CWE-116, ". Improper Encoding encoding or Escaping escaping of Output" output

Bibliography

ISO/IEC 646-1991

ISO 7-bit coded character set for information interchange

<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="ddb95c3ae86a27d2-3f5dc554-45d64c68-b766af48-257ac7b63c3ac69f4c87fc8f"><ac:plain-text-body><![CDATA[

[[Kuhn 2006

AA. Bibliography#Kuhn 06]]

UTF-8 and Unicode FAQ for UNIX/Linux

]]></ac:plain-text-body></ac:structured-macro>

<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="a0be5fce1e608ed8-b1506cf8-4bab49ea-8ad09a80-ba1b0ed2fd81109753fa30c5"><ac:plain-text-body><![CDATA[

[[Wheeler 2003

AA. Bibliography#Wheeler03]]

5.4 File Names

]]></ac:plain-text-body></ac:structured-macro>

<ac:structured-macro ac:name="unmigrated-wiki-markup" ac:schema-version="1" ac:macro-id="97a7a301ea12bd6b-93948cf9-46574f6b-98cf8171-fc9af19c97cb5380c6e23685"><ac:plain-text-body><![CDATA[

[[VU#439395

AA. Bibliography#VU439395]]

]]></ac:plain-text-body></ac:structured-macro>

...