From 61dc2ca65959f92bec3a646c3fb81c23aaf4947f Mon Sep 17 00:00:00 2001 From: open-license-manager <rillf@maildrop.cc> Date: 周一, 13 10月 2014 06:35:23 +0800 Subject: [PATCH] resolved warnings --- src/library/LicenseReader.cpp | 57 ++++++++++++++++++++++++++++++++------------------------- 1 files changed, 32 insertions(+), 25 deletions(-) diff --git a/src/library/LicenseReader.cpp b/src/library/LicenseReader.cpp index 3f49c19..e9abce6 100644 --- a/src/library/LicenseReader.cpp +++ b/src/library/LicenseReader.cpp @@ -21,10 +21,9 @@ #include "pc-identifiers.h" #include "LicenseReader.h" #include "base/StringUtils.h" -#include "base/public-key.h" +#include "base/logger.h" +#include "public-key.h" #include <build_properties.h> - -#include "os/os-cpp.h" namespace license { @@ -51,8 +50,9 @@ EventRegistry FullLicenseInfo::validate(int sw_version) { EventRegistry er; os_initialize(); - bool sigVerified = OsFunctions::verifySignature(printForSign().c_str(), + FUNCTION_RETURN sigVer = verifySignature(printForSign().c_str(), license_signature.c_str()); + bool sigVerified = sigVer == FUNC_RET_OK; if (sigVerified) { er.addEvent(LICENSE_VERIFIED, SVRT_INFO); } else { @@ -152,15 +152,16 @@ string client_signature = trim_copy( ini.GetValue(productNamePtr, "client_signature", "")); /*client_signature.erase( - std::remove(client_signature.begin(), client_signature.end(), '-'), - client_signature.end());*/ + std::remove(client_signature.begin(), client_signature.end(), '-'), + client_signature.end());*/ int from_sw_version = ini.GetLongValue(productNamePtr, - "from_sw_version", FullLicenseInfo::UNUSED_SOFTWARE_VERSION); + "from_sw_version", + FullLicenseInfo::UNUSED_SOFTWARE_VERSION); int to_sw_version = ini.GetLongValue(productNamePtr, "to_sw_version", FullLicenseInfo::UNUSED_SOFTWARE_VERSION); FullLicenseInfo licInfo(*it, product, license_signature, - (int) license_version, from_date, to_date, - client_signature,from_sw_version,to_sw_version); + (int) license_version, from_date, to_date, client_signature, + from_sw_version, to_sw_version); licenseInfoOut.push_back(licInfo); atLeastOneLicenseComplete = true; } else { @@ -199,8 +200,7 @@ } } } else { - eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, SVRT_WARN, - varName); + eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, SVRT_WARN, varName); } } return licenseFoundWithExplicitLocation; @@ -225,19 +225,19 @@ for (auto it = existing_pos.begin(); it != existing_pos.end(); ++it) { diskFiles.push_back(*it); - eventRegistry.addEvent(LICENSE_FILE_FOUND, SVRT_INFO, *it); + eventRegistry.addEvent(LICENSE_FILE_FOUND, SVRT_INFO, + *it); } } else { - eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, - SVRT_WARN, env_var_value); + eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, SVRT_WARN, + env_var_value); } } else { eventRegistry.addEvent(ENVIRONMENT_VARIABLE_NOT_DEFINED, SVRT_WARN); } } else { - eventRegistry.addEvent(ENVIRONMENT_VARIABLE_NOT_DEFINED, - SVRT_WARN); + eventRegistry.addEvent(ENVIRONMENT_VARIABLE_NOT_DEFINED, SVRT_WARN); } } return licenseFileFoundWithEnvVariable; @@ -249,17 +249,24 @@ diskFiles, eventRegistry); bool foundNearModule = false; if (licenseLocation.openFileNearModule) { - string temptativeLicense = OsFunctions::getModuleName() + ".lic"; - ifstream f(temptativeLicense.c_str()); - if (f.good()) { - foundNearModule = true; - diskFiles.push_back(temptativeLicense); - eventRegistry.addEvent(LICENSE_FILE_FOUND, SVRT_INFO, temptativeLicense); + char fname[MAX_PATH] = { 0 }; + FUNCTION_RETURN fret = getModuleName(fname); + if (fret == FUNC_RET_OK) { + string temptativeLicense = string(fname) + ".lic"; + ifstream f(temptativeLicense.c_str()); + if (f.good()) { + foundNearModule = true; + diskFiles.push_back(temptativeLicense); + eventRegistry.addEvent(LICENSE_FILE_FOUND, SVRT_INFO, + temptativeLicense); + } else { + eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, SVRT_WARN, + temptativeLicense); + } + f.close(); } else { - eventRegistry.addEvent(LICENSE_FILE_NOT_FOUND, SVRT_WARN, - temptativeLicense); + LOG_WARN("Error determining module name."); } - f.close(); } bool licenseFileFoundWithEnvVariable = findFileWithEnvironmentVariable( diskFiles, eventRegistry); -- Gitblit v1.9.1