Chromium Code Reviews
|
|
Chromium Code Reviews
Issue
1463333003:
Increase the maximum MDNS device limit from 64 to 2048. (Closed)
DescriptionIncrease the maximum MDNS device limit from 64 to 2048.
The new size should be large enough so as to not interfere with
the API in the field or in testing labs, but not so large that it
allows malicious persons to bog down Chrome with large forged
MDNS device listings.
R=leilei@chromium.org
CC=mfoltz@chromium.org
Committed: https://crrev.com/aff93dd52022e2ce91fb6f204589b312a5570f0d
Cr-Commit-Position: refs/heads/master@{#361548}
Patch Set 1 #
Messages
Total messages: 24 (9 generated)
lgtm
Description was changed from ========== Increase the maximum MDNS device limit from 64 to 2048. The new size should be large enough so as to not interfere with the API in the field or in testing lab, but not so large that it allows malicious persons to bog down Chrome with large forged MDNS device listings. R=leilei@chromium.org CC=mfoltz@chromium.org ========== to ========== Increase the maximum MDNS device limit from 64 to 2048. The new size should be large enough so as to not interfere with the API in the field or in testing labs, but not so large that it allows malicious persons to bog down Chrome with large forged MDNS device listings. R=leilei@chromium.org CC=mfoltz@chromium.org ==========
The CQ bit was checked by kmarshall@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1463333003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1463333003/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: chromium_presubmit on tryserver.chromium.linux (JOB_FAILED, no build URL)
kmarshall@chromium.org changed reviewers: + rockot@chromium.org
rockot@ for extensions/
ping!
Oops! Sorry, lgtm!
Thankee!
The CQ bit was checked by kmarshall@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1463333003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1463333003/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by kmarshall@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1463333003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1463333003/1
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by kmarshall@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1463333003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1463333003/1
Message was sent while issue was closed.
Committed patchset #1 (id:1)
Message was sent while issue was closed.
Patchset 1 (id:??) landed as https://crrev.com/aff93dd52022e2ce91fb6f204589b312a5570f0d Cr-Commit-Position: refs/heads/master@{#361548} |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
