Closed Bug 1486137 Opened Last year Closed Last year

TRR: wrong family define used in code sending the ECS

Categories

(Core :: Networking: DNS, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: bagder, Assigned: bagder)

References

Details

(Whiteboard: [necko-triaged][trr])

Attachments

(3 files)

A totally harmless mistake, but still:

In TRR.cpp we have this:

144:    aBody += AF_INET; // FAMILY, 2 octets

... but that family field does not use the AF defines, it uses the IANA family numbers registry [1] so by setting it to AF_INET (2) it actually means IPv6, while it should set it to (1) to mean IPv4.

Since the code is sending a zero size address, sending IPv6 or IPv4 doesn't matter but by using this constant, the code is misleading the reader.

[1] = https://www.iana.org/assignments/address-family-numbers/address-family-numbers.xhtml
... since AF_INET wouldn't actually use IPv4!

MozReview-Commit-ID: FCf7psawENI
make this block on the original ecs bug and set the affected flags please.
Comment on attachment 9004197 [details]
bug 1486137 - TRR: fix the FAMILY field in the ECS code

Patrick McManus [:mcmanus] has approved the revision.
Attachment #9004197 - Flags: review+
Pushed by daniel@haxx.se:
https://hg.mozilla.org/integration/autoland/rev/0192efea1b66
TRR: fix the FAMILY field in the ECS code r=mcmanus
Blocks: 1466462
Backed out changeset 0192efea1b66 (bug 1486137) for failing at netwerk/test/unit/test_trr.js on a CLOSED TREE

Backout link: https://hg.mozilla.org/integration/autoland/rev/a360d3279b9f2ac82a7e81f3878a6eff63a466e8

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=0192efea1b664db8d20a691401ec5d3f9d3a2122

Log link: https://treeherder.mozilla.org/logviewer.html#?job_id=196188366&repo=autoland&lineNumber=2106

Log snippet: 

[task 2018-08-28T13:19:41.850Z] 13:19:41     INFO -  TEST-START | netwerk/test/unit/test_trr.js
[task 2018-08-28T13:19:42.315Z] 13:19:42  WARNING -  TEST-UNEXPECTED-FAIL | netwerk/test/unit/test_trr.js | xpcshell return code: 0
[task 2018-08-28T13:19:42.316Z] 13:19:42     INFO -  TEST-INFO took 464ms
[task 2018-08-28T13:19:42.317Z] 13:19:42     INFO -  >>>>>>>
[task 2018-08-28T13:19:42.317Z] 13:19:42     INFO -  (xpcshell/head.js) | test MAIN run_test pending (1)
[task 2018-08-28T13:19:42.318Z] 13:19:42     INFO -  PID 13323 | start!
[task 2018-08-28T13:19:42.321Z] 13:19:42     INFO -  TEST-PASS | netwerk/test/unit/test_trr.js | run_test - [run_test : 18] "44462" != null
[task 2018-08-28T13:19:42.322Z] 13:19:42     INFO -  TEST-PASS | netwerk/test/unit/test_trr.js | run_test - [run_test : 19] "44462" != ""
[task 2018-08-28T13:19:42.322Z] 13:19:42     INFO -  (xpcshell/head.js) | test pending (2)
[task 2018-08-28T13:19:42.322Z] 13:19:42     INFO -  PID 13323 | starting test 0
Flags: needinfo?(daniel)
That was indeed a very stupid mistake. Fixing and doing a new take soon.
Flags: needinfo?(daniel)
... since AF_INET wouldn't actually use IPv4!

MozReview-Commit-ID: FCf7psawENI
Comment on attachment 9005578 [details]
bug 1486137 - TRR: fix the FAMILY field in the ECS code

Valentin Gosu [:valentin] has approved the revision.
Attachment #9005578 - Flags: review+
Pushed by daniel@haxx.se:
https://hg.mozilla.org/integration/autoland/rev/d808a15ac4bc
TRR: fix the FAMILY field in the ECS code r=valentin
https://hg.mozilla.org/mozilla-central/rev/d808a15ac4bc
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
MozReview-Commit-ID: DzOu1dRlWtg
Comment on attachment 9008656 [details]
bug 1486137 - fix the stream ID check when counting TRR streams

Nicholas Hurley [:nwgh][:hurley] (he/him) has approved the revision.
Attachment #9008656 - Flags: review+
Pushed by daniel@haxx.se:
https://hg.mozilla.org/integration/autoland/rev/4dfb623bb11e
fix the stream ID check when counting TRR streams r=nwgh
Argh, this commit landed with the wrong bug number. It is for bug 1489121.
Backout by cbrindusan@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/0fc842ad0629
Backed out changeset 4dfb623bb11e for landing with the wrong bug number. a=backout
You need to log in before you can comment on or make changes to this bug.