marc export can blow up with bad records

Bug #1350345 reported by Jason Stephenson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned
2.6
Fix Released
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.

Revision history for this message
Jason Stephenson (jstephenson) wrote :
Changed in evergreen:
status: New → Confirmed
milestone: none → 2.next
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.