From 938d3e852f8ef51f9f284e13e3a79c2f501f6d74 Mon Sep 17 00:00:00 2001 From: Guillaume Nault Date: Wed, 2 Nov 2016 20:34:16 +0100 Subject: pppd-compat: move computation of tx_bytes and rx_bytes tx_bytes and rx_bytes are only used in the ->stop_time branch, so let's compute them here rather than at the top of the function. Also, let's replace '4294967296llu' by 'UINT32_MAX + 1', to make it clear that a gigaword equals 2^32 bytes and is used to keep track of how many times the 32 bits tx and rx counters have overflowed. Signed-off-by: Guillaume Nault --- accel-pppd/extra/pppd_compat.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'accel-pppd/extra') diff --git a/accel-pppd/extra/pppd_compat.c b/accel-pppd/extra/pppd_compat.c index 5518c61..d54743c 100644 --- a/accel-pppd/extra/pppd_compat.c +++ b/accel-pppd/extra/pppd_compat.c @@ -525,14 +525,10 @@ static void build_addr(struct ipv6db_addr_t *a, uint64_t intf_id, struct in6_add static void fill_env(char **env, char *mem, struct pppd_compat_pd *pd) { struct ap_session *ses = pd->ses; - uint64_t tx_bytes, rx_bytes; size_t mem_sz = ENV_MEM; int write_sz; int n = 0; - tx_bytes = (uint64_t)ses->acct_tx_bytes + 4294967296llu*ses->acct_output_gigawords; - rx_bytes = (uint64_t)ses->acct_rx_bytes + 4294967296llu*ses->acct_input_gigawords; - env[n] = mem; write_sz = snprintf(mem, mem_sz, "PEERNAME=%s", pd->ses->username); if (write_sz < 0 || write_sz >= mem_sz) @@ -599,6 +595,13 @@ static void fill_env(char **env, char *mem, struct pppd_compat_pd *pd) } if (pd->ses->stop_time) { + uint64_t gword_sz = (uint64_t)UINT32_MAX + 1; + uint64_t tx_bytes; + uint64_t rx_bytes; + + tx_bytes = ses->acct_tx_bytes + gword_sz * ses->acct_output_gigawords; + rx_bytes = ses->acct_rx_bytes + gword_sz * ses->acct_input_gigawords; + env[n] = mem; write_sz = snprintf(mem, mem_sz, "CONNECT_TIME=%lu", (unsigned long)(pd->ses->stop_time - -- cgit v1.2.3