Overly-Broad Throws Declaration

Revision as of 16:19, 28 September 2008 by KirstenS (Talk | contribs)

Jump to: navigation, search

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

Last revision (mm/dd/yy): 09/28/2008

Vulnerabilities Table of Contents

ASDR Table of Contents


The method throws a generic exception making it harder for callers to do a good job of error handling and recovery.

Declaring a method to throw Exception or Throwable makes it difficult for callers to do good error handling and error recovery. Java's exception mechanism is set up to make it easy for callers to anticipate what can go wrong and write code to handle each specific exceptional circumstance. Declaring that a method throws a generic form of exception defeats this system.

Risk Factors



The following method throws three types of exceptions.

	public void doExchange()
	  throws IOException, InvocationTargetException,
			 SQLException {

While it might seem tidier to write

	public void doExchange()
	  throws Exception {

doing so hampers the caller's ability to understand and handle the exceptions that occur. Further, if a later revision of doExchange() introduces a new type of exception that should be treated differently than previous exceptions, there is no easy way to enforce this requirement.

Related Attacks

Related Vulnerabilities

Related Controls

Related Technical Impacts