Fix a TOCTOU and symlink attack in netd.

Bug: 7530471
Change-Id: I01fc79d647831dea903e4b852b874eb7bbed25fb

Reconcile with jb-mr1-factory-release jb-mr1-release - do not merge

Change-Id: I0a50c89812fb0f6e635a02ed4c9db690d749e3cf

merge in jb-mr1-release history after reset to jb-mr1-dev

remove Pan service which is not used anymore.

because we replaced bluez bluetooth stack with bluedroid bluetooth stack.
This is multiple project check-in for ISSUE 6849488 Bluedroid stack, remove system/bluetooth.
Pan service depend on system/bluetooth. So both Pan service and system/bluetooth should be removed.

bug 6849488

Change-Id: I7a5a828b9b4327dbd6b6a13ec7ff4a9129ba0ca5

merge in jb-mr1-release history after reset to jb-mr1-dev

am 78b00d81: Merge "Don\'t include <linux/...> header files directly."

* commit '78b00d81ebb7e2720fe17c116bcfd086bdd1f4e7':
  Don't include <linux/...> header files directly.

Merge "Don't include <linux/...> header files directly."

Don't include <linux/...> header files directly.

These change from kernel release to release, and no longer contain some of
the stuff you need.

Change-Id: I3fc7176cf2246aebfc0aa2a833dfa04ea8d931fc

Demoting a non-critical error to a warning: unable to load net cmd dll.

Bug: 7141069
Change-Id: Ib4326f80fd562d845095bad38f45ceb9f3be119e

merge in jb-mr1-release history after reset to jb-mr1-dev

Add LOG_TAG to NetdConstants.

Bug: 7114117
Change-Id: Ifef2cfca6c1361f89ac68eead58df70efc58a8a7

merge in jb-mr1-release history after reset to jb-mr1-dev

am 8a3da263: Merge "Fix race condition in pthread_create()"

* commit '8a3da263f9fde42efbc54d19465ebd1f83ba35cd':
  Fix race condition in pthread_create()

Merge "Fix race condition in pthread_create()"

Fix race condition in pthread_create()

Do not send "&this->mThread" to pthread_create() as newly
created thread can "delete this" before pthread_create() saves
thread id. This would lead to heap corruption.

Change-Id: I9c4cd5e1002b85985514a36ae542778a783db6fb

merge in jb-mr1-release history after reset to jb-mr1-dev

Fix idletimer iptable rules.

Now idletimer should be triggered by every packet.

Cherry-picked from commit 8c54ec5de3818ff0c0cfc735c817be2516454415 in
master. DO NOT MERGE

Change-Id: I48e6fed5cb48edc51b5fbe87091e5dbb3f862cb7

merge in jb-mr1-release history after reset to jb-mr1-dev

Changes the interface command API from static lib to a dynamic one

Change-Id: I9116491a3f5dfc4590ffab367a224fde37a4fd50

merge in jb-mr1-release history after reset to jb-mr1-dev

Merge "Fix idletimer iptable rules."

netd: Add initial interface driver command support

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

Fix idletimer iptable rules.

Now idletimer should be triggered by every packet.

Change-Id: I6c6ea70cb9b37a58dcb0fd8e88df5f987e5ef150

ndc: get rid of arbritrary command length limit

Change-Id: I6123aa916a1f69bbfba58c26e6f150dfdec8c991

Merge "Changed format of interface class acitivity message."

Merge "Introduce basic firewall controls."

Introduce basic firewall controls.

Creates a separate firewall chain that can be used to allow/deny
traffic based on rules.

Firewall is disabled by default.  When enabled, it supports four
types of rules: allowing traffic based on iface name, based on egress
source IP, based on egress destination IP and port, and based on
local UID.

Bug: 5756357
Change-Id: I97f894dca6bddb93b3c56478c5297f79d727cdab

Changed format of interface class acitivity message.

For the sake of consistency, format of interface class activity message
is changed to "NNN IfaceClass <active/idle> <label>" and using single
response code for interface class activity.

Change-Id: I34dc5c37cc611d05172060b2a1e4ceac0a0f0588

HACK: Need to startup dhcpd for non-system bluetooth stack

Add TODO comments to remove start/stopReverseTethering
Return error code with exit() in child process in startReverseTethering
when executing dhcpcd fails.
This is needed so we can get BT certification.
The cleanup is tracked in stopship b/6820232

Change-Id: I98a1c8aedfa042b1824de7d8c69f2600c9bfcf29

change logx to alogx

Change-Id: Ib535b04a6b18d5d35012795b07a705c6483f343d

Added bluetooth reverse tether start/stop cmd

Consolidate iptables chain management.

Move creation and management of module iptables chains up into
CommandListener, which gives better visibility into ordering.

Change-Id: If0c94187c6e59a20840b035d7241057f45a0f74b

Merge "Add a class lable parameter to idletimer."

Add a class lable parameter to idletimer.

Now the idletimer should be setup by this command:
ndc idletimer add <interface> <timeout> <class label>

Class label can be a number or string to identify device type, such as
mobile or wifi. Idletimer will send label in the netd message instead
of the actual interface name.

Change-Id: Ic8b69dc5e38a4436c74723697905643a1214aa2a

netd: interface: Allow setcfg command without addresses

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

Merge "Clean up softap interface"

Clean up softap interface

Remove old IOCTL based interface

Change-Id: I5c46964d025848ce8a0e0195ff213ccd8b33d508

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.

resolved conflicts for merge of e36a3a2f to jb-dev-plus-aosp

Change-Id: I566b0e96327f318313c17a11d35539554b075f59

Merge "Run dnsmasq with options --keep-in-foreground and --pid-file."

am c9ed2f47: (-s ours) Allow additional interface flags. Do not merge

* commit 'c9ed2f4706fa46e41a6ebbcd9a529d75c1ac84c5':
  Allow additional interface flags. Do not merge

Allow additional interface flags. Do not merge

Cherry-pick from master to fix b/6537483

Bug: 5686458
Change-Id: I9c5147446636d32bb3a420a05199870cca5f013d

am 034ed176: Reconcile with jb-release nakasi-factoryrom-release

* commit '034ed1763cf10f060f73d10ee6b4abef896fc3ef':
  Do not send extra data the client is not consuming.

Reconcile with jb-release nakasi-factoryrom-release

Change-Id: I51d86c45479eb7be5e06325d344a6d1c21fd5247

am 40baed83: netd: NetlinkHandler: explicitly ignore "platform" and "backlight" events

* commit '40baed83f89e8d8f834cdfa94d0615076463c04b':
  netd: NetlinkHandler: explicitly ignore "platform" and "backlight" events

am 29e8de22: BandwidthController: Fix double counting

* commit '29e8de22ad806bdcaa971ffeeb360c9f908aa346':
  BandwidthController: Fix double counting

am b1d24094: BandwidthController: Avoid setting a uid multiple times in NaughtyApps

* commit 'b1d24094c2c5d48bbb3dfad4a0551ff0bf77ce6c':
  BandwidthController: Avoid setting a uid multiple times in NaughtyApps

am f66d6e9d: BandwidthController: hookup qtaguid for tracking closest to devices.

* commit 'f66d6e9db6be3e94b80c59fab99e237d2e2968c5':
  BandwidthController: hookup qtaguid for tracking closest to devices.

merge in jb-release history after reset to jb-dev

netd: NetlinkHandler: explicitly ignore "platform" and "backlight" events

