Quick question about DBIh_SET_ERR_SV

[Martin J. Evans]

Just a quick question about DBIh_SET_ERR_SV. I've produced a patch for
DBD::Oracle and it failing to handle non-unicode strings but running our
test code against DBI 1.609 produces a segfault. It appears:

DBIh_SET_ERR_SV(h, imp_xxh, errcode_sv, errstr_sv, &sv_undef , &sv_undef);

works fine and

DBIh_SET_ERR_SV(h, imp_xxh, errcode_sv, errstr_sv, Nullsv, Nullsv);

does not, no doubt due to the following code in set_err_sv:

  if (SvREADONLY(state))  state  = sv_mortalcopy(state);

Is that right as they seem inconsistent?

Martin