commit 15b21b7e8bc76f8c2abf29d80a7d5ee1738b462d
author Treehugger Robot <treehugger-gerrit@google.com> Wed May 13 09:24:07 2020 +0000
committer Maciej Zenczykowski <maze@google.com> Wed May 13 16:34:23 2020 +0000
tree d4212f769f61372e2e736d60dfe062681f95d047
parent 4696979412c53211daec954de4b74afc616c632a

TetherController - use %u not %d for uint32_t formatting

(not sure why this is not triggering a compiler warning of some kind)

Tested: builds
Bug: 150736748
Signed-off-by: Maciej Żenczykowski <maze@google.com>
Change-Id: Icd20698fbc98dd8537921fe18d1a68f554095d9c
Merged-In: Icd20698fbc98dd8537921fe18d1a68f554095d9c

server/TetherController.cpp

diff --git a/server/TetherController.cpp b/server/TetherController.cpp
index dd8622b..09eb1ec 100644
--- a/server/TetherController.cpp
+++ b/server/TetherController.cpp
@@ -1142,7 +1142,7 @@
         std::string dst = l2ToString(value.macHeader.h_dest, sizeof(value.macHeader.h_dest));
         inet_ntop(AF_INET6, &key.neigh6, addr, sizeof(addr));
 
-        dw.println("%d %s -> %d %s %s %04x", key.iif, addr, value.oif, src.c_str(), dst.c_str(),
+        dw.println("%u %s -> %u %s %s %04x", key.iif, addr, value.oif, src.c_str(), dst.c_str(),
                    ntohs(value.macHeader.h_proto));
 
         return Result<void>();
@@ -1158,7 +1158,7 @@
     dw.println("BPF stats (downlink): iif -> packets bytes errors");
     const auto printStatsMap = [&dw](const uint32_t& key, const TetherStatsValue& value,
                                      const BpfMap<uint32_t, TetherStatsValue>&) {
-        dw.println("%d -> %" PRIu64 " %" PRIu64 " %" PRIu64, key, value.rxPackets, value.rxBytes,
+        dw.println("%u -> %" PRIu64 " %" PRIu64 " %" PRIu64, key, value.rxPackets, value.rxBytes,
                    value.rxErrors);
 
         return Result<void>();