This site is the archived OWASP Foundation Wiki and is no longer accepting Account Requests.
To view the new OWASP Foundation website, please visit https://owasp.org

Difference between revisions of "Unsafe Mobile Code"

From OWASP
Jump to: navigation, search
m (Moved page into the right category. See Java space page for me details. Content has not been reviewed in this edit.)
 
(13 intermediate revisions by 2 users not shown)
Line 2: Line 2:
 
{{Template:Fortify}}
 
{{Template:Fortify}}
  
__TOC__
+
Last revision (mm/dd/yy): '''{{REVISIONMONTH}}/{{REVISIONDAY}}/{{REVISIONYEAR}}'''
 +
 
 +
[[ASDR_TOC_Vulnerabilities|Vulnerabilities Table of Contents]]
 +
 
 +
==Description==
 +
Mobile code, such as a Java Applet, is code that is transmitted across a network and executed on a remote machine. Because mobile code developers have little if any control of the environment in which their code will execute, special security concerns become relevant. One of the biggest environmental threats results from the risk that the mobile code will run side-by-side with other, potentially malicious, mobile code. Because all of the popular web browsers execute code from multiple sources together in the same JVM, many of the security guidelines for mobile code are focused on preventing manipulation of your objects' state and behavior by adversaries who have access to the same virtual machine where your program is running.
 +
 
 +
 
 +
===Access Violation===
 +
The program violates secure coding principles for mobile code by returning a private array variable from a public access method.
 +
 
 +
Returning a private array variable from a public access method allows the calling code to modify the contents of the array, effectively giving the array public access and contradicting the intentions of the programmer who made it private.
 +
 
 +
'''Example'''
 +
 
 +
The following Java Applet code mistakenly returns a private array variable from a public access method.
  
[[ASDR Table of Contents]]
+
<pre>
 +
public final class urlTool extends Applet {
 +
private URL[] urls;
 +
public URL[] getURLs() {
 +
return urls;
 +
}
 +
...
 +
}
 +
</pre>
 +
.
  
 +
===Dangerous Array Declaration===
 +
The program violates secure coding principles for mobile code by declaring an array public, final, and static.
  
Last revision (mm/dd/yy): '''{{REVISIONMONTH}}/{{REVISIONDAY}}/{{REVISIONYEAR}}'''
+
In most cases an array declared public, final, and static is a bug. Because arrays are mutable objects, the final constraint requires that the array object itself be assigned only once, but makes no guarantees about the values of the array elements. Since the array is public, a malicious program can change the values stored in the array. In most situations the array should be made private.
  
[[Category:FIXME|This is the text from the old template. This needs to be rewritten using the new template.]]
+
'''Example'''
  
 +
The following Java Applet code mistakenly declares an array public, final, and static.
  
==Description==
+
<pre>
 +
public final class urlTool extends Applet {
 +
public final static URL[] urls;
 +
...
 +
}
 +
</pre>
  
The program violates secure coding principles for mobile code by returning a private array variable from a public access method.
+
===Dangerous Public Field===
 +
The program violates secure coding principles for mobile code by declaring a member variable public but not final.
  
Returning a private array variable from a public access method allows the calling code to modify the contents of the array, effectively giving the array public access and contradicting the intentions of the programmer who made it private.
+
All public member variables in an Applet and in classes used by an Applet should be declared final to prevent an attacker from manipulating or gaining unauthorized access to the internal state of the Applet.
  
For more details about what is mobile code and its security concerns, please see [[:Category:Unsafe Mobile Code]].
+
'''Example'''
  
 +
The following Java Applet code mistakenly declares a member variable public but not final.
  
==Risk Factors==
+
<pre>
 +
public final class urlTool extends Applet {
 +
public URL url;
 +
...
 +
}
 +
</pre>
  
* Talk about the [[OWASP Risk Rating Methodology|factors]] that make this vulnerability likely or unlikely to actually happen
+
===Inner Class===
* Discuss the technical impact of a successful exploit of this vulnerability
+
The program violates secure coding principles for mobile code by making use of an inner class.
* Consider the likely [business impacts] of a successful attack
 
  
 +
Inner classes quietly introduce several security concerns because of the way they are translated into Java bytecode. In Java source code, it appears that an inner class can be declared to be accessible only by the enclosing class, but Java bytecode has no concept of an inner class, so the compiler must transform an inner class declaration into a peer class with package level access to the original outer class. More insidiously, since an inner class can access private fields in their enclosing class, once an inner class becomes a peer class in bytecode, the compiler converts private fields accessed by the inner class into protected fields.
  
==Examples==
+
'''Example'''
  
The following Java Applet code mistakenly returns a private array variable from a public access method.
+
The following Java Applet code mistakenly makes use of an inner class.
  
 
<pre>
 
<pre>
 
