commit | c528bf60a127b82a841c9d349a7f549d61679dfe | [log] [tgz] |
---|---|---|
author | Peter Qiu <zqiu@chromium.org> | Mon Jan 26 11:19:30 2015 -0800 |
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | Tue Jan 27 00:22:00 2015 +0000 |
tree | f793e204c0eba6b9fb8fd9d4591e5f0a90ce16a3 | |
parent | ffa56374b5cfca653f458d072630bb02dbc7bcd7 [diff] |
shill: correctly apply filter for ARP traffic Currently, only one static const variable is defined to store all ARP filters. So only the first filter will work, all subsequent filter will not overwrite the static const variable. Remove the static qualifier so that a new filter will be created and initialized on the stack for each CreateSocket call. BUG=chromium:422159 TEST=USE="asan clang" FEATURES=test emerge-$BOARD shill Manual Test: Verify through debug logs that ARP replies are received during active link monitoring, and ARP requests are received during passive link monitoring. Change-Id: I31ae89db9b70da4c29e9d998e52affed31b7e750 Reviewed-on: https://chromium-review.googlesource.com/243245 Trybot-Ready: Zeping Qiu <zqiu@chromium.org> Tested-by: Zeping Qiu <zqiu@chromium.org> Reviewed-by: Paul Stewart <pstew@chromium.org> Commit-Queue: Zeping Qiu <zqiu@chromium.org>