Ticket #713 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

Library.cgi crash when using depositor

Reported by: ak19 Owned by: ak19
Priority: moderate Milestone:
Component: Depositor Severity: major
Keywords: crash Cc:

Description

A bug caused the Depositor to crash library.cgi upon selecting a collection.

The bug: recpt code was previously looping on parameter names in args, applying substring on them in order to compare them to a prefix for equality. However, the loop also tried to substring the arg parameter name when this was shorter than the prefix, causing illegal indexing errors.

The fix for this bug needed to be applied to two locations, which both repeated the error (wizardaction and depositoraction).

Fix committed with code revision 23012, with fixes to: wizardaction.cpp and depositoraction.cpp

Change History

Changed 9 years ago by ak19

  • status changed from new to closed
  • resolution set to fixed

Changed 9 years ago by ak19

A third similar fix was required to runtime-src\src\recpt\depositoraction.cpp.

Still part of bugfix in depositor: depositor now crashed further down the depositing process, because a similar error (error in code logic) existed elsewhere in the code.

Revision 23029.

Note: See TracTickets for help on using tickets.