Fixed bug CORE-5528 : internal Firebird consistency check (limbo impossible (184), file: vio.cpp line: 2379)

Also, make all 3 in-limbo transaction handlers equal (at VIO_chase, VIO_get_current and prepare_update) to be
more consistent

Author: hvlad

src/jrd/vio.cpp

@@ -2294,6 +2294,11 @@ bool VIO_get_current(thread_db* tdbb,
 			VIO_data(tdbb, rpb, pool);
 			return true;
 
+		case tra_limbo:
+			if (!(transaction->tra_flags & TRA_ignore_limbo))
+				ERR_post(Arg::Gds(isc_rec_in_limbo) << Arg::Num(rpb->rpb_transaction_nr));
+			// fall thru
+
 		case tra_active:
 			// clear lock error from status vector
 			fb_utils::init_status(tdbb->tdbb_status_vector);
@@ -2338,10 +2343,6 @@ bool VIO_get_current(thread_db* tdbb,
 			}
 			break;
 
-		case tra_limbo:
-			BUGCHECK(184);		// limbo impossible
-			break;
-
 		default:
 			fb_assert(false);
 		}
@@ -5362,11 +5363,14 @@ static int prepare_update(	thread_db*		tdbb,
 							 Arg::Gds(isc_update_conflict) <<
 							 Arg::Gds(isc_concurrent_transaction) << Arg::Num(update_conflict_trans));
 				}
-			case tra_active:
-				return PREPARE_LOCKERR;
 
 			case tra_limbo:
-				ERR_post(Arg::Gds(isc_deadlock) << Arg::Gds(isc_trainlim));
+				if (!(transaction->tra_flags & TRA_ignore_limbo))
+					ERR_post(Arg::Gds(isc_rec_in_limbo) << Arg::Num(rpb->rpb_transaction_nr));
+				// fall thru
+
+			case tra_active:
+				return PREPARE_LOCKERR;
 
 			case tra_dead:
 				break;