Coverity fixes

metsma · metsma · commit 57dbb22cf7fd · 2026-04-29T08:53:35.000+03:00
IB-8899

Signed-off-by: Raul Metsma &lt;raul@metsma.ee&gt;
diff --git a/src/ASiC_S.cpp b/src/ASiC_S.cpp
@@ -96,7 +96,7 @@ ASiC_S::ASiC_S(const string &path)
         THROW("ASiC-S container does not contain any signatures.");
 }
 
-void ASiC_S::addDataFileChecks(const string &fileName, const string &mediaType)
+void ASiC_S::addDataFileChecks(string_view fileName, const string &mediaType)
 {
     ASiContainer::addDataFileChecks(fileName, mediaType);
     if(!dataFiles().empty())
diff --git a/src/ASiC_S.h b/src/ASiC_S.h
@@ -45,7 +45,7 @@ namespace digidoc
         ASiC_S(const std::string &path);
         DISABLE_COPY(ASiC_S);
 
-        void addDataFileChecks(const std::string &path, const std::string &mediaType) override;
+        void addDataFileChecks(std::string_view path, const std::string &mediaType) override;
         void canSave() final;
         void save(const ZipSerialize &s) final;
 
diff --git a/src/ASiContainer.cpp b/src/ASiContainer.cpp
@@ -149,7 +149,7 @@ vector<Signature *> ASiContainer::signatures() const
  */
 void ASiContainer::addDataFile(const string &path, const string &mediaType)
 {
-    string fileName = File::fileName(path);
+    auto fileName = File::fileName(path);
     addDataFileChecks(fileName, mediaType);
     auto nativePath = File::encodeName(path);
     auto size = File::fileSize(nativePath);
@@ -165,8 +165,8 @@ void ASiContainer::addDataFile(const string &path, const string &mediaType)
         *data << is->rdbuf();
         is = std::move(data);
     }
-    d->properties[fileName] = { appInfo(), File::modifiedTime(path), size };
-    d->documents.push_back(new DataFilePrivate(std::move(is), std::move(fileName), mediaType));
+    d->properties[string(fileName)] = { appInfo(), File::modifiedTime(path), size };
+    d->documents.push_back(new DataFilePrivate(std::move(is), string(fileName), mediaType));
 }
 
 void ASiContainer::addDataFile(unique_ptr<istream> is, const string &fileName, const string &mediaType)
@@ -177,14 +177,14 @@ void ASiContainer::addDataFile(unique_ptr<istream> is, const string &fileName, c
     d->documents.push_back(new DataFilePrivate(std::move(is), fileName, mediaType));
 }
 
-void ASiContainer::addDataFileChecks(const string &fileName, const string &mediaType)
+void ASiContainer::addDataFileChecks(string_view fileName, const string &mediaType)
 {
     if(!d->signatures.empty())
         THROW("Can not add document to container which has signatures, remove all signatures before adding new document.");
     if(fileName == "mimetype")
         THROW("mimetype is reserved file.");
     if(any_of(d->documents.cbegin(), d->documents.cend(), [&](DataFile *file) { return fileName == file->fileName(); }))
-        THROW("Document with same file name '%s' already exists.", fileName.c_str());
+        THROW("Document with same file name '%.*s' already exists.", STR_VIEW_FMT(fileName));
     if(mediaType.find('/') == string::npos)
         THROW("MediaType does not meet format requirements (RFC2045, section 5.1) '%s'.", mediaType.c_str());
 }
diff --git a/src/ASiContainer.h b/src/ASiContainer.h
@@ -61,7 +61,7 @@ namespace digidoc
       protected:
           ASiContainer(std::string_view mimetype);
 
-          virtual void addDataFileChecks(const std::string &path, const std::string &mediaType);
+          virtual void addDataFileChecks(std::string_view path, const std::string &mediaType);
           void addDataFilePrivate(DataFile *dataFile);
           Signature* addSignature(std::unique_ptr<Signature> &&signature);
           virtual void canSave() = 0;
diff --git a/src/DataFile.cpp b/src/DataFile.cpp
@@ -114,7 +114,7 @@ DataFilePrivate::DataFilePrivate(const ZipSerialize &z, string filename, string
              (size = r(buf.data(), buf.size())) > 0; currentStreamSize += size)
         {
             if(!fs->write(buf.data(), size))
-                THROW("Failed to write '%s' data to stream. Stream size: %d", m_filename.c_str(), currentStreamSize);
+                THROW("Failed to write '%s' data to stream. Stream size: %zu", m_filename.c_str(), currentStreamSize);
         }
         m_is = std::move(fs);
     }
