netd: NatController: don't setup iptables hooks in constructor.

iptables top-level chain updates should happen within CommandListener()
when it invokes the various modules' setupIptablesHooks().
And remove the extra DROP rule.


Change-Id: I33d2cfbd5444516f855ff85152c472352944cc77
bandwidthcontroller: hide iptables errors when they don't matter

Some commands are run to be able to recover after failures.
Those cleanup commands are generally allowed to fail.
But the lower level system commands would log an error.
Now that error is hidden if nobody will care about the result.
A "#define LOG_NDEBUG 0" will show those failing commands.

Removed leftover LOG_NDEBUG in CommandListener.

Bug: 6377175
Change-Id: I1205fb077f7d0496969bd2a0b5da42025bc5a8dc
am e4c501c3: netd: NatController: don\'t setup iptables hooks in constructor.

* commit 'e4c501c3a78f21d6d6e4ff0ba9090185cff1be8c':
  netd: NatController: don't setup iptables hooks in constructor.
am ad729ac1: bandwidthcontroller: hide iptables errors when they don\'t matter

* commit 'ad729ac11e080b1f1bdd9cd32807d9ae74ce2a63':
  bandwidthcontroller: hide iptables errors when they don't matter
am e07effe6: netd: Netlink: Use KOBJECT_UEVENT instead of IDLETIMER

* commit 'e07effe6f8e9340dbee9428b29672adfb647c413':
  netd: Netlink: Use KOBJECT_UEVENT instead of IDLETIMER
Run dnsmasq with options --keep-in-foreground and --pid-file.

Tell dnsmasq to run with option --keep-in-foreground instead
of --no-daemon, which is meant for debug purposes only. Also
use option --pid-file to tell dnsmasq not use a pid file.

Change-Id: I82858d911e94407b2511d8ce8388be553f2f09f6
am f66d6e9d: BandwidthController: hookup qtaguid for tracking closest to devices.

* commit 'f66d6e9db6be3e94b80c59fab99e237d2e2968c5':
  BandwidthController: hookup qtaguid for tracking closest to devices.
am b1d24094: BandwidthController: Avoid setting a uid multiple times in NaughtyApps

* commit 'b1d24094c2c5d48bbb3dfad4a0551ff0bf77ce6c':
  BandwidthController: Avoid setting a uid multiple times in NaughtyApps
am 29e8de22: BandwidthController: Fix double counting

* commit '29e8de22ad806bdcaa971ffeeb360c9f908aa346':
  BandwidthController: Fix double counting
am 40baed83: netd: NetlinkHandler: explicitly ignore "platform" and "backlight" events

* commit '40baed83f89e8d8f834cdfa94d0615076463c04b':
  netd: NetlinkHandler: explicitly ignore "platform" and "backlight" events
Reconcile with jb-release nakasi-factoryrom-release

Change-Id: I51d86c45479eb7be5e06325d344a6d1c21fd5247
am 034ed176: Reconcile with jb-release nakasi-factoryrom-release

* commit '034ed1763cf10f060f73d10ee6b4abef896fc3ef':
  Do not send extra data the client is not consuming.
Allow additional interface flags. Do not merge

Cherry-pick from master to fix b/6537483

Bug: 5686458
Change-Id: I9c5147446636d32bb3a420a05199870cca5f013d
am c9ed2f47: (-s ours) Allow additional interface flags. Do not merge

* commit 'c9ed2f4706fa46e41a6ebbcd9a529d75c1ac84c5':
  Allow additional interface flags. Do not merge
Merge "Run dnsmasq with options --keep-in-foreground and --pid-file."
resolved conflicts for merge of e36a3a2f to jb-dev-plus-aosp

Change-Id: I566b0e96327f318313c17a11d35539554b075f59
am 9997f9a3: resolved conflicts for merge of e36a3a2f to jb-dev-plus-aosp

* commit '9997f9a3edac2baac244c2fe8e49685a3be9f482':
  Run dnsmasq with options --keep-in-foreground and --pid-file.
Clean up softap interface

Remove old IOCTL based interface

Change-Id: I5c46964d025848ce8a0e0195ff213ccd8b33d508
Merge "Clean up softap interface"
netd: interface: Allow setcfg command without addresses

Change-Id: I47ebe05b2f3daf9754b4baf2e62d46dea76f67b1
Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>