]> Cypherpunks repositories - gostls13.git/commit
net: fix LookupTXT of long records on Windows
authorRuss Cox <rsc@golang.org>
Wed, 22 Nov 2017 21:15:11 +0000 (16:15 -0500)
committerRuss Cox <rsc@golang.org>
Thu, 23 Nov 2017 01:17:18 +0000 (01:17 +0000)
commitb6cf58d5b82898a2c92d50aead07163d5bc3c184
treef648fcfc543254e2a7c878d33a1fbbfeba34b1b3
parent571ee0436f388a5304a08a3f029113306bfd54d5
net: fix LookupTXT of long records on Windows

The response to a TXT lookup is a sequence of RRs,
each of which contains a sequence of string fragments.

The correct handling of the response is to do:

    for each rr {
        list = append(list, strings.Join(rr.fragments, ""))
    }

(like in at dnsRR_TXT.Walk, used on most platforms).

The Windows code incorrectly does:

    for each rr {
        list = append(list, rr.fragments...)
    }

This CL fixes it to concatenate fragments, as it must.

Fixes #21472.

Change-Id: I78cce96f172e5e90da9a212b0343457f6d5f92e8
Reviewed-on: https://go-review.googlesource.com/79555
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
src/net/lookup_test.go
src/net/lookup_windows.go