- Fix code analyzer warning, and bail out of handle_tcp_writing

  and handle_tls_writing early when data->query is NULL.

Author: wcawijngaards

doc/ChangeLog

@@ -4,6 +4,8 @@
 5 August 2025: Wouter
 	- Merge #454 from jaredmauch: handle rare case but seen in
 	  production where data->query is NULL.
+	- Fix code analyzer warning, and bail out of handle_tcp_writing
+	  and handle_tls_writing early when data->query is NULL.
 
 18 July 2025: Wouter
 	- Merge #449: Add useful logging for XoT transfers.

server.c

@@ -4726,8 +4726,9 @@ handle_tcp_writing(int fd, short event, void* arg)
 	struct event_base* ev_base;
 	uint32_t now = 0;
 
-	if ((event & EV_TIMEOUT)) {
+	if ((event & EV_TIMEOUT) || !q) {
 		/* Connection timed out.  */
+		/* Or data->query is NULL, in which case nothing to do. */
 		cleanup_tcp_handler(data);
 		return;
 	}
@@ -5329,8 +5330,9 @@ handle_tls_writing(int fd, short event, void* arg)
 	buffer_type* write_buffer;
 	uint32_t now = 0;
 
-	if ((event & EV_TIMEOUT)) {
+	if ((event & EV_TIMEOUT) || !q) {
 		/* Connection timed out.  */
+		/* Or data->query is NULL, in which case nothing to do. */
 		cleanup_tcp_handler(data);
 		return;
 	}