MDEV-37104 Fix potential buffer overrun reported by the compiler.

vaintroub · vaintroub · commit c8cf8d352899 · 2025-06-27T21:21:57.000+02:00
net_store_length(buf, length) might need 9 bytes buffer for parameter
length bigger than 16M.
diff --git a/sql/log_event_server.cc b/sql/log_event_server.cc
@@ -7124,7 +7124,7 @@ static inline
 void store_compressed_length(String &str_buf, ulonglong length)
 {
   // Store Type and packed length
-  uchar buf[4];
+  uchar buf[9];
   uchar *buf_ptr = net_store_length(buf, length);
 
   str_buf.append(reinterpret_cast<char *>(buf), buf_ptr-buf);

Original file line number	Diff line number	Diff line change
`@@ -7124,7 +7124,7 @@ static inline`
`7124`	`7124`	`void store_compressed_length(String &str_buf, ulonglong length)`
`7125`	`7125`	`{`
`7126`	`7126`	`// Store Type and packed length`
`7127`		`- uchar buf[4];`
	`7127`	`+ uchar buf[9];`
`7128`	`7128`	`uchar *buf_ptr = net_store_length(buf, length);`
`7129`	`7129`
`7130`	`7130`	`str_buf.append(reinterpret_cast<char *>(buf), buf_ptr-buf);`