diff options
author | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2019-12-19 04:51:56 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2019-12-19 04:51:56 +0000 |
commit | 187d2ea5707fe646252a692cb5bcc200220e1aa5 (patch) | |
tree | 90523a478016fa06d773cd49869dc9485776f77d | |
parent | f5a85089ce22e88d9a7a4c67cffc4cafcf16ce94 (diff) | |
parent | 28a40f3c5033ab16f665927321348f95fc169be4 (diff) | |
download | platform_hardware_ril-187d2ea5707fe646252a692cb5bcc200220e1aa5.tar.gz platform_hardware_ril-187d2ea5707fe646252a692cb5bcc200220e1aa5.tar.bz2 platform_hardware_ril-187d2ea5707fe646252a692cb5bcc200220e1aa5.zip |
Merge "Encode MCC with 3 digit in CellIdentity" am: 61d7c1ccd0 am: 86d6ae7ca4 am: 28a40f3c50
Change-Id: I16c05087b5a369e41bd4af352cea1cf249bf4b1d
-rw-r--r-- | include/telephony/ril_mcc.h | 71 | ||||
-rwxr-xr-x | libril/ril_service.cpp | 18 |
2 files changed, 81 insertions, 8 deletions
diff --git a/include/telephony/ril_mcc.h b/include/telephony/ril_mcc.h new file mode 100644 index 00000000..dc56b126 --- /dev/null +++ b/include/telephony/ril_mcc.h @@ -0,0 +1,71 @@ +/* + * Copyright (C) 2019 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#ifndef RIL_MCC_H +#define RIL_MCC_H + +#include <climits> +#include <cstdio> +#include <string> + +namespace ril { +namespace util { +namespace mcc { + +/** + * Decode an integer mcc and encode as 3 digit string + * + * @param an integer mcc, its range should be in 0 to 999. + * + * @return string representation of an encoded MCC or an empty string + * if the MCC is not a valid MCC value. + */ +static inline std::string decode(int mcc) { + char mccStr[4] = {0}; + if (mcc > 999 || mcc < 0) return ""; + + snprintf(mccStr, sizeof(mccStr), "%03d", mcc); + return mccStr; +} + +// echo -e "#include \"hardware/ril/include/telephony/ril_mcc.h\"\nint main()"\ +// "{ return ril::util::mcc::test(); }" > ril_test.cpp \ +// && g++ -o /tmp/ril_test -DTEST_RIL_MCC ril_test.cpp; \ +// rm ril_test.cpp; /tmp/ril_test && [ $? ] && echo "passed" +#ifdef TEST_RIL_MCC +static int test() { + const struct mcc_ints { const int in; const char * out; } legacy_mccs[] = { + {INT_MAX, ""}, + {1, "001"}, + {11, "011"}, + {111, "111"}, + {0, "000"}, + {9999, ""}, + {-12, ""}, + }; + + for (int i=0; i < sizeof(legacy_mccs) / sizeof(struct mcc_ints); i++) { + if (decode(legacy_mccs[i].in).compare(legacy_mccs[i].out)) return 1; + } + + return 0; +} +#endif + +} +} +} +#endif /* !defined(RIL_MCC_H) */ diff --git a/libril/ril_service.cpp b/libril/ril_service.cpp index 6220640d..c6556722 100755 --- a/libril/ril_service.cpp +++ b/libril/ril_service.cpp @@ -27,6 +27,7 @@ #include <hwbinder/ProcessState.h> #include <telephony/ril.h> #include <telephony/ril_mnc.h> +#include <telephony/ril_mcc.h> #include <ril_service.h> #include <hidl/HidlTransportSupport.h> #include <utils/SystemClock.h> @@ -3561,7 +3562,7 @@ void fillCellIdentityResponse(CellIdentity &cellIdentity, RIL_CellIdentity_v16 & case RIL_CELL_INFO_TYPE_GSM: { cellIdentity.cellIdentityGsm.resize(1); cellIdentity.cellIdentityGsm[0].mcc = - std::to_string(rilCellIdentity.cellIdentityGsm.mcc); + ril::util::mcc::decode(rilCellIdentity.cellIdentityGsm.mcc); cellIdentity.cellIdentityGsm[0].mnc = ril::util::mnc::decode(rilCellIdentity.cellIdentityGsm.mnc); @@ -3579,7 +3580,7 @@ void fillCellIdentityResponse(CellIdentity &cellIdentity, RIL_CellIdentity_v16 & case RIL_CELL_INFO_TYPE_WCDMA: { cellIdentity.cellIdentityWcdma.resize(1); cellIdentity.cellIdentityWcdma[0].mcc = - std::to_string(rilCellIdentity.cellIdentityWcdma.mcc); + ril::util::mcc::decode(rilCellIdentity.cellIdentityWcdma.mcc); cellIdentity.cellIdentityWcdma[0].mnc = ril::util::mnc::decode(rilCellIdentity.cellIdentityWcdma.mnc); @@ -3608,7 +3609,7 @@ void fillCellIdentityResponse(CellIdentity &cellIdentity, RIL_CellIdentity_v16 & case RIL_CELL_INFO_TYPE_LTE: { cellIdentity.cellIdentityLte.resize(1); cellIdentity.cellIdentityLte[0].mcc = - std::to_string(rilCellIdentity.cellIdentityLte.mcc); + ril::util::mcc::decode(rilCellIdentity.cellIdentityLte.mcc); cellIdentity.cellIdentityLte[0].mnc = ril::util::mnc::decode(rilCellIdentity.cellIdentityLte.mnc); @@ -3626,7 +3627,7 @@ void fillCellIdentityResponse(CellIdentity &cellIdentity, RIL_CellIdentity_v16 & case RIL_CELL_INFO_TYPE_TD_SCDMA: { cellIdentity.cellIdentityTdscdma.resize(1); cellIdentity.cellIdentityTdscdma[0].mcc = - std::to_string(rilCellIdentity.cellIdentityTdscdma.mcc); + ril::util::mcc::decode(rilCellIdentity.cellIdentityTdscdma.mcc); cellIdentity.cellIdentityTdscdma[0].mnc = ril::util::mnc::decode(rilCellIdentity.cellIdentityTdscdma.mnc); @@ -7931,7 +7932,7 @@ void convertRilCellInfoListToHal(void *response, size_t responseLen, hidl_vec<Ce records[i].gsm.resize(1); CellInfoGsm *cellInfoGsm = &records[i].gsm[0]; cellInfoGsm->cellIdentityGsm.mcc = - std::to_string(rillCellInfo->CellInfo.gsm.cellIdentityGsm.mcc); + ril::util::mcc::decode(rillCellInfo->CellInfo.gsm.cellIdentityGsm.mcc); cellInfoGsm->cellIdentityGsm.mnc = ril::util::mnc::decode(rillCellInfo->CellInfo.gsm.cellIdentityGsm.mnc); cellInfoGsm->cellIdentityGsm.lac = @@ -7955,7 +7956,7 @@ void convertRilCellInfoListToHal(void *response, size_t responseLen, hidl_vec<Ce records[i].wcdma.resize(1); CellInfoWcdma *cellInfoWcdma = &records[i].wcdma[0]; cellInfoWcdma->cellIdentityWcdma.mcc = - std::to_string(rillCellInfo->CellInfo.wcdma.cellIdentityWcdma.mcc); + ril::util::mcc::decode(rillCellInfo->CellInfo.wcdma.cellIdentityWcdma.mcc); cellInfoWcdma->cellIdentityWcdma.mnc = ril::util::mnc::decode(rillCellInfo->CellInfo.wcdma.cellIdentityWcdma.mnc); cellInfoWcdma->cellIdentityWcdma.lac = @@ -8003,7 +8004,7 @@ void convertRilCellInfoListToHal(void *response, size_t responseLen, hidl_vec<Ce records[i].lte.resize(1); CellInfoLte *cellInfoLte = &records[i].lte[0]; cellInfoLte->cellIdentityLte.mcc = - std::to_string(rillCellInfo->CellInfo.lte.cellIdentityLte.mcc); + ril::util::mcc::decode(rillCellInfo->CellInfo.lte.cellIdentityLte.mcc); cellInfoLte->cellIdentityLte.mnc = ril::util::mnc::decode(rillCellInfo->CellInfo.lte.cellIdentityLte.mnc); cellInfoLte->cellIdentityLte.ci = @@ -8033,7 +8034,8 @@ void convertRilCellInfoListToHal(void *response, size_t responseLen, hidl_vec<Ce records[i].tdscdma.resize(1); CellInfoTdscdma *cellInfoTdscdma = &records[i].tdscdma[0]; cellInfoTdscdma->cellIdentityTdscdma.mcc = - std::to_string(rillCellInfo->CellInfo.tdscdma.cellIdentityTdscdma.mcc); + ril::util::mcc::decode( + rillCellInfo->CellInfo.tdscdma.cellIdentityTdscdma.mcc); cellInfoTdscdma->cellIdentityTdscdma.mnc = ril::util::mnc::decode( rillCellInfo->CellInfo.tdscdma.cellIdentityTdscdma.mnc); |