[PATCH] Use deep copy for dns_domain and discovery_domain

[Lukas Slebodnik]

ehlo,

srv_data->dns_domain and srv_data->discovery_domain pointed to the same memory.

srv_data->dns_domain was freed in function resolve_srv_done()
but srv_data->discovery_domain was used on another place.

If there is a reason, that dns_domain and discovery_domain should be the same,
then It is possible to unite them to one "domain" member

Patch is attached

LS

[Jakub Hrozek]

On Mon, Jun 03, 2013 at 12:44:19PM +0200, Lukas Slebodnik wrote:
>  ehlo,
>  
>  srv_data->dns_domain and srv_data->discovery_domain pointed to the same memory.
>  
>  srv_data->dns_domain was freed in function resolve_srv_done()
>  but srv_data->discovery_domain was used on another place.
>  
>  If there is a reason, that dns_domain and discovery_domain should be the same,
>  then It is possible to unite them to one "domain" member
>  
>  Patch is attached
>  
>  LS 
Ack!

[Jakub Hrozek]

On Mon, Jun 03, 2013 at 03:06:02PM +0200, Jakub Hrozek wrote:
>  On Mon, Jun 03, 2013 at 12:44:19PM +0200, Lukas Slebodnik wrote:
>  > ehlo,
>  > 
>  > srv_data->dns_domain and srv_data->discovery_domain pointed to the same
> memory.
>  > 
>  > srv_data->dns_domain was freed in function resolve_srv_done()
>  > but srv_data->discovery_domain was used on another place.
>  > 
>  > If there is a reason, that dns_domain and discovery_domain should be the same,
>  > then It is possible to unite them to one "domain" member
>  > 
>  > Patch is attached
>  > 
>  > LS
>  
>  Ack! 
Pushed to master.