Thanks for jumping on this, everyone. It's a big, bad one.
I created a formal commit out of Mike's patch. In a separate commit, I also removed the numtype variable declaration to reduce a bit of noise at compile-time, per Jeff's note, and returned some spaces to tabs (sigh) in one line of Mike's patch.
I've pushed those two commits to a branch called lp1164575 in the security repo at <email address hidden>:security/Evergreen. And I've applied the patch to our production system as of an hour ago (using the hot-fix methods documented at http://evergreen-ils.org/blog/?p=884) to get some real-world testing in.
Thanks for jumping on this, everyone. It's a big, bad one.
I created a formal commit out of Mike's patch. In a separate commit, I also removed the numtype variable declaration to reduce a bit of noise at compile-time, per Jeff's note, and returned some spaces to tabs (sigh) in one line of Mike's patch.
I've pushed those two commits to a branch called lp1164575 in the security repo at <email address hidden> :security/ Evergreen. And I've applied the patch to our production system as of an hour ago (using the hot-fix methods documented at http:// evergreen- ils.org/ blog/?p= 884) to get some real-world testing in.