[AXISCPP-707] Incorrect use of delete in AxisEngineException::processException()

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: current (nightly)
• Component/s: Server - Engine
• Environment: Windows XP Pro, VS2003
• Assignee: Marcus Dushshantha Chandradasa
• Reporter: Henrik Nordberg
• Created: 22/Jun/05 03:47
• Updated: 01/Jul/05 12:28
• Resolved: 01/Jul/05 12:28

Description

There is a delete statement in AxisEngineException::processException() which
has a couple of problems:
First, if you were supposed to delete the pointer passed in, delete[] should
be used (in C++). But, at least in the case of the exception thrown when a
library fails to load, a constant character pointer is passed in (in this
case "") – see this line in XMLParserFactory::loadLib():
throw AxisEngineException(SERVER_ENGINE_LOADING_PARSER_FAILED,
PLATFORM_LOADLIB_ERROR);

Why not just use a const std::string& instead of char*?

Issue Links

• relates to: AXISCPP-237 dlopen failures should show reason for failure