#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: |
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 (2)
comment:1 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:2 by , 14 years ago
Note:
See TracTickets
for help on using tickets.
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.