From a7f5bff2bbdd902c5ed8e08584922884aef14037 Mon Sep 17 00:00:00 2001 From: Jordi Sanfeliu Date: Tue, 11 Mar 2014 11:18:58 +0100 Subject: [PATCH] make sure to load only .conf files --- monitorix | 4 +++- monitorix.cgi | 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/monitorix b/monitorix index 4667a7f..ddc2d80 100755 --- a/monitorix +++ b/monitorix @@ -446,13 +446,15 @@ if(opendir(DIR, $config{include_dir})) { my @files = grep { !/^[.]/ } readdir(DIR); close(DIR); foreach my $c (sort @files) { + next unless -f $config{include_dir} . "/$c"; + next unless $c =~ m/\.conf$/; my $conf_inc = new Config::General( -ConfigFile => $config{include_dir} . "/$c", ); my %config_inc = $conf_inc->getall; my $g = $config_inc{graph_name}; if(!$g) { - logger("ERROR: graph name not defined."); + logger("ERROR: graph name not defined in '$c'."); next; } if(grep {trim($_) eq $g} (split(',', $config{graph_name}))) { diff --git a/monitorix.cgi b/monitorix.cgi index 8d7e370..2389dd8 100755 --- a/monitorix.cgi +++ b/monitorix.cgi @@ -221,6 +221,8 @@ if(opendir(DIR, $config{include_dir})) { my @files = grep { !/^[.]/ } readdir(DIR); close(DIR); foreach my $c (sort @files) { + next unless -f $config{include_dir} . "/$c"; + next unless $c =~ m/\.conf$/; my $conf_inc = new Config::General( -ConfigFile => $config{include_dir} . "/$c", );