Commit e2cfe61f authored May 07, 2021 by Mike Yu

Exclude the latency in DnsStats if it is relevant to EPERM

The latency from permission error should not be counted because
there is no real DNS packet sent out. This will help us see a more
correct server latency in DNS statistics.

Original change: https://android-review.googlesource.com/c/platform/packages/modules/DnsResolver/+/1708169

Bug: 188153519
Test: cd packages/modules/DnsResolver && atest
Change-Id: Id4d4f422421daf71680ce87ea345b0e494bd71fc
Merged-In: Id4d4f422421daf71680ce87ea345b0e494bd71fc

parent 5921a874

res_send.cpp

+1 −1

Original line number	Diff line number	Diff line
		@@ -595,9 +595,9 @@ int res_nsend(res_state statp, const uint8_t* buf, int buflen, uint8_t* ans, int
		// TODO: Replace usable_servers of legacy stats with new one.
		resolv_cache_add_resolver_stats_sample(
		statp->netid, revision_id, serverSockAddr, sample, params.max_samples);
		}
		resolv_stats_add(statp->netid, receivedServerAddr, dnsQueryEvent);
		}
		}

		if (resplen == 0) continue;
		if (fallbackTCP) {