diff options
author | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-10-24 16:54:48 +0700 |
---|---|---|
committer | Vadim Yanitskiy <vyanitskiy@sysmocom.de> | 2020-10-25 15:24:21 +0700 |
commit | c7a750d428cab8af1dadad5fab791220951c1b34 (patch) | |
tree | 43ee4e8f2de46a6e3159c14dec6d2fcb1edad294 | |
parent | 0ff9c9fca3bcf422cd361305ec4a323e2aff5c1f (diff) |
Transceiver: explicitly init m{Rx,Tx}LowerLoopThread
Coverity warns us that a non-static class members:
- mRxLowerLoopThread, and
- mTxLowerLoopThread,
are not initialized in this constructor nor in any functions that
it calls. I don't think it's critical, because we do initialize
them in Transceiver::start(), but let's make them nullptr.
Change-Id: If9e06aa7965f17383ab6599c15945e8ce2703bbf
Fixes: CID#214952
-rw-r--r-- | Transceiver52M/Transceiver.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/Transceiver52M/Transceiver.cpp b/Transceiver52M/Transceiver.cpp index 881633e..4980861 100644 --- a/Transceiver52M/Transceiver.cpp +++ b/Transceiver52M/Transceiver.cpp @@ -136,6 +136,7 @@ Transceiver::Transceiver(const struct trx_cfg *cfg, GSM::Time wTransmitLatency, RadioInterface *wRadioInterface) : cfg(cfg), mClockSocket(-1), + mRxLowerLoopThread(nullptr), mTxLowerLoopThread(nullptr), mTransmitLatency(wTransmitLatency), mRadioInterface(wRadioInterface), mChans(cfg->num_chans), mOn(false), mForceClockInterface(false), mTxFreq(0.0), mRxFreq(0.0), mTSC(0), mMaxExpectedDelayAB(0), |