marc export can blow up with bad records

Bug #1350345 reported by Jason Stephenson on 2014-07-30
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Undecided
Unassigned
2.6
Undecided
Unassigned

Bug Description

Evergreen: 2.6.0+
OpenSRF: N/A
Postgres: N/A
GNU/Linux: N/A

Chris Sharp reported in IRC this morning that marc_export can blow up on certain brief records when outputting them as USMARC:

http://irc.evergreen-ils.org/evergreen/2014-07-30#i_114153

I supplied a quick patch to trap and report the error using an eval block around the MARC::Record->as_usmarc call that seemed to be the source of the crash.

Chris applied the patch and got output about the offending record and the export continued.

Deciding it was a good idea to add this code to marc_export, and to include similar checks for authority records, and for the output to MARCXML, as well, I made a little branch and opened this bug.

I'll leave it up to others to decide if this is an enhancement or a bug fix that should be backported to 2.6.

Changed in evergreen:
status: New → Confirmed
milestone: none → 2.next
Ben Shum (bshum) wrote :

Let's call this a bug fix. Pushing to master and rel_2_6. Thanks Chris and Jason!

Changed in evergreen:
milestone: 2.next → 2.7.0
status: Confirmed → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers