Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dba85b45 authored by Erik Kline's avatar Erik Kline Committed by android-build-merger
Browse files

Fix up some comments in IPv6 tethering code.

am: d1dcbefe

Change-Id: Icb77a16f14591aa54cc0302b8a7bd9b377ec5eea
parents 78223aa3 d1dcbefe
Loading
Loading
Loading
Loading
+6 −2
Original line number Diff line number Diff line
@@ -169,7 +169,9 @@ class IPv6TetheringInterfaceServices {
                // We need to be able to send unicast RAs, and clients might
                // like to ping the default router's link-local address.  Note
                // that we never remove the link-local route from the network
                // until Tethering disables tethering on the interface.
                // until Tethering disables tethering on the interface. We
                // only need to add the link-local prefix once, but in the
                // event we add it more than once netd silently ignores EEXIST.
                addedPrefixes.add(LINK_LOCAL_PREFIX);
            }

@@ -177,7 +179,9 @@ class IPv6TetheringInterfaceServices {
                final ArrayList<RouteInfo> toBeAdded = getLocalRoutesFor(addedPrefixes);
                try {
                    // It's safe to call addInterfaceToLocalNetwork() even if
                    // the interface is already in the local_network.
                    // the interface is already in the local_network. Note also
                    // that adding routes that already exist does not cause an
                    // error (EEXIST is silently ignored).
                    mNMService.addInterfaceToLocalNetwork(mIfName, toBeAdded);
                } catch (RemoteException e) {
                    Log.e(TAG, "Failed to add IPv6 routes to local table: ", e);
+1 −1
Original line number Diff line number Diff line
@@ -552,7 +552,7 @@ public class RouterAdvertisementDaemon {
        for (Inet6Address dns : dnses) {
            // NOTE: If the full of list DNS servers doesn't fit in the packet,
            // this code will cause a buffer overflow and the RA won't include
            // include this instance of the option at all.
            // this instance of the option at all.
            //
            // TODO: Consider looking at ra.remaining() to determine how many
            // DNS servers will fit, and adding only those.