diff --git a/src/SiVaContainer.cpp b/src/SiVaContainer.cpp
@@ -157,7 +157,7 @@ SiVaContainer::SiVaContainer(const string &path, ContainerOpenCB *cb, bool useHa
     else if(File::fileExtension(path, {"pdf"}))
     {
         d->mediaType = "application/pdf";
-        d->dataFiles.push_back(new DataFilePrivate(std::move(ifs), fileName, "application/pdf"));
+        d->dataFiles.push_back(new DataFilePrivate(std::move(ifs), string(fileName), "application/pdf"));
     }
     else if(File::fileExtension(path, {"asice", "sce", "asics", "scs"}))
     {
diff --git a/src/SignatureTST.cpp b/src/SignatureTST.cpp
@@ -61,7 +61,8 @@ struct SignatureTST::Data {
         if (auto it = cache->find(method); it != cache->cend()) {
             return it->second;
         }
-        return (*cache)[std::move(method)] = digest(Digest(method)).result();
+        auto result = digest(Digest(method)).result();
+        return (*cache)[std::move(method)] = std::move(result);
     }
 };
 
@@ -82,7 +83,8 @@ SignatureTST::SignatureTST(bool manifest, const ZipSerialize &z, ASiC_S *asicSDo
             auto ref = doc/"SigReference";
             string uri = util::File::fromUriPath(ref["URI"]);
             metadata.emplace_back(std::move(file), std::move(mime), xml.str());
-            metadata.emplace_back(std::move(uri), string(ref["MimeType"]), z.read(uri));
+            auto refData = z.read(uri);
+            metadata.emplace_back(std::move(uri), string(ref["MimeType"]), std::move(refData));
             file.clear();
 
             for(auto ref = doc/"DataObjectReference"; ref; ref++)
@@ -172,7 +174,7 @@ void SignatureTST::extendSignatureProfile(Signer *signer)
     }, true);
     auto i = metadata.insert(metadata.cbegin(), {"META-INF/ASiCArchiveManifest.xml", "text/xml", std::move(data)});
     vector<unsigned char> der = TS(i->digest(), signer->userAgent());
-    metadata.insert(next(i), {tstName, TST_MIMETYPE, string{der.cbegin(), der.cend()}});
+    metadata.insert(next(i), {std::move(tstName), TST_MIMETYPE, string{der.cbegin(), der.cend()}});
 }
 
 X509Cert SignatureTST::TimeStampCertificate() const
diff --git a/src/crypto/PKCS11Signer.cpp b/src/crypto/PKCS11Signer.cpp
@@ -99,10 +99,10 @@ static auto PKCS11List(CK_FUNCTION_LIST *f, Args&&... args)
     using T = std::remove_pointer_t<std::tuple_element_t<Traits::nargs - 2, typename Traits::arg_types>>;
     std::vector<T> result;
     CK_ULONG count = 0;
-    if((f->*Getter)(std::forward<Args>(args)..., static_cast<T*>(nullptr), &count) != CKR_OK || count == 0)
+    if((f->*Getter)(args..., static_cast<T*>(nullptr), &count) != CKR_OK || count == 0)
         return result;
     result.resize(size_t(count));
-    if((f->*Getter)(std::forward<Args>(args)..., result.data(), &count) != CKR_OK)
+    if((f->*Getter)(args..., result.data(), &count) != CKR_OK)
         result.clear();
     return result;
 }
diff --git a/src/util/File.cpp b/src/util/File.cpp
@@ -64,7 +64,7 @@ using f_utimbuf = struct utimbuf;
 
 stack<fs::path> File::tempFiles;
 
