fix codeql warning issue

hitsz-ids · Aug 31, 2023 · 8f71cd8 · 8f71cd8
1 parent 673d850
commit 8f71cd8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/emu/src/emulator.cc b/emu/src/emulator.cc
@@ -1215,7 +1215,7 @@ int SDF_ExternalVerify_ECC(void *hSessionHandle,
                            ECCSignature *pucSignature) {
     int ret = SDR_OK;
     BN_CTX *ctx = nullptr;
-    BIGNUM *order, *R, *m, *X, *t, *x, *y, *r, *s;
+    BIGNUM *order = nullptr, *R = nullptr, *m = nullptr, *X = nullptr, *t = nullptr, *x = nullptr, *y = nullptr, *r = nullptr, *s = nullptr;
     EC_POINT *point = nullptr;
     EC_GROUP *ECgroup = EC_GROUP_new_by_curve_name(NID_sm2);
     EC_POINT *pub_key = EC_POINT_new(ECgroup);

diff --git a/src/session_factory.cc b/src/session_factory.cc
@@ -459,11 +459,11 @@ std::vector<uint16_t> SessionFactory::updateWeightRoundTable(ConnectorType ct) {
   } else if (ConnectorType::CT_TSS == ct) {
     conns = &ts_connectors_;
   }
-  if (conns != nullptr) {
+  if (conns != nullptr && conns->size() > 0) {
     connector_round.resize(this->weight_sum_[static_cast<int>(ct)]);
     weight_current_array.resize(conns->size());
     for (int n = 0; n < this->weight_sum_[static_cast<int>(ct)]; n++) {
-      size_t maxSub;
+      size_t maxSub = 0;
       for (size_t m = 0; m < conns->size(); m++) {
         weight_current_array[m] += (*conns)[m]->getWeight();
         if (m == 0 || (*conns)[m]->getWeight() > weight_current_array[maxSub]) {