[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH] Verbindung nach DNS-Request schließen



On Tue, Nov 10, 2009 at 05:29:11PM +0100, Alexander Hartmut Kluth wrote:
> + dns.c: Nach Request wird die Verbindung nun geschlossen
> 
> Signed-off-by: Alexander Hartmut Kluth <hartmut@xxxxxxxxxx>
> ---
>  src/modules/tcpip/dns.c |    5 +++--
>  1 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/src/modules/tcpip/dns.c b/src/modules/tcpip/dns.c
> index 0f6e18d..df36fd8 100644
> --- a/src/modules/tcpip/dns.c
> +++ b/src/modules/tcpip/dns.c
> @@ -274,7 +274,8 @@ dword dns_request(char* domain)
>                      result = *((dword*) (reply + 10));
>                      dns_cache_add(domain, result);
>                      
> -                    // TODO Verbindung schliessen
> +                    tcp_close_connection(conn);
> +
>                      return result;
>                  } else {
>                      reply += answer_len;
> @@ -286,7 +287,7 @@ dword dns_request(char* domain)
>          wait_for_rpc();
>      }
>  
> -    // TODO Verbindung schliessen
> +    tcp_close_connection(conn);
>  
>      return 0;
>  }

Weiter oben gibt es noch ein return mehr, dort sollte man die Verbindung
auch schliessen.

Das richtige Problem mit dem Patch ist aber, dass das Auflösen eines
Domainnamens jetzt ewig dauert. Ich nehme an, er bleibt beim Schließen
in der Schleife hängen und kommt nur durch einen Timeout wieder raus.
Das müsste man auf jeden Fall debuggen, bevor ich den Patch reinschieben
kann.