-static string decodeName(fs::path path)
+static string decodeName(const fs::path &path)
 {
     auto name = path.u8string();
     return {reinterpret_cast<const char*>(name.data()), name.size()};
@@ -149,7 +149,7 @@ bool File::fileExtension(string_view path, initializer_list<string_view> list)
 /**
  * Returns file size
  */
-unsigned long File::fileSize(const std::filesystem::path &path) noexcept
+unsigned long File::fileSize(const fs::path &path) noexcept
 {
     error_code ec;
     auto result = fs::file_size(path, ec);
@@ -162,7 +162,7 @@ unsigned long File::fileSize(const std::filesystem::path &path) noexcept
  * @param path full path of the file.
  * @return returns file name from the file full path in UTF-8.
  */
-string File::fileName(const string& path)
+string_view File::fileName(string_view path) noexcept
 {
     size_t pos = path.find_last_of("/\\");
     return pos == string::npos ? path : path.substr(pos + 1);
@@ -343,8 +343,8 @@ constexpr bool fromHexChar(auto pos, auto end, auto &value)
 {
     if(distance(pos, end) < 2)
         return false;
-    auto *p = &*pos;
-    return from_chars(p, p + 2, value, 16).ec == std::errc{};
+    auto *p = to_address(pos);
+    return from_chars(p, p + 2, value, 16).ec == errc{};
 }
 
 string File::fromUriPath(string_view path)
diff --git a/src/util/File.h b/src/util/File.h
@@ -42,7 +42,7 @@ namespace digidoc
               static bool fileExists(const std::string& path);
               static bool fileExtension(std::string_view path, std::initializer_list<std::string_view> list);
               static unsigned long fileSize(const std::filesystem::path &path) noexcept;
-              static std::string fileName(const std::string& path);
+              static std::string_view fileName(std::string_view path) noexcept;
               static std::string directory(const std::string& path);
               static std::string path(std::string dir, std::string_view relativePath);
               static std::filesystem::path tempFileName();
diff --git a/src/util/log.cpp b/src/util/log.cpp
@@ -69,39 +69,65 @@ string Log::formatArgList(const char* fmt, va_list args)
     return result;
 }
 
+static ostream &logStream(Conf *conf) noexcept
+{
+    static fstream f;
+    static string lastPath;
+    try {
+        const auto &path = conf->logFile();
+        if(path != lastPath)
+        {
+            lastPath = path;
+            f.close();
+            if(!path.empty())
+                f.open(File::encodeName(path), fstream::out|fstream::app);
+        }
+    }
+    catch(const exception &e)
+    {
+        f.close();
+        cerr << "Failed to open log file '" << lastPath << "': " << e.what() << '\n';
+    }
+    return f.is_open() ? static_cast<ostream &>(f) : cout;
+}
+
 void Log::out(LogType type, const char *file, unsigned int line, const char *format, ...)
 {
     Conf *conf = Conf::instance();
     if(!conf || conf->logLevel() < type)
         return;
 
-    ostream *o = &cout;
-    fstream f;
-    if(!conf->logFile().empty())
-    {
-        f.open(File::encodeName(conf->logFile()), fstream::out|fstream::app);
-        o = &f;
-    }
+    ostream &o = logStream(conf);
     time_t t = time(nullptr);
     tm tm {};
 #ifdef _WIN32
     gmtime_s(&tm, &t);
 #else
     gmtime_r(&t, &tm);
 #endif
-    *o << put_time(&tm, "%Y-%m-%dT%TZ") << ' ';
+    o << put_time(&tm, "%Y-%m-%dT%TZ") << ' ';
     switch(type)
     {
-    case ErrorType: *o << 'E'; break;
-    case WarnType: *o << 'W'; break;
-    case InfoType: *o << 'I'; break;
-    case DebugType: *o << 'D'; break;
+    case ErrorType: o << 'E'; break;
+    case WarnType: o << 'W'; break;
+    case InfoType: o << 'I'; break;
+    case DebugType: o << 'D'; break;
     }
-    *o << " [" << File::fileName(file) << ':' << line << "] - ";
+    o << " [" << File::fileName(file) << ':' << line << "] - ";
 
     va_list args{};
     va_start(args, format);
-    *o << formatArgList(format, args) << '\n';
+    try {
+        if(string_view(format).contains('%'))
+            o << formatArgList(format, args) << '\n';
+        else
+            o << format << '\n';
+    }
+    catch(const exception &e)
+    {
+        cerr << "Failed to format log message: " << e.what() << '\n';
+        o << format << '\n';
+    }
     va_end(args);
 }
 
@@ -111,17 +137,10 @@ void Log::dbgPrintfMemImpl(const char *msg, const unsigned char *data, size_t si
     if(!conf || conf->logLevel() < DebugType)
         return;
 
-    ostream *o = &cout;
-    fstream f;
-    if(!conf->logFile().empty())
-    {
-        f.open(File::encodeName(conf->logFile()), fstream::out|fstream::app);
-        o = &f;
-    }
-
-    *o << "DEBUG [" << File::fileName(file) << ':' << line << "] - " << msg << " { ";
-    *o << hex << uppercase << setfill('0');
+    ostream &o = logStream(conf);
+    o << "DEBUG [" << File::fileName(file) << ':' << line << "] - " << msg << " { ";
+    o << hex << uppercase << setfill('0');
     for(size_t i = 0; i < size; ++i)
-        *o << setw(2) << static_cast<int>(data[i]) << ' ';
-    *o << dec << nouppercase << setfill(' ') << "}:" << size << '\n';
+        o << setw(2) << static_cast<int>(data[i]) << ' ';
+    o << dec << nouppercase << setfill(' ') << "}:" << size << '\n';
 }

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@ ASiC_S::ASiC_S(const string &path)`
`96`	`96`	`THROW("ASiC-S container does not contain any signatures.");`
`97`	`97`	`}`
`98`	`98`
`99`		`-void ASiC_S::addDataFileChecks(const string &fileName, const string &mediaType)`
	`99`	`+void ASiC_S::addDataFileChecks(string_view fileName, const string &mediaType)`
`100`	`100`	`{`
`101`	`101`	`ASiContainer::addDataFileChecks(fileName, mediaType);`
`102`	`102`	`if(!dataFiles().empty())`
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ vector<Signature *> ASiContainer::signatures() const`
`149`	`149`	`*/`
`150`	`150`	`void ASiContainer::addDataFile(const string &path, const string &mediaType)`
`151`	`151`	`{`
`152`		`- string fileName = File::fileName(path);`
	`152`	`+ auto fileName = File::fileName(path);`
`153`	`153`	`addDataFileChecks(fileName, mediaType);`
`154`	`154`	`auto nativePath = File::encodeName(path);`
`155`	`155`	`auto size = File::fileSize(nativePath);`
`@@ -165,8 +165,8 @@ void ASiContainer::addDataFile(const string &path, const string &mediaType)`
`165`	`165`	`*data << is->rdbuf();`
`166`	`166`	`is = std::move(data);`
`167`	`167`	`}`
`168`		`- d->properties[fileName] = { appInfo(), File::modifiedTime(path), size };`
`169`		`- d->documents.push_back(new DataFilePrivate(std::move(is), std::move(fileName), mediaType));`
	`168`	`+ d->properties[string(fileName)] = { appInfo(), File::modifiedTime(path), size };`
	`169`	`+ d->documents.push_back(new DataFilePrivate(std::move(is), string(fileName), mediaType));`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`void ASiContainer::addDataFile(unique_ptr<istream> is, const string &fileName, const string &mediaType)`
`@@ -177,14 +177,14 @@ void ASiContainer::addDataFile(unique_ptr<istream> is, const string &fileName, c`
`177`	`177`	`d->documents.push_back(new DataFilePrivate(std::move(is), fileName, mediaType));`
`178`	`178`	`}`
`179`	`179`
`180`		`-void ASiContainer::addDataFileChecks(const string &fileName, const string &mediaType)`
	`180`	`+void ASiContainer::addDataFileChecks(string_view fileName, const string &mediaType)`
`181`	`181`	`{`
`182`	`182`	`if(!d->signatures.empty())`
`183`	`183`	`THROW("Can not add document to container which has signatures, remove all signatures before adding new document.");`
`184`	`184`	`if(fileName == "mimetype")`
`185`	`185`	`THROW("mimetype is reserved file.");`
`186`	`186`	`if(any_of(d->documents.cbegin(), d->documents.cend(), [&](DataFile *file) { return fileName == file->fileName(); }))`
`187`		`- THROW("Document with same file name '%s' already exists.", fileName.c_str());`
	`187`	`+ THROW("Document with same file name '%.*s' already exists.", STR_VIEW_FMT(fileName));`
`188`	`188`	`if(mediaType.find('/') == string::npos)`
`189`	`189`	`THROW("MediaType does not meet format requirements (RFC2045, section 5.1) '%s'.", mediaType.c_str());`
`190`	`190`	`}`
Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ DataFilePrivate::DataFilePrivate(const ZipSerialize &z, string filename, string`
`114`	`114`	`(size = r(buf.data(), buf.size())) > 0; currentStreamSize += size)`
`115`	`115`	`{`
`116`	`116`	`if(!fs->write(buf.data(), size))`
`117`		`- THROW("Failed to write '%s' data to stream. Stream size: %d", m_filename.c_str(), currentStreamSize);`
	`117`	`+ THROW("Failed to write '%s' data to stream. Stream size: %zu", m_filename.c_str(), currentStreamSize);`
`118`	`118`	`}`
`119`	`119`	`m_is = std::move(fs);`
`120`	`120`	`}`
Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,7 @@ SiVaContainer::SiVaContainer(const string &path, ContainerOpenCB *cb, bool useHa`
`157`	`157`	`else if(File::fileExtension(path, {"pdf"}))`
`158`	`158`	`{`
`159`	`159`	`d->mediaType = "application/pdf";`
`160`		`- d->dataFiles.push_back(new DataFilePrivate(std::move(ifs), fileName, "application/pdf"));`
	`160`	`+ d->dataFiles.push_back(new DataFilePrivate(std::move(ifs), string(fileName), "application/pdf"));`
`161`	`161`	`}`
`162`	`162`	`else if(File::fileExtension(path, {"asice", "sce", "asics", "scs"}))`
`163`	`163`	`{`