diff --git a/cfg.mk b/cfg.mk index e93114c83e..0475bd953d 100644 --- a/cfg.mk +++ b/cfg.mk @@ -421,9 +421,9 @@ sc_prohibit_gethostname: $(_sc_search_regexp) sc_prohibit_readdir: - @prohibit='\b(read|close)dir *\(' \ + @prohibit='\b(read|close|open)dir *\(' \ exclude='exempt from syntax-check' \ - halt='use virDirRead and VIR_DIR_CLOSE' \ + halt='use virDirOpen, virDirRead and VIR_DIR_CLOSE' \ $(_sc_search_regexp) sc_prohibit_gettext_noop: @@ -1265,3 +1265,6 @@ exclude_file_name_regexp--sc_prohibit_dt_without_code = \ exclude_file_name_regexp--sc_prohibit_always-defined_macros = \ ^tests/virtestmock.c$$ + +exclude_file_name_regexp--sc_prohibit_readdir = \ + ^tests/.*mock\.c$$ diff --git a/src/util/virfile.c b/src/util/virfile.c index 84df1824d8..a45279abf8 100644 --- a/src/util/virfile.c +++ b/src/util/virfile.c @@ -2725,7 +2725,7 @@ virFileRemove(const char *path, static int virDirOpenInternal(DIR **dirp, const char *name, bool ignoreENOENT, bool quiet) { - *dirp = opendir(name); + *dirp = opendir(name); /* exempt from syntax-check */ if (!*dirp) { if (quiet) return -1;