Fix SQL length initialization for trace prepare event (#8738)

Noremos · Artyom Abakumov · web-flow · commit 32a50dc76ab9 · 2025-09-12T14:41:22.000+03:00
Co-authored-by: Artyom Abakumov &lt;artyom.abakumov@red-soft.ru&gt;
diff --git a/src/jrd/trace/TraceDSQLHelpers.h b/src/jrd/trace/TraceDSQLHelpers.h
@@ -54,11 +54,13 @@ class TraceDSQLPrepare
 		m_start_clock = fb_utils::query_performance_counter();
 
 		static const char empty_string[] = "";
-		if (!m_string_len || !string)
+		if (!string)
 		{
 			m_string = empty_string;
 			m_string_len = 0;
 		}
+		else if (m_string_len == 0)
+			m_string_len = fb_strlen(m_string);
 	}
 
 	~TraceDSQLPrepare()
diff --git a/src/utilities/ntrace/TracePluginImpl.cpp b/src/utilities/ntrace/TracePluginImpl.cpp
@@ -525,6 +525,8 @@ void TracePluginImpl::logRecordStmt(const char* action, ITraceDatabaseConnection
 
 		if (reg)
 		{
+			fb_assert(false);
+
 			string temp;
 			temp.printf(NEWLINE "Statement %" SQUADFORMAT", <unknown, bug?>:" NEWLINE, stmt_id);
 			record.insert(0, temp);

Original file line number	Diff line number	Diff line change
`@@ -54,11 +54,13 @@ class TraceDSQLPrepare`
`54`	`54`	`m_start_clock = fb_utils::query_performance_counter();`
`55`	`55`
`56`	`56`	`static const char empty_string[] = "";`
`57`		`- if (!m_string_len \|\| !string)`
	`57`	`+ if (!string)`
`58`	`58`	`{`
`59`	`59`	`m_string = empty_string;`
`60`	`60`	`m_string_len = 0;`
`61`	`61`	`}`
	`62`	`+ else if (m_string_len == 0)`
	`63`	`+ m_string_len = fb_strlen(m_string);`
`62`	`64`	`}`
`63`	`65`
`64`	`66`	`~TraceDSQLPrepare()`
Original file line number	Diff line number	Diff line change
`@@ -525,6 +525,8 @@ void TracePluginImpl::logRecordStmt(const char* action, ITraceDatabaseConnection`
`525`	`525`
`526`	`526`	`if (reg)`
`527`	`527`	`{`
	`528`	`+ fb_assert(false);`
	`529`	`+`
`528`	`530`	`string temp;`
`529`	`531`	`temp.printf(NEWLINE "Statement %" SQUADFORMAT", <unknown, bug?>:" NEWLINE, stmt_id);`
`530`	`532`	`record.insert(0, temp);`