Start here

Home
About Klocwork
What's new
Fixed issues
Release notes
Installation

Reference

C/C++ checkers
Java checkers
C# checkers
MISRA C 2004 checkers
MISRA C++ 2008 checkers
MISRA C 2012 checkers
MISRA C 2012 checkers with Amendment 1
Commands
Metrics
Troubleshooting
Reference

Product components

C/C++ Integration build analysis
Java Integration build analysis
Desktop analysis
Refactoring
Klocwork Static Code Analysis
Klocwork Code Review
Structure101
Tuning
Custom checkers

Coding environments

Visual Studio
Eclipse for C/C++
Eclipse for Java
IntelliJ IDEA
Other

Administration

Project configuration
Build configuration
Administration
Analysis performance
Server performance
Security/permissions
Licensing
Klocwork Static Code Analysis Web API
Klocwork Code Review Web API

Community

View help online
Visit RogueWave.com
Klocwork Support
Rogue Wave Videos

Legal

Legal information

MISRA.CATCH.WRONGORD

Ellipsis exception handler is not the last one in a try-catch block.

MISRA-C++ Rule 15-3-6 (required): Where multiple handlers are provided in a single try-catch statement or function-try-block for a derived class and some or all of its bases, the handlers shall be ordered most-derived to base class.

Rationale

When testing to see if the type of an exception matches the type of a handler, a derived class exception will match with a handler for its base class. If the base class handler is found before the handler for the derived class, the base class handler will be used. The derived class handler is unreachable code and can never be executed.

Example

// classes used for exception handling
class B { };
class D: public B { };

try
{
   // ...
}

catch ( D &d ) // Compliant — Derived class caught before base class
{
   // ...
}

catch ( B &b ) // Compliant — Base class caught after derived class
{
   // ...
}

// Using the classes from above ...
try
{
   // ...
}

catch ( B &b ) // Non-compliant — will catch derived classes as well
{
   // ...
}

catch ( D &d ) // Non-compliant — Derived class will be caught above
{
               // Any code here will be unreachable,
               // breaking Rule 0—1—1
}