Skip to content

Commit

Permalink
Merge pull request #2703 from a3a3el/bugfix/handle-dt_unknown
Browse files Browse the repository at this point in the history
scanner: don't skip directory entries with type `DT_UNKNOWN`
  • Loading branch information
djcb authored May 3, 2024
2 parents 332cf24 + 5a1ca77 commit aaf8fbf
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions lib/mu-scanner.cc
Original file line number Diff line number Diff line change
Expand Up @@ -215,9 +215,13 @@ Scanner::Private::process_dir(const std::string& path, bool is_maildir)
while (running_) {
errno = 0;
if (const auto& dentry{::readdir(dir)}; dentry) {
#if HAVE_DIRENT_D_TYPE /* opttimization: filter out non-dirs early */
#if HAVE_DIRENT_D_TYPE /* optimization: filter out non-dirs early. NB not all file-systems support
* returning the file-type in `d_type`, so don't skip `DT_UNKNOWN`.
*/
if (maildirs_only_mode() &&
dentry->d_type != DT_DIR && dentry->d_type != DT_LNK)
dentry->d_type != DT_DIR &&
dentry->d_type != DT_LNK &&
dentry->d_type != DT_UNKNOWN)
continue;
#endif /*HAVE_DIRENT_D_TYPE*/
dir_entries.emplace_back(dentry);
Expand Down

0 comments on commit aaf8fbf

Please sign in to comment.