Closes the responseWriter after serving the Answer

Nino-K · Nino-K · commit d40fc0467391 · 2023-01-11T12:19:46.000-08:00
Currently we are leaving the response writer open for both
connected sockets (TCP) and non-connected sockets (UPD),
this change closes the response writer accordingly.

Signed-off-by: Nino Kodabande &lt;nkodabande@suse.com&gt;
diff --git a/pkg/hostagent/dns/dns.go b/pkg/hostagent/dns/dns.go
@@ -150,6 +150,7 @@ func (h *Handler) handleQuery(w dns.ResponseWriter, req *dns.Msg) {
 		reply   dns.Msg
 		handled bool
 	)
+	defer w.Close()
 	reply.SetReply(req)
 	logrus.Debugf("handleQuery received DNS query: %v", req)
 	for _, q := range req.Question {

Original file line number	Diff line number	Diff line change
`@@ -150,6 +150,7 @@ func (h Handler) handleQuery(w dns.ResponseWriter, req dns.Msg) {`
`150`	`150`	`reply dns.Msg`
`151`	`151`	`handled bool`
`152`	`152`	`)`
	`153`	`+ defer w.Close()`
`153`	`154`	`reply.SetReply(req)`
`154`	`155`	`logrus.Debugf("handleQuery received DNS query: %v", req)`
`155`	`156`	`for _, q := range req.Question {`