diff options
| author | Serhii Kostiuk <serhii.o.kostiuk@globallogic.com> | 2020-06-05 13:55:04 +0300 |
|---|---|---|
| committer | Serhii Kostiuk <serhii.o.kostiuk@globallogic.com> | 2020-06-05 13:55:04 +0300 |
| commit | c9b6389ac4e53315e7d543091c821ea4b9771ed3 (patch) | |
| tree | 727ca1ecf1b91033dacd37447f6d70040c94eff4 /src | |
| parent | 73a5dc4711da4d21a7f650aec4534ecb3691ca15 (diff) | |
| download | libmts-io-c9b6389ac4e53315e7d543091c821ea4b9771ed3.tar.gz libmts-io-c9b6389ac4e53315e7d543091c821ea4b9771ed3.tar.bz2 libmts-io-c9b6389ac4e53315e7d543091c821ea4b9771ed3.zip | |
[GP-651] LNA7: Allow to start the OMA DM procedure when it is required
Fixed totally invalid string formatting for the stepCb argument.
The previous implementation caused Json::Value to select the following constructor:
```Value(const char* begin, const char* end); ///< Copy all, incl zeroes.```
Which, of course, resulted in OOM violations. Oops.
Diffstat (limited to 'src')
| -rw-r--r-- | src/MTS_IO_QuectelRadio.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/MTS_IO_QuectelRadio.cpp b/src/MTS_IO_QuectelRadio.cpp index 360d988..4498919 100644 --- a/src/MTS_IO_QuectelRadio.cpp +++ b/src/MTS_IO_QuectelRadio.cpp @@ -498,7 +498,7 @@ ICellularRadio::CODE QuectelRadio::startOmaDm(ICellularRadio::UpdateCb& stepCb) if (sResponse.find(sOdmAbnormal) != std::string::npos) { printError("%s| OMA DM procedure failed due to internal error: [%s]", getName().c_str(), sResponse.c_str()); if (stepCb) { - stepCb(Json::Value("OMA DM Error: OMA DM failed due to internal error: [%s]", sOdmAbnormal.c_str())); + stepCb(Json::Value("OMA DM Error: OMA DM failed due to internal error")); } eCode = FAILURE; break; |