[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.