mirror of
https://github.com/clearlinux/systemd-stable.git
synced 2026-10-16 02:30:04 +00:00
Revert "journal: do not check for number of files"
This reverts commit b914ea8d37.
We really need to put a limit on all our resources, everywhere, and in
particular if we operate on external data.
Hence, let's reintroduce the limit, but bump it substantially, so that
it is guaranteed to be higher than any realistic RLIMIT_NOFILE setting.
This commit is contained in:
@@ -43,6 +43,8 @@
|
||||
#include "replace-var.h"
|
||||
#include "fileio.h"
|
||||
|
||||
#define JOURNAL_FILES_MAX 7168
|
||||
|
||||
#define JOURNAL_FILES_RECHECK_USEC (2 * USEC_PER_SEC)
|
||||
|
||||
#define REPLACE_VAR_MAX 256
|
||||
@@ -1196,6 +1198,11 @@ static int add_any_file(sd_journal *j, const char *path) {
|
||||
if (ordered_hashmap_get(j->files, path))
|
||||
return 0;
|
||||
|
||||
if (ordered_hashmap_size(j->files) >= JOURNAL_FILES_MAX) {
|
||||
log_warning("Too many open journal files, not adding %s.", path);
|
||||
return set_put_error(j, -ETOOMANYREFS);
|
||||
}
|
||||
|
||||
r = journal_file_open(path, O_RDONLY, 0, false, false, NULL, j->mmap, NULL, &f);
|
||||
if (r < 0)
|
||||
return r;
|
||||
|
||||
Reference in New Issue
Block a user