public final class urlTool extends Applet {
 
public final class urlTool extends Applet {
private URL[] urls;
+
private final class urlHelper {
public URL[] getURLs() {
+
...
return urls;
 
 
}
 
}
 +
...
 +
}
 +
</pre>
 +
 +
===Public finalize() Method===
 +
The program violates secure coding principles for mobile code by declaring a finalize()method public.
 +
 +
A program should never call finalize explicitly, except to call super.finalize() inside an implementation of finialize(). In mobile code situations, the otherwise error prone practice of manual garbage collection can become a security threat if an attacker can maliciously invoke one of your finalize() methods because it is declared with public access. If you are using finalize() as it was designed, there is no reason to declare finalize() with anything other than protected access.
 +
 +
'''Example'''
 +
 +
The following Java Applet code mistakenly declares a public finalize() method.
 +
 +
<pre>
 +
public final class urlTool extends Applet {
 +
public void finalize() {
 
...
 
...
 +
}
 +
...
 
}
 
}
 +
 
</pre>
 
</pre>
 +
 +
==Risk Factors==
 +
 +
* Talk about the [[OWASP Risk Rating Methodology|factors]] that make this vulnerability likely or unlikely to actually happen
 +
* Discuss the technical impact of a successful exploit of this vulnerability
 +
* Consider the likely [business impacts] of a successful attack
 +
 +
 +
==Examples==
 +
  
  
Line 80: Line 147:
 
[[Category:Implementation]]
 
[[Category:Implementation]]
 
[[Category:Code Snippet]]
 
[[Category:Code Snippet]]
[[Category:Unsafe Mobile Code]]
 
 
[[Category:Vulnerability]]
 
[[Category:Vulnerability]]

Latest revision as of 21:51, 10 November 2017

This is a Vulnerability. To view all vulnerabilities, please see the Vulnerability Category page.

This article includes content generously donated to OWASP by MicroFocus Logo.png

Last revision (mm/dd/yy): 11/10/2017

Vulnerabilities Table of Contents

Description

Mobile code, such as a Java Applet, is code that is transmitted across a network and executed on a remote machine. Because mobile code developers have little if any control of the environment in which their code will execute, special security concerns become relevant. One of the biggest environmental threats results from the risk that the mobile code will run side-by-side with other, potentially malicious, mobile code. Because all of the popular web browsers execute code from multiple sources together in the same JVM, many of the security guidelines for mobile code are focused on preventing manipulation of your objects' state and behavior by adversaries who have access to the same virtual machine where your program is running.


Access Violation

The program violates secure coding principles for mobile code by returning a private array variable from a public access method.

Returning a private array variable from a public access method allows the calling code to modify the contents of the array, effectively giving the array public access and contradicting the intentions of the programmer who made it private.

Example

The following Java Applet code mistakenly returns a private array variable from a public access method.

	public final class urlTool extends Applet {
		private URL[] urls;
		public URL[] getURLs() {
			return urls;
		}
			...
	}

.

Dangerous Array Declaration

The program violates secure coding principles for mobile code by declaring an array public, final, and static.

In most cases an array declared public, final, and static is a bug. Because arrays are mutable objects, the final constraint requires that the array object itself be assigned only once, but makes no guarantees about the values of the array elements. Since the array is public, a malicious program can change the values stored in the array. In most situations the array should be made private.

Example

The following Java Applet code mistakenly declares an array public, final, and static.

	public final class urlTool extends Applet {
		public final static URL[] urls;
		...
	}

Dangerous Public Field

The program violates secure coding principles for mobile code by declaring a member variable public but not final.

All public member variables in an Applet and in classes used by an Applet should be declared final to prevent an attacker from manipulating or gaining unauthorized access to the internal state of the Applet.

Example

The following Java Applet code mistakenly declares a member variable public but not final.

	public final class urlTool extends Applet {
		public URL url;
		...
	}

Inner Class

The program violates secure coding principles for mobile code by making use of an inner class.

Inner classes quietly introduce several security concerns because of the way they are translated into Java bytecode. In Java source code, it appears that an inner class can be declared to be accessible only by the enclosing class, but Java bytecode has no concept of an inner class, so the compiler must transform an inner class declaration into a peer class with package level access to the original outer class. More insidiously, since an inner class can access private fields in their enclosing class, once an inner class becomes a peer class in bytecode, the compiler converts private fields accessed by the inner class into protected fields.

Example

The following Java Applet code mistakenly makes use of an inner class.

	public final class urlTool extends Applet {
		private final class urlHelper {
			...
		}
		...
	}

Public finalize() Method

The program violates secure coding principles for mobile code by declaring a finalize()method public.

A program should never call finalize explicitly, except to call super.finalize() inside an implementation of finialize(). In mobile code situations, the otherwise error prone practice of manual garbage collection can become a security threat if an attacker can maliciously invoke one of your finalize() methods because it is declared with public access. If you are using finalize() as it was designed, there is no reason to declare finalize() with anything other than protected access.

Example

The following Java Applet code mistakenly declares a public finalize() method.

	public final class urlTool extends Applet {
		public void finalize() {
			...
		}
		...
	}

Risk Factors

  • Talk about the factors that make this vulnerability likely or unlikely to actually happen
  • Discuss the technical impact of a successful exploit of this vulnerability
  • Consider the likely [business impacts] of a successful attack


Examples

Related Attacks


Related Vulnerabilities


Related Controls

Related Technical Impacts


References

TBD