kio-extras-24.05.0-1.fc41

List of Defects

Error: COMPILER_WARNING (CWE-563): [#def1]
kio-extras-24.05.0-build/kio-extras-24.05.0/activities/common/database/Database.cpp: scope_hint: In member function ‘QSqlQuery Common::Database::execQuery(const QString&, bool) const’
kio-extras-24.05.0-build/kio-extras-24.05.0/activities/common/database/Database.cpp:224:58: warning[-Wunused-parameter]: unused parameter ‘ignoreErrors’
#  224 | QSqlQuery Database::execQuery(const QString &query, bool ignoreErrors) const
#      |                                                     ~~~~~^~~~~~~~~~~~
#  222|   }
#  223|   
#  224|-> QSqlQuery Database::execQuery(const QString &query, bool ignoreErrors) const
#  225|   {
#  226|   #ifdef QT_NO_DEBUG

Error: COMPILER_WARNING (CWE-477): [#def2]
kio-extras-24.05.0-build/kio-extras-24.05.0/activities/fileitemplugin/FileItemLinkingPluginActionLoader.cpp: scope_hint: In member function ‘virtual void FileItemLinkingPluginActionLoader::run()’
kio-extras-24.05.0-build/kio-extras-24.05.0/activities/fileitemplugin/FileItemLinkingPluginActionLoader.cpp:84:30: warning[-Wdeprecated-declarations]: ‘void QSqlField::setType(QVariant::Type)’ is deprecated: Use setMetaType() instead
#   84 |                 field.setType(QVariant::String);
#      |                 ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
/usr/include/qt6/QtSql/QSqlField:1: included_from: Included from here.
kio-extras-24.05.0-build/kio-extras-24.05.0/activities/fileitemplugin/FileItemLinkingPluginActionLoader.cpp:21: included_from: Included from here.
/usr/include/qt6/QtSql/qsqlfield.h:62:10: note: declared here
#   62 |     void setType(QVariant::Type type) { setMetaType(QMetaType(int(type))); }
#      |          ^~~~~~~
#   82|                   QStringList escapedFiles;
#   83|                   QSqlField field;
#   84|->                 field.setType(QVariant::String);
#   85|   
#   86|                   const auto urlList = items.urlList();

Error: CLANG_WARNING: [#def3]
kio-extras-24.05.0-build/kio-extras-24.05.0/fish/fish.cpp:541:5: warning[deadcode.DeadStores]: Value stored to 'rc' is never read
#  539|       }
#  540|       ::close(fd[1]);
#  541|->     rc = fcntl(fd[0], F_GETFL, &flags);
#  542|       rc = fcntl(fd[0], F_SETFL, flags | O_NONBLOCK);
#  543|       childFd = fd[0];

Error: CLANG_WARNING: [#def4]
kio-extras-24.05.0-build/kio-extras-24.05.0/fish/fish.cpp:542:5: warning[deadcode.DeadStores]: Value stored to 'rc' is never read
#  540|       ::close(fd[1]);
#  541|       rc = fcntl(fd[0], F_GETFL, &flags);
#  542|->     rc = fcntl(fd[0], F_SETFL, flags | O_NONBLOCK);
#  543|       childFd = fd[0];
#  544|   

Error: COMPILER_WARNING: [#def5]
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp: scope_hint: In function ‘char* scan_escape_direct(char*, QByteArray&)’
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:1873:16: warning[-Wimplicit-fallthrough=]: this statement may fall through
# 1873 |         curpos = 0;
#      |         ~~~~~~~^~~
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:1874:5: note: here
# 1874 |     case 'b':
#      |     ^~~~
# 1871|       case 'l':
# 1872|           cstr = "<HR>";
# 1873|->         curpos = 0;
# 1874|       case 'b':
# 1875|       case 'v':

Error: COMPILER_WARNING: [#def6]
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp: scope_hint: In function ‘char* scan_table(char*)’
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:2483:21: warning[-Wimplicit-fallthrough=]: this statement may fall through
# 2483 |                     if (prev->has(ti1))
#      |                     ^~
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:2485:17: note: here
# 2485 |                 default:
#      |                 ^~~~~~~
# 2481|                       break;
# 2482|                   case '^':
# 2483|->                     if (prev->has(ti1))
# 2484|                           prev->at(ti1).rowspan++;
# 2485|                   default:

Error: COMPILER_WARNING: [#def7]
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:2525:48: warning[-Wimplicit-fallthrough=]: this statement may fall through
# 2525 |                     currow->at(curfield).space += 4;
#      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~
kio-extras-24.05.0-build/kio-extras-24.05.0/man/man2html.cpp:2526:17: note: here
# 2526 |                 case 'R':
#      |                 ^~~~
# 2523|                   switch (currow->at(curfield).align) {
# 2524|                   case 'N':
# 2525|->                     currow->at(curfield).space += 4;
# 2526|                   case 'R':
# 2527|                       out_html(" ALIGN=right");

Error: COMPILER_WARNING (CWE-563): [#def8]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp: scope_hint: In member function ‘virtual KIO::WorkerResult MTPWorker::chmod(const QUrl&, int)’
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp:840:48: warning[-Wunused-parameter]: unused parameter ‘url’
#  840 | KIO::WorkerResult MTPWorker::chmod(const QUrl &url, int permissions)
#      |                                    ~~~~~~~~~~~~^~~
#  838|   }
#  839|   
#  840|-> KIO::WorkerResult MTPWorker::chmod(const QUrl &url, int permissions)
#  841|   {
#  842|       return WorkerResult::pass();

Error: COMPILER_WARNING (CWE-563): [#def9]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp:840:57: warning[-Wunused-parameter]: unused parameter ‘permissions’
#  840 | KIO::WorkerResult MTPWorker::chmod(const QUrl &url, int permissions)
#      |                                                     ~~~~^~~~~~~~~~~
#  838|   }
#  839|   
#  840|-> KIO::WorkerResult MTPWorker::chmod(const QUrl &url, int permissions)
#  841|   {
#  842|       return WorkerResult::pass();

Error: COMPILER_WARNING (CWE-563): [#def10]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp: scope_hint: In member function ‘virtual KIO::WorkerResult MTPWorker::chown(const QUrl&, const QString&, const QString&)’
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp:845:48: warning[-Wunused-parameter]: unused parameter ‘url’
#  845 | KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#      |                                    ~~~~~~~~~~~~^~~
#  843|   }
#  844|   
#  845|-> KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#  846|   {
#  847|       return WorkerResult::pass();

Error: COMPILER_WARNING (CWE-563): [#def11]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp:845:68: warning[-Wunused-parameter]: unused parameter ‘owner’
#  845 | KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#      |                                                     ~~~~~~~~~~~~~~~^~~~~
#  843|   }
#  844|   
#  845|-> KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#  846|   {
#  847|       return WorkerResult::pass();

Error: COMPILER_WARNING (CWE-563): [#def12]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kio_mtp.cpp:845:90: warning[-Wunused-parameter]: unused parameter ‘group’
#  845 | KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#      |                                                                           ~~~~~~~~~~~~~~~^~~~~
#  843|   }
#  844|   
#  845|-> KIO::WorkerResult MTPWorker::chown(const QUrl &url, const QString &owner, const QString &group)
#  846|   {
#  847|       return WorkerResult::pass();

Error: COMPILER_WARNING (CWE-563): [#def13]
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kiod_module/mtpstorage.cpp: scope_hint: In member function ‘QDBusObjectPath MTPStorage::getFilesAndFolders2(const QString&)’
kio-extras-24.05.0-build/kio-extras-24.05.0/mtp/kiod_module/mtpstorage.cpp:564:64: warning[-Wunused-parameter]: unused parameter ‘path’
#  564 | QDBusObjectPath MTPStorage::getFilesAndFolders2(const QString &path)
#      |                                                 ~~~~~~~~~~~~~~~^~~~
#  562|   }
#  563|   
#  564|-> QDBusObjectPath MTPStorage::getFilesAndFolders2(const QString &path)
#  565|   {
#  566|   #if defined(HAVE_LIBMTP_Get_Children)

Error: COMPILER_WARNING (CWE-563): [#def14]
kio-extras-24.05.0-build/kio-extras-24.05.0/recentlyused/recentlyused.cpp: scope_hint: In function ‘KActivities::Stats::ResultModel* runQuery(const QUrl&, int)’
kio-extras-24.05.0-build/kio-extras-24.05.0/recentlyused/recentlyused.cpp:83:44: warning[-Wunused-parameter]: unused parameter ‘limit’
#   83 | ResultModel *runQuery(const QUrl &url, int limit)
#      |                                        ~~~~^~~~~
#   81|   }
#   82|   
#   83|-> ResultModel *runQuery(const QUrl &url, int limit)
#   84|   {
#   85|       qCDebug(KIO_RECENTLYUSED_LOG) << "runQuery for url" << url.toString();

Error: CLANG_WARNING: [#def15]
kio-extras-24.05.0-build/kio-extras-24.05.0/recentlyused/recentlyused.cpp:257:9: warning[cplusplus.NewDeleteLeaks]: Potential leak of memory pointed to by 'model'
#  255|           }
#  256|   
#  257|->         listEntries(udslist);
#  258|   
#  259|           return KIO::WorkerResult::pass();

Error: COMPILER_WARNING (CWE-195): [#def16]
kio-extras-24.05.0-build/kio-extras-24.05.0/sftp/kio_sftp.cpp: scope_hint: In member function ‘QCoro::Generator<SFTPWorker::ReadResponse> SFTPWorker::asyncRead(sftp_file, size_t)’
kio-extras-24.05.0-build/kio-extras-24.05.0/sftp/kio_sftp.cpp:2402:23: warning[-Wsign-compare]: comparison of integer expressions of different signedness: ‘const long unsigned int’ and ‘int’
# 2402 |         if (bytesread == -1) {
#      |             ~~~~~~~~~~^~~~~
# 2400|           const auto bytesread = request.readChunks(filedata);
# 2401|           // Read pending get requests
# 2402|->         if (bytesread == -1) {
# 2403|               co_yield SFTPWorker::ReadResponse(KIO::ERR_CANNOT_READ);
# 2404|               break;

Error: COMPILER_WARNING (CWE-1023): [#def17]
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kded/watcher.cpp: scope_hint: In member function ‘bool Watcher::isInterestingUrl(const QString&)’
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kded/watcher.cpp:147:16: warning[-Wswitch]: enumeration value ‘SMBURLTYPE_PRINTER’ not handled in switch
#  147 |         switch (url.getType()) {
#      |                ^
#  145|       {
#  146|           SMBUrl url{QUrl(str)};
#  147|->         switch (url.getType()) {
#  148|           case SMBURLTYPE_UNKNOWN:
#  149|           case SMBURLTYPE_ENTIRE_NETWORK:

Error: CLANG_WARNING: [#def18]
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kio_smb_dir.cpp:228:5: warning[deadcode.DeadStores]: Value stored to 'errNum' is never read
#  226|           return WorkerResult::fail(KIO::ERR_DOES_NOT_EXIST, src.toDisplayString());
#  227|       }
#  228|->     errNum = 0;
#  229|       if (resume.resuming) {
#  230|           qCDebug(KIO_SMB_LOG) << "seeking to size" << resume.destinationOffset;

Error: CLANG_WARNING: [#def19]
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kio_smb_file.cpp:325:13: warning[deadcode.DeadStores]: Value stored to 'errNum' is never read
#  323|           filefd = smbc_open(m_current_url.toSmbcUrl(), O_RDWR, 0);
#  324|           if (filefd < 0) {
#  325|->             errNum = errno;
#  326|           } else {
#  327|               errNum = 0;

Error: CLANG_WARNING: [#def20]
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kio_smb_file.cpp:327:13: warning[deadcode.DeadStores]: Value stored to 'errNum' is never read
#  325|               errNum = errno;
#  326|           } else {
#  327|->             errNum = 0;
#  328|           }
#  329|   

Error: CLANG_WARNING: [#def21]
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/kio_smb_dir.cpp:21: included_from: Included from here.
kio-extras-24.05.0-build/kio-extras-24.05.0/smb/transfer_resume.h:17:8: warning[core.uninitialized.Assign]: Value assigned to field 'resuming' in implicit constructor is garbage or undefined
#   15|   
#   16|   // Carries the context of a file transfer.
#   17|-> struct TransferContext {
#   18|       // When resuming a file. This is false when starting a new .part!
#   19|       // To establish if a partial file is used the completeDestination should be compared with the partDestination.

Error: CLANG_WARNING: [#def22]
kio-extras-24.05.0-build/kio-extras-24.05.0/thumbnail/imagefilter.cpp:112:21: warning[core.uninitialized.Assign]: Assigned value is garbage or undefined
#  110|               stackpix = &stack[stackindex];
#  111|   
#  112|->             sum_out += *stackpix;
#  113|               sum_in -= *stackpix;
#  114|           } // for (x = 0, ...)

Error: CLANG_WARNING: [#def23]
kio-extras-24.05.0-build/kio-extras-24.05.0/thumbnail/imagefilter.cpp:184:21: warning[core.uninitialized.Assign]: Assigned value is garbage or undefined
#  182|               stackpix = &stack[stackindex];
#  183|   
#  184|->             sum_out += *stackpix;
#  185|               sum_in -= *stackpix;
#  186|           } // for (y = 0, ...)

Error: CLANG_WARNING: [#def24]
kio-extras-24.05.0-build/kio-extras-24.05.0/thumbnail/thumbnail.cpp:511:17: warning[deadcode.DeadStores]: Value stored to 'skipValidItems' is never read
#  509|               ++iterations;
#  510|               if (iterations > 500) {
#  511|->                 skipValidItems = skipped = 0;
#  512|                   break;
#  513|               }

Scan Properties