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

Commit c304a157 authored by Lorenzo Colitti's avatar Lorenzo Colitti
Browse files

Only actually log the DNS events we have. 

DnsEventListenerService#logAndClear was always logging 100 DNS
events regardless of how many were actually logged. The rest of
the events were either all zero or previously-recorded events.

Bug: 28204408
Change-Id: I9da3b07a2e7b7512397a3eb6568a877815eb3312
parent bbd26bc0

services/core/java/com/android/server/connectivity/DnsEventListenerService.java

+3 −1
Original line number Diff line number Diff line
@@ -78,9 +78,11 @@ public class DnsEventListenerService extends IDnsEventListener.Stub { 
                return;

            }



            // Only log as many events as we actually have.

            byte[] eventTypes = Arrays.copyOf(mEventTypes, mEventCount);

            byte[] returnCodes = Arrays.copyOf(mReturnCodes, mEventCount);

            int[] latenciesMs = Arrays.copyOf(mLatenciesMs, mEventCount);

            DnsEvent.logEvent(mNetId, mEventTypes, mReturnCodes, mLatenciesMs);

            DnsEvent.logEvent(mNetId, eventTypes, returnCodes, latenciesMs);

            maybeLog(String.format("Logging %d results for netId %d", mEventCount, mNetId));

            mEventCount = 0;

        }