]> git.ipfire.org Git - people/pmueller/ipfire-2.x.git/blobdiff - html/cgi-bin/services.cgi
services.cgi: Fix status/actions on services with name != addon name
[people/pmueller/ipfire-2.x.git] / html / cgi-bin / services.cgi
index 2b7ee75a1a08bd9685cd51c022280318cd8308d5..de946d755b422334ebdcd248d5b046f9a942c748 100644 (file)
@@ -2,7 +2,7 @@
 ###############################################################################
 #                                                                             #
 # IPFire.org - A linux based firewall                                         #
-# Copyright (C) 2005-2010  IPFire Team                                        #
+# Copyright (C) 2005-2021  IPFire Team                                        #
 #                                                                             #
 # This program is free software: you can redistribute it and/or modify        #
 # it under the terms of the GNU General Public License as published by        #
@@ -20,7 +20,7 @@
 ###############################################################################
 
 use strict;
-
+use feature "switch";
 # enable only the following on debugging purpose
 #use warnings;
 #use CGI::Carp 'fatalsToBrowser';
@@ -29,12 +29,13 @@ require '/var/ipfire/general-functions.pl';
 require "${General::swroot}/lang.pl";
 require "${General::swroot}/header.pl";
 require "${General::swroot}/graphs.pl";
+require "/opt/pakfire/lib/functions.pl";
 
 my %color = ();
 my %mainsettings = ();
 my %netsettings=();
 &General::readhash("${General::swroot}/main/settings", \%mainsettings);
-&General::readhash("/srv/web/ipfire/html/themes/".$mainsettings{'THEME'}."/include/colors.txt", \%color);
+&General::readhash("/srv/web/ipfire/html/themes/ipfire/include/colors.txt", \%color);
 &General::readhash("${General::swroot}/ethernet/settings", \%netsettings);
 
 #workaround to suppress a warning when a variable is used only once
@@ -49,13 +50,14 @@ my %servicenames =(
        $Lang::tr{'dhcp server'} => 'dhcpd',
        $Lang::tr{'web server'} => 'httpd',
        $Lang::tr{'cron server'} => 'fcron',
-       $Lang::tr{'dns proxy server'} => 'dnsmasq',
+       $Lang::tr{'dns proxy server'} => 'unbound',
        $Lang::tr{'logging server'} => 'syslogd',
        $Lang::tr{'kernel logging server'} => 'klogd',
        $Lang::tr{'ntp server'} => 'ntpd',
        $Lang::tr{'secure shell server'} => 'sshd',
        $Lang::tr{'vpn'} => 'charon',
        $Lang::tr{'web proxy'} => 'squid',
+       $Lang::tr{'intrusion detection system'} => 'suricata',
        'OpenVPN' => 'openvpn'
 );
 
