Merge pull request #26 from emgstabilis/25-generation-of-pc-identifier-fail...

Maximilien Siavelis

2019-08-08 c0d61ea38a684301c6937d8d4067765abdc5aaaa

 src/library/pc-identifiers.c

@@ -39,11 +39,11 @@

   if (identifiers == NULL || *num_identifiers == 0) {
      result_adapterInfos = getAdapterInfos(NULL, &adapter_num);
      if (result_adapterInfos != FUNC_RET_OK) {
      if ((result_adapterInfos != FUNC_RET_OK) || (adapter_num == 0)) {
         return generate_disk_pc_id(identifiers, num_identifiers, false);
      }
      result_diskinfos = getDiskInfos(NULL, &disk_num);
      if (result_diskinfos != FUNC_RET_OK) {
      if ((result_diskinfos != FUNC_RET_OK) || (disk_num == 0)) {
         return generate_ethernet_pc_id(identifiers, num_identifiers, true);
      }
      *num_identifiers = disk_num * adapter_num;
@@ -84,7 +84,11 @@
         }
      }
end:
      *num_identifiers = cmin(*num_identifiers, adapter_num * disk_num);
#ifdef _MSC_VER
        *num_identifiers = min(*num_identifiers, adapter_num * disk_num);
#else
        *num_identifiers = cmin(*num_identifiers, adapter_num * disk_num);
#endif
      free(diskInfoPtr);
      free(adapterInfoPtr);
   }
@@ -149,7 +153,7 @@
      return result_diskinfos;
   }
   diskInfos = (DiskInfo*) malloc(disk_num * sizeof(DiskInfo));
   //memset(diskInfos,0,disk_num * sizeof(DiskInfo));
   memset(diskInfos,0,disk_num * sizeof(DiskInfo));
   result_diskinfos = getDiskInfos(diskInfos, &disk_num);
   if (result_diskinfos != FUNC_RET_OK) {
      free(diskInfos);
@@ -175,7 +179,7 @@
      if (use_label) {
         if (diskInfos[i].label[0] != 0) {
            memset(identifiers[j], 0, sizeof(PcIdentifier)); //!!!!!!!
            strncpy(identifiers[j], diskInfos[i].label,
            strncpy((char*)identifiers[j], diskInfos[i].label,
                  sizeof(PcIdentifier));
            j++;
         }
@@ -277,7 +281,7 @@
      PcSignature pc_identifier_out) {
   //TODO base62 encoding, now uses base64
   PcIdentifier concat_identifiers[2];
   char* b64_data;
   char* b64_data = NULL;
   int b64_size = 0;
   size_t concatIdentifiersSize = sizeof(PcIdentifier) * 2;
   //concat_identifiers = (PcIdentifier *) malloc(concatIdentifiersSize);
@@ -285,6 +289,7 @@
   memcpy(&concat_identifiers[1], identifier2, sizeof(PcIdentifier));
   b64_data = base64(concat_identifiers, concatIdentifiersSize, &b64_size);
   if (b64_size > sizeof(PcSignature)) {
      free(b64_data);
      return FUNC_RET_BUFFER_TOO_SMALL;
   }
   sprintf(pc_identifier_out, "%.4s-%.4s-%.4s-%.4s", &b64_data[0],
@@ -341,7 +346,7 @@
      PcIdentifier identifier2_out, PcSignature pc_signature_in) {
   //TODO base62 encoding, now uses base64

   unsigned char * concat_identifiers;
   unsigned char * concat_identifiers = NULL;
   char base64ids[17];
   int identifiers_size;

@@ -349,6 +354,7 @@
         &base64ids[8], &base64ids[12]);
   concat_identifiers = unbase64(base64ids, 16, &identifiers_size);
   if (identifiers_size > sizeof(PcIdentifier) * 2) {
      free(concat_identifiers);
      return FUNC_RET_BUFFER_TOO_SMALL;
   }
   memcpy(identifier1_out, concat_identifiers, sizeof(PcIdentifier));
@@ -382,6 +388,7 @@
   for (i = 0; i < 2; i++) {
      current_strategy_id = strategy_from_pc_id(user_identifiers[i]);
      if (current_strategy_id == STRATEGY_UNKNOWN) {
         free(calculated_identifiers);
         return LICENSE_MALFORMED;
      }
      if (current_strategy_id != previous_strategy_id) {