diff options
author | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-05-16 15:20:39 +0200 |
---|---|---|
committer | Pau Espin Pedrol <pespin@sysmocom.de> | 2018-05-16 15:20:43 +0200 |
commit | 0b0748a4be1a35673312e08196611b83a2dcb9af (patch) | |
tree | f0c4e3b5fd38ab897cace43d264f5740c9419014 /src/tbf.cpp | |
parent | 45143d270c9c2f1ce4e102fbb1c0456a8e39d787 (diff) |
tbf: Fix memset(0) on object with no trivial copy-assignment
As warned by gcc 8.1.0:
osmo-pcu/src/tbf.cpp: In constructor ‘gprs_rlcmac_tbf::gprs_rlcmac_tbf(BTS*, gprs_rlcmac_tbf_direction)’:
osmo-pcu/src/tbf.cpp:222:33: error: ‘void* memset(void*, int, size_t)’ clearing an object of type ‘struct gprs_rlc’ with no trivial copy-assignment; use assignment or value-initialization instead [-Werror=class-memaccess]
memset(&m_rlc, 0, sizeof(m_rlc));
^
In file included from osmo-pcu/src/tbf.h:24,
from osmo-pcu/src/bts.h:37,
from osmo-pcu/src/tbf.cpp:22:
osmo-pcu/src/rlc.h:234:8: note: ‘struct gprs_rlc’ declared here
struct gprs_rlc {
^~~~~~~~
Change-Id: Ifb0529b9ae6cd4300e5cbbd9151054792edbfe06
Diffstat (limited to 'src/tbf.cpp')
-rw-r--r-- | src/tbf.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tbf.cpp b/src/tbf.cpp index d5fbb3ff..14c1ee28 100644 --- a/src/tbf.cpp +++ b/src/tbf.cpp @@ -219,9 +219,9 @@ gprs_rlcmac_tbf::gprs_rlcmac_tbf(BTS *bts_, gprs_rlcmac_tbf_direction dir) : memset(&pdch, 0, sizeof(pdch)); memset(&T, 0, sizeof(T)); memset(&N, 0, sizeof(N)); - memset(&m_rlc, 0, sizeof(m_rlc)); memset(&gsm_timer, 0, sizeof(gsm_timer)); + m_rlc.init(); m_llc.init(); m_name_buf[0] = '\0'; |