Commit f37450f
committed
dns: use length for building TXT string
Rely on the length reported by C-Ares rather than `\0`-termination
for creating the JS string for a dns TXT response.
Fixes: #30688
PR-URL: #30690
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 099d3fd commit f37450fCopy full SHA for f37450f
File tree
Expand file treeCollapse file tree
2 files changed
+3
-2
lines changedOpen diff view settings
Filter options
- src
- test/parallel
Expand file treeCollapse file tree
2 files changed
+3
-2
lines changedOpen diff view settings
Collapse file
+2-1Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
904 | 904 | |
905 | 905 | |
906 | 906 | |
907 | | - |
| 907 | + |
| 908 | + |
908 | 909 | |
909 | 910 | |
910 | 911 | |
|
Collapse file
test/parallel/test-dns-resolveany.js
Copy file name to clipboardExpand all lines: test/parallel/test-dns-resolveany.js+1-1Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
11 | 11 | |
12 | 12 | |
13 | 13 | |
14 | | - |
| 14 | + |
15 | 15 | |
16 | 16 | |
17 | 17 | |
|
0 commit comments