Hi,

On Thu, 2015-04-02 at 21:36 +0300, Slava Monich wrote:
> The ones received over UDP didn't have it.
> ---
>  src/dnsproxy.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/src/dnsproxy.c b/src/dnsproxy.c
> index 9787b68..0698387 100644
> --- a/src/dnsproxy.c
> +++ b/src/dnsproxy.c
> @@ -3467,6 +3467,9 @@ static bool udp_listener_event(GIOChannel *channel, 
> GIOCondition condition,
>               return true;
>       }
>  
> +     req->name = g_strdup(query);
> +     req->request = g_malloc(len);
> +     memcpy(req->request, buf, len);
>       req->timeout = g_timeout_add_seconds(5, request_timeout, req);
>       request_list = g_slist_append(request_list, req);
>  

To me it seems only TCP is using req->name, so not all code paths need
to set it. What issues did you notice before this patch?

Jukka, any comments?

Cheers,

        Patrik


_______________________________________________
connman mailing list
connman@connman.net
https://lists.connman.net/mailman/listinfo/connman

Reply via email to