@@ -63,7 +65,7 @@ my %link =(
        $Lang::tr{'dhcp server'} => "<a href=\'dhcp.cgi\'>$Lang::tr{'dhcp server'}</a>",
        $Lang::tr{'web server'} => $Lang::tr{'web server'},
        $Lang::tr{'cron server'} => $Lang::tr{'cron server'},
-       $Lang::tr{'dns proxy server'} => $Lang::tr{'dns proxy server'},
+       $Lang::tr{'dns proxy server'} => "<a href=\'dns.cgi\'>$Lang::tr{'dns proxy server'}</a>",
        $Lang::tr{'logging server'} => $Lang::tr{'logging server'},
        $Lang::tr{'kernel logging server'} => $Lang::tr{'kernel logging server'},
        $Lang::tr{'ntp server'} => "<a href=\'time.cgi\'>$Lang::tr{'ntp server'}</a>",
@@ -71,30 +73,15 @@ my %link =(
        $Lang::tr{'vpn'} => "<a href=\'vpnmain.cgi\'>$Lang::tr{'vpn'}</a>",
        $Lang::tr{'web proxy'} => "<a href=\'proxy.cgi\'>$Lang::tr{'web proxy'}</a>",
        'OpenVPN' => "<a href=\'ovpnmain.cgi\'>OpenVPN</a>",
-       "$Lang::tr{'intrusion detection system'} (GREEN)" => "<a href=\'ids.cgi\'>$Lang::tr{'intrusion detection system'} (GREEN)</a>",
-       "$Lang::tr{'intrusion detection system'} (RED)" => "<a href=\'ids.cgi\'>$Lang::tr{'intrusion detection system'} (RED)</a>",
-       "$Lang::tr{'intrusion detection system'} (ORANGE)" => "<a href=\'ids.cgi\'>$Lang::tr{'intrusion detection system'} (ORANGE)</a>",
-       "$Lang::tr{'intrusion detection system'} (BLUE)" => "<a href=\'ids.cgi\'>$Lang::tr{'intrusion detection system'} (BLUE)</a>"
+       "$Lang::tr{'intrusion detection system'}" => "<a href=\'ids.cgi\'>$Lang::tr{'intrusion detection system'}</a>",
 );
 
-my $lines=0; # Used to count the outputlines to make different bgcolor
-
-my $iface = '';
-if (open(FILE, "${General::swroot}/red/iface")){
-       $iface = <FILE>;
-       close FILE;
-       chomp $iface;
-}
-
-$servicenames{"$Lang::tr{'intrusion detection system'} (RED)"}   = "snort_${iface}";
-$servicenames{"$Lang::tr{'intrusion detection system'} (GREEN)"} = "snort_$netsettings{'GREEN_DEV'}";
+# Hash to overwrite the process name of a process if it differs fromt the launch command.
+my %overwrite_exename_hash = (
+       "suricata" => "Suricata-Main"
+);
 
-if ($netsettings{'ORANGE_DEV'} ne ''){
-       $servicenames{"$Lang::tr{'intrusion detection system'} (ORANGE)"} = "snort_$netsettings{'ORANGE_DEV'}";
-}
-if ($netsettings{'BLUE_DEV'} ne ''){
-       $servicenames{"$Lang::tr{'intrusion detection system'} (BLUE)"} = "snort_$netsettings{'BLUE_DEV'}";
-}
+my $lines=0; # Used to count the outputlines to make different bgcolor
 
 my @querry = split(/\?/,$ENV{'QUERY_STRING'});
 $querry[0] = '' unless defined $querry[0];
@@ -117,7 +104,7 @@ if ( $querry[0] =~ "processescpu"){
        print <<END
 <div align='center'>
 <table width='80%' cellspacing='1' class='tbl'>
-<tr bgcolor='$color{'color20'}'>
+<tr>
        <th align='left'><b>$Lang::tr{'services'}</b></th>
        <th align='center' ><b>$Lang::tr{'status'}</b></th>
        <th align='center'><b>PID</b></th>
@@ -154,15 +141,22 @@ END
        &Header::openbox('100%', 'left', "Addon - $Lang::tr{services}");
        my $paramstr=$ENV{QUERY_STRING};
        my @param=split(/!/, $paramstr);
-       if ($param[1] ne ''){
-               system("/usr/local/bin/addonctrl @param[0] @param[1] > /dev/null 2>&1");
+       # Make sure action parameter is actually one of the allowed service actions
+       given ($param[1]) {
+               when ( ['start', 'stop', 'enable', 'disable'] ) {
+                       # Make sure pak-name and service name don't contain any illegal character
+                       if ( $param[0] !~ /[^a-zA-Z_0-9\-]/ &&
+                            $param[2] !~ /[^a-zA-Z_0-9\-]/ ) {
+                               &General::system("/usr/local/bin/addonctrl", "$param[0]", "$param[1]", "$param[2]");
+                       }
+               }
        }
 
        print <<END
 <div align='center'>
 <table width='80%' cellspacing='1' class='tbl'>
-<tr bgcolor='$color{'color20'}'>
-       <th align='center'><b>Addon</b></th>
+<tr>
+       <th align='left'><b>Addon $Lang::tr{service}</b></th>
        <th align='center'><b>Boot</b></th>
        <th align='center' colspan=2><b>$Lang::tr{'action'}</b></th>
        <th align='center'><b>$Lang::tr{'status'}</b></th>
@@ -174,36 +168,38 @@ END
 
        my $lines=0; # Used to count the outputlines to make different bgcolor
 
-       # Generate list of installed addon pak's
-       my @pak = `find /opt/pakfire/db/installed/meta-* 2>/dev/null | cut -d"-" -f2`;
-       foreach (@pak){
-               chomp($_);
-
-               # Check which of the paks are services
-               my @svc = `find /etc/init.d/$_ 2>/dev/null | cut -d"/" -f4`;
-               foreach (@svc){
-                       # blacklist some packages
-                       #
-                       # alsa has trouble with the volume saving and was not really stopped
-                       # mdadm should not stopped with webif because this could crash the system
-                       #
-                       chomp($_);
-                       if ( ($_ ne "alsa") && ($_ ne "mdadm") ) {
+       my @paks;
+       my @addon_services;
+
+       # Generate list of installed addon pak services
+       my %paklist = &Pakfire::dblist("installed");
+
+       foreach my $pak (keys %paklist) {
+               my %metadata = &Pakfire::getmetadata($pak, "installed");
+                       
+               my $service;
+
+               if ("$metadata{'Services'}") {
+                       foreach $service (split(/ /, "$metadata{'Services'}")) {
                                $lines++;
-                               if ($lines % 2){
+                               if ($lines % 2) {
                                        print "<tr>";
                                        $col="bgcolor='$color{'color22'}'";
-                               }else{
+                               } else {
                                        print "<tr>";
                                        $col="bgcolor='$color{'color20'}'";
                                }
-                               print "<td align='left' $col>$_</td> ";
-                               my $status = isautorun($_,$col);
+
+                               # Add addon name to displayname of service if servicename differs from addon
+                               my $displayname = ($pak ne $service) ? "$service ($pak)" : $service;
+                               print "<td align='left' $col width='31%'>$displayname</td> ";
+
+                               my $status = isautorun($pak,$service,$col);
                                print "$status ";
-                               print "<td align='center' $col><A HREF=services.cgi?$_!start><img alt='$Lang::tr{'start'}' title='$Lang::tr{'start'}' src='/images/go-up.png' border='0' /></A></td>";
-                               print "<td align='center' $col><A HREF=services.cgi?$_!stop><img alt='$Lang::tr{'stop'}' title='$Lang::tr{'stop'}' src='/images/go-down.png' border='0' /></A></td> ";
-                               my $status = &isrunningaddon($_,$col);
-                               $status =~ s/\\e\[[0-1]\;[0-9]+m//g;
+                               print "<td align='center' $col width='8%'><a href='services.cgi?$pak!start!$service'><img alt='$Lang::tr{'start'}' title='$Lang::tr{'start'}' src='/images/go-up.png' border='0' /></a></td>";
+                               print "<td align='center' $col width='8%'><a href='services.cgi?$pak!stop!$service'><img alt='$Lang::tr{'stop'}' title='$Lang::tr{'stop'}' src='/images/go-down.png' border='0' /></a></td> ";
+                               my $status = isrunningaddon($pak,$service,$col);
+                               $status =~ s/\\e\[[0-1]\;[0-9]+m//g;
 
                                chomp($status);
                                print "$status";
@@ -227,27 +223,27 @@ END
        &Header::closepage();
 }
 
-sub isautorun{
-       my $cmd = $_[0];
-       my $col = $_[1];
-       my $status = "<td align='center' $col></td>";
-       my $init = `find /etc/rc.d/rc3.d/S??${cmd} 2>/dev/null`;
-       chomp ($init);
-       if ($init ne ''){
-               $status = "<td align='center' $col><A HREF=services.cgi?$_!disable><img alt='$Lang::tr{'deactivate'}' title='$Lang::tr{'deactivate'}' src='/images/on.gif' border='0' width='16' height='16' /></A></td>";
-       }
-       $init = `find /etc/rc.d/rc3.d/off/S??${cmd} 2>/dev/null`;
-       chomp ($init);
-       if ($init ne ''){
-               $status = "<td align='center' $col><A HREF=services.cgi?$_!enable><img alt='$Lang::tr{'activate'}' title='$Lang::tr{'activate'}' src='/images/off.gif' border='0' width='16' height='16' /></A></td>";
+sub isautorun (@) {
+       my ($pak, $service, $col) = @_;
+       my @testcmd = &General::system_output("/usr/local/bin/addonctrl", "$pak", "boot-status", "$service");
+       my $testcmd = @testcmd[0];
+       my $status = "<td align='center' $col><img alt='$Lang::tr{'service boot setting unavailable'}' title='$Lang::tr{'service boot setting unavailable'}' src='/images/dialog-warning.png' border='0' width='16' height='16' /></td>";
+
+       # Check if autorun for the given service is enabled.
+       if ( $testcmd =~ /enabled\ on\ boot/ ) {
+               # Adjust status.
+               $status = "<td align='center' $col><a href='services.cgi?$pak!disable!$service'><img alt='$Lang::tr{'deactivate'}' title='$Lang::tr{'deactivate'}' src='/images/on.gif' border='0' width='16' height='16' /></a></td>";
+       } elsif ( $testcmd =~ /disabled\ on\ boot/ ) {
+               # Adjust status.
+               $status = "<td align='center' $col><a href='services.cgi?$pak!enable!$service'><img alt='$Lang::tr{'activate'}' title='$Lang::tr{'activate'}' src='/images/off.gif' border='0' width='16' height='16' /></a></td>";
        }
 
+       # Return the status.
        return $status;
 }
 
-sub isrunning{
-       my $cmd = $_[0];
-       my $col = $_[1];
+sub isrunning (@) {
+       my ($cmd, $col) = @_;
        my $status = "<td align='center' bgcolor='${Header::colourred}'><font color='white'><b>$Lang::tr{'stopped'}</b></font></td><td colspan='2' $col></td>";
        my $pid = '';
        my $testcmd = '';
@@ -255,7 +251,20 @@ sub isrunning{
        my $memory;
 
        $cmd =~ /(^[a-z]+)/;
-       $exename = $1;
+
+       # Check if the exename needs to be overwritten.
+       # This happens if the expected process name string
+       # differs from the real one. This may happened if
+       # a service uses multiple processes or threads.
+       if (exists($overwrite_exename_hash{$1})) {
+               # Grab the string which will be reported by
+               # the process from the corresponding hash.
+               $exename = $overwrite_exename_hash{$1};
+       } else {
+               # Directly expect the launched command as
+               # process name.
+               $exename = $1;
+       }
 
        if (open(FILE, "/var/run/${cmd}.pid")){
                $pid = <FILE>; chomp $pid;
@@ -285,16 +294,17 @@ sub isrunning{
        return $status;
 }
 
-sub isrunningaddon{
-       my $cmd = $_[0];
-       my $col = $_[1];
+sub isrunningaddon (@) {
+       my ($pak, $service, $col) = @_;
+
        my $status = "<td align='center' bgcolor='${Header::colourred}'><font color='white'><b>$Lang::tr{'stopped'}</b></font></td><td colspan='2' $col></td>";
        my $pid = '';
        my $testcmd = '';
        my $exename;
        my @memory;
 
-       my $testcmd = `/usr/local/bin/addonctrl $_ status 2>/dev/null`;
+       my @testcmd = &General::system_output("/usr/local/bin/addonctrl", "$pak", "status", "$service");
+       my $testcmd = @testcmd[0];
 
        if ( $testcmd =~ /is\ running/ && $testcmd !~ /is\ not\ running/){
                $status = "<td align='center' bgcolor='${Header::colourgreen}'><font color='white'><b>$Lang::tr{'running'}</b></font></td>";