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

Commit 60c8ce61 authored by Ken Chen's avatar Ken Chen
Browse files

Add toString() to android_net_context 

Added a build-in function instead of tedious logging.

Bug: 256507559
Test: compared logs before and after this change, they are the same.
Change-Id: I533adf41644db0001d29eebeb8c72790d02411ca
parent ec13272e

DnsProxyListener.cpp

+4 −12
Original line number Diff line number Diff line
@@ -816,9 +816,7 @@ void DnsProxyListener::GetAddrInfoHandler::doDns64Synthesis(int32_t* rv, addrinf 
}



void DnsProxyListener::GetAddrInfoHandler::run() {

    LOG(INFO) << "GetAddrInfoHandler::run: {" << mNetContext.app_netid << " "

              << mNetContext.app_mark << " " << mNetContext.dns_netid << " " << mNetContext.dns_mark

              << " " << mNetContext.uid << " " << mNetContext.flags << "}";

    LOG(INFO) << "GetAddrInfoHandler::run: {" << mNetContext.toString() << "}";



    addrinfo* result = nullptr;

    Stopwatch s;
@@ -1000,9 +998,7 @@ DnsProxyListener::ResNSendHandler::ResNSendHandler(SocketClient* c, std::string 
    : Handler(c), mMsg(std::move(msg)), mFlags(flags), mNetContext(netcontext) {}



void DnsProxyListener::ResNSendHandler::run() {

    LOG(INFO) << "ResNSendHandler::run: " << mFlags << " / {" << mNetContext.app_netid << " "

              << mNetContext.app_mark << " " << mNetContext.dns_netid << " " << mNetContext.dns_mark

              << " " << mNetContext.uid << " " << mNetContext.flags << "}";

    LOG(INFO) << "ResNSendHandler::run: " << mFlags << " / {" << mNetContext.toString() << "}";



    Stopwatch s;

    maybeFixupNetContext(&mNetContext, mClient->getPid());
@@ -1235,9 +1231,7 @@ void DnsProxyListener::GetHostByNameHandler::doDns64Synthesis(int32_t* rv, hoste 
}



void DnsProxyListener::GetHostByNameHandler::run() {

    LOG(INFO) << "GetHostByNameHandler::run: {" << mNetContext.app_netid << " "

              << mNetContext.app_mark << " " << mNetContext.dns_netid << " " << mNetContext.dns_mark

              << " " << mNetContext.uid << " " << mNetContext.flags << "}";

    LOG(INFO) << "GetHostByNameHandler::run: {" << mNetContext.toString() << "}";

    Stopwatch s;

    maybeFixupNetContext(&mNetContext, mClient->getPid());

    const uid_t uid = mClient->getUid();
@@ -1399,9 +1393,7 @@ void DnsProxyListener::GetHostByAddrHandler::doDns64ReverseLookup(hostent* hbuf, 
}



void DnsProxyListener::GetHostByAddrHandler::run() {

    LOG(INFO) << "GetHostByAddrHandler::run: {" << mNetContext.app_netid << " "

              << mNetContext.app_mark << " " << mNetContext.dns_netid << " " << mNetContext.dns_mark

              << " " << mNetContext.uid << " " << mNetContext.flags << "}";

    LOG(INFO) << "GetHostByAddrHandler::run: {" << mNetContext.toString() << "}";

    Stopwatch s;

    maybeFixupNetContext(&mNetContext, mClient->getPid());

    const uid_t uid = mClient->getUid();

include/netd_resolv/resolv.h

+6 −0
Original line number Diff line number Diff line
@@ -28,6 +28,7 @@ 


#pragma once



#include <android-base/format.h>

#include <arpa/nameser.h>

#include <netinet/in.h>
@@ -65,6 +66,11 @@ struct android_net_context { 
    unsigned flags;

    // Variable to store the pid of the application sending DNS query.

    pid_t pid = NET_CONTEXT_INVALID_PID;



    std::string toString() const {

        return fmt::format("{} {} {} {} {} {}", app_netid, app_mark, dns_netid, dns_mark, uid,

                           flags);

    }

};



#define NET_CONTEXT_FLAG_USE_LOCAL_NAMESERVERS 0x00000001