Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#713 closed defect (fixed)

Library.cgi crash when using depositor

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


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 (2)

comment:1 by ak19, 13 years ago

Resolution: fixed
Status: newclosed

comment:2 by ak19, 13 years ago

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.