Fix incorrect maximum size when reading dbb parameter values in SHOW DATABASE (#8413)

* Fix incorrect maximum size when reading dbb parameter values

Incorrect maximum size occur for isc_info_oldest_transaction, isc_info_oldest_active, isc_info_oldest_snapshot, isc_info_next_transaction.
Also change maximum size for all parameter values, so we don't get the same error in future.

* Cast transaction number to correct type

---------

Co-authored-by: Artyom Ivanov <[email protected]>

Author: TreeHunter9

src/isql/show.epp

@@ -438,12 +438,12 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case isc_info_page_size:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			isqlGlob.printf("PAGE_SIZE %" SQUADFORMAT"%s", value_out, separator);
 			break;
 
 		case isc_info_db_size_in_pages:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(NUMBER_PAGES, msg, SafeArg() << value_out);
@@ -454,7 +454,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case fb_info_pages_used:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(NUMBER_USED_PAGES, msg, SafeArg() << value_out);
@@ -465,7 +465,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case fb_info_pages_free:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(NUMBER_FREE_PAGES, msg, SafeArg() << value_out);
@@ -476,7 +476,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case fb_info_crypt_state:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				Firebird::string s;
@@ -506,7 +506,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case isc_info_sweep_interval:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(SWEEP_INTERV, msg, SafeArg() << value_out);
@@ -517,32 +517,32 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case isc_info_forced_writes:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			isqlGlob.printf("Forced Writes are %s%s", (value_out == 1 ? "ON" : "OFF"), separator);
 			break;
 
 		case isc_info_oldest_transaction :
-			value_out = p.getInt();
-			isqlGlob.printf("Transaction - oldest = %" SQUADFORMAT"%s", value_out, separator);
+			value_out = p.getBigInt();
+			isqlGlob.printf("Transaction - oldest = %" UQUADFORMAT"%s", static_cast<TraNumber>(value_out), separator);
 			break;
 
 		case isc_info_oldest_active :
-			value_out = p.getInt();
-			isqlGlob.printf("Transaction - oldest active = %" SQUADFORMAT"%s", value_out, separator);
+			value_out = p.getBigInt();
+			isqlGlob.printf("Transaction - oldest active = %" UQUADFORMAT"%s", static_cast<TraNumber>(value_out), separator);
 			break;
 
 		case isc_info_oldest_snapshot :
-			value_out = p.getInt();
-			isqlGlob.printf("Transaction - oldest snapshot = %" SQUADFORMAT"%s", value_out, separator);
+			value_out = p.getBigInt();
+			isqlGlob.printf("Transaction - oldest snapshot = %" UQUADFORMAT"%s", static_cast<TraNumber>(value_out), separator);
 			break;
 
 		case isc_info_next_transaction :
-			value_out = p.getInt();
-			isqlGlob.printf("Transaction - Next = %" SQUADFORMAT"%s", value_out, separator);
+			value_out = p.getBigInt();
+			isqlGlob.printf("Transaction - Next = %" UQUADFORMAT"%s", static_cast<TraNumber>(value_out), separator);
 			break;
 
 		case isc_info_base_level:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(BASE_LEVEL, msg, SafeArg() << value_out);
@@ -553,21 +553,21 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case isc_info_limbo:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (translate)
 			{
 				IUTILS_msg_get(LIMBO, msg, SafeArg() << value_out);
 				isqlGlob.printf("%s%s", msg, separator);
 			}
 			else
-				isqlGlob.printf("Transaction in limbo = %" SQUADFORMAT"%s", value_out, separator);
+				isqlGlob.printf("Transaction in limbo = %" UQUADFORMAT"%s", static_cast<TraNumber>(value_out), separator);
 			break;
 
 		case isc_info_ods_version:
 			isqlGlob.major_ods = p.getInt();
 			break;
 		case isc_info_ods_minor_version:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			isqlGlob.printf("ODS = %" SLONGFORMAT".%" SQUADFORMAT"%s",
 					(SLONG) isqlGlob.major_ods, value_out, separator);
 			break;
@@ -578,7 +578,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 			break;
 
 		case fb_info_protocol_version:
-			value_out = p.getInt();
+			value_out = p.getBigInt();
 			if (value_out)
 				isqlGlob.printf("Protocol version = %" SQUADFORMAT"%s", value_out, separator);
 			else
@@ -656,7 +656,7 @@ void SHOW_dbb_parameters(Firebird::IAttachment* db_handle,
 
 		case fb_info_replica_mode:
 			{
-				value_out = p.getInt();
+				value_out = p.getBigInt();
 				const char* mode =
 					(value_out == fb_info_replica_none) ? "NONE" :
 					(value_out == fb_info_replica_read_only) ? "READ_ONLY" :