The new vsync api uses uevents to notify userspace... 60 events/sec. :(
Don't log unexpected event's subsystem if it is one of the known ones.

Change-Id: Ie900898b326c9451b76ac5f6ba047870fd62093a

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

BandwidthController: Fix double counting

Only have qtaguid only count in the bw_INPUT/bw_OUTPUT chains
and not in the costly_* subchains.
We don't use --goto anymore.

Change-Id: I2eded2e0fa12b46843d32e74406cfeda85f242d1

merge in jb-release history after reset to jb-dev

am e07effe6: netd: Netlink: Use KOBJECT_UEVENT instead of IDLETIMER

* commit 'e07effe6f8e9340dbee9428b29672adfb647c413':
  netd: Netlink: Use KOBJECT_UEVENT instead of IDLETIMER

am ad729ac1: bandwidthcontroller: hide iptables errors when they don\'t matter

* commit 'ad729ac11e080b1f1bdd9cd32807d9ae74ce2a63':
  bandwidthcontroller: hide iptables errors when they don't matter

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

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

BandwidthController: Avoid setting a uid multiple times in NaughtyApps

Use the naughtyAppUids list before inserting a new app.

Bug: 6292513
Change-Id: I6726b7d589569a6198b8a0ff32c32c0e40ed5319

BandwidthController: hookup qtaguid for tracking closest to devices.

Before, qtaguid would rely on netdev (up/down) callbacks to read out the
dev rx/tx stats.
Now it uses (raw) PREROUTING and (mangle) POSTROUTING iptable hooks.
We use mangle on the out path because nat is not available in ipv6.
The NatController only masquerades in nat POSTROUTING so we don't count
traffic that might get dropped.

Bug: 5444089
Change-Id: I997b8b3c81264a545f1f575baaf0445c0afc8f72

netd: Netlink: Use KOBJECT_UEVENT instead of IDLETIMER

The initial idletimer support required using NETLINK_IDLETIMER
netlink socket type.
Instead we now just re-use the existing KOBJECT_UEVENT socket.

Change-Id: I951f2f29182c7cf7f29a054a0eabc88dc25f6d1f

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

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

merge in jb-release history after reset to jb-dev

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

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

Merge "Store certain MDNS fields as quoted strings"

merge in jb-release history after reset to master

Store certain MDNS fields as quoted strings

They sometimes contain spaces so a space-delimited response requires more care.

bug: 6353048
Change-Id: I84d5cc156bf4d9f1419b7202e9aa00ddb7209b2f

netd: Idletimer vs Nat vs Bandwidth controllers

* modified iptables users to work in controller specific custom chains.
 - each controller only works withing his own custom chains and not the
  top level ones (INPUT, OUTPUT, FORWARD, POSTROUTING,...)
 - CommandListener now invokes setupIptablesHooks() for each controller
   once. That is the only time they are allowed to access the top-level
   chains.

* Added idletimer controller.
  From https://android-git.corp.google.com/g/#/c/180769/2
 - supported commands
   . ndc idletimer enable
   . ndc idletimer add <iface> <timeout>
   . ndc idletimer remove <iface> <timeout_used_during_add>
 There is a framework change elsewhere that receives netlink messages.

Signed-off-by: Ashish Sharma <ashishsharma@google.com>
Signed-off-by: JP Abgrall <jpa@google.com>
Change-Id: Ia57450c09166ce20f21d1e3b49047ef1e98f2a3d

Add "metered" flag to DHCP offer vendor info.

Bug: 6344821
Change-Id: Ic8cd235a1cbdd848ff839f6f0f03161de2090751

merge in jb-release history after reset to master

Initial release of mdns interface.

Uses extern/mdnsresponder and communicate with the framework via nativedaemonconnector.

Change-Id: I5c090528197afa090836d7cb5bf75dfba33ff11c

netd: Add support for interface idletimer netlink notifications.

Change-Id: Ifda5b8ecf68533eea42b133dbe0a581b4401ab11

Merge remote-tracking branch 'goog/ics-aah'

am 19ac8238: am 99c65114: Reconcile with ics-mr1-release

* commit '19ac823851c7f8691d5aecbc5a0149ba72ed806c':

am 99c65114: Reconcile with ics-mr1-release

* commit '99c65114b976fd8ceb54021cede0dd63f8826a02':

Reconcile with ics-mr1-release

Change-Id: I6cf64c289005dccb3d60c972686b7c335aa015a4

am d7a9ff6d: am 17654593: Reconcile with ics-mr1-release

* commit 'd7a9ff6ddc2895d298598e53cf7b967742e0d4da':

am 17654593: Reconcile with ics-mr1-release

* commit '176545930d60a366734bbf70c3b60332a606c01e':

Reconcile with ics-mr1-release

Change-Id: If53997c291c7c51332b57f572fbbfc70f7527166

merge in jb-release history after reset to master

softap: Remove wmm_enabled=1 flag from SoftAP config

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

merge in jb-release history after reset to master

merge in ics-mr1-release history after reset to ics-mr1

merge in ics-mr1-release history after reset to ics-mr1

Merge "We are not reverting this CL since it was a temporary fix and we just submitted the proper one."

We are not reverting this CL since it was a temporary fix and we just submitted the proper one.

Revert "Do not send extra data the client is not consuming."

This reverts commit 6217272e1a41fc9827250c675a0499d2091ec6ac

Merge "Ignore Sigpipe in netd"

Merge "ndc: Add default sequence number to command"

Ignore Sigpipe in netd

Bug:6141847

Ignore sigpipe in netd. This will enable us remove the temporary
changes in DnsProxyListener that we added to temporarily resolve
issue 6138617.

Change-Id: I4bc13ac0fcf9dd0e7641c2d4459b1eab0ed50f2a

merge in ics-mr1-release history after reset to ics-mr1

Do not send extra data the client is not consuming.

Bug:6138617

This is to temporarily fix a problem in netd. In case of an error do
not send extra data that the client is not expecting. This causes a
netd crash.

Change-Id: Iadb375e326577857034c2a7d2e962e4242d4735f

Do not send extra data the client is not consuming.

Bug:6138617

This is to temporarily fix a problem in netd. In case of an error do
not send extra data that the client is not expecting. This causes a
netd crash.

Change-Id: Iadb375e326577857034c2a7d2e962e4242d4735f

ndc: Add default sequence number to command

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

am e1176866: am 88ec4ed2: am 1babab9f: netd: do check on command, not buffer.

* commit 'e1176866629c90bc1403926186bf2b7efc736a2a':

am e2f132b7: (-s ours) am 19d7b786: am 1babab9f: netd: do check on command, not buffer.

* commit 'e2f132b7a224d1c2287292132b93f53b09d617cd':
  netd: do check on command, not buffer.

Merge "Prevent potential stall on dns proxy operations."

am 88ec4ed2: am 1babab9f: netd: do check on command, not buffer.

* commit '88ec4ed21deb1340675ba026fda46e6183c2e219':

am 19d7b786: am 1babab9f: netd: do check on command, not buffer.

* commit '19d7b786eed1c69a2d9d6d0764ad9cf8e97b9f72':
  netd: do check on command, not buffer.

am 1babab9f: netd: do check on command, not buffer.

* commit '1babab9febdbd05b7b6d72e0728f49ee8d4b9bd2':
  netd: do check on command, not buffer.

am 1babab9f: netd: do check on command, not buffer.

* commit '1babab9febdbd05b7b6d72e0728f49ee8d4b9bd2':
  netd: do check on command, not buffer.