diff --git a/alerts.php b/alerts.php
index 6886d9ccfc..999ae76737 100755
--- a/alerts.php
+++ b/alerts.php
@@ -26,7 +26,7 @@ foreach (dbFetchRows("SELECT *, A.id AS id FROM `alerts` AS A, `devices` AS D WH
   $msg = $alert['message'];
   $alert_text .= "$date $host $msg";
 
-  dbUpdate(array('alerted' => '1'), 'alerts', '`id` = ?' array($id))
+  dbUpdate(array('alerted' => '1'), 'alerts', '`id` = ?', array($id));
 
 }
 
diff --git a/contrib/generate-iplist.php b/contrib/generate-iplist.php
index a4b324fc3b..fd5e10048a 100755
--- a/contrib/generate-iplist.php
+++ b/contrib/generate-iplist.php
@@ -20,7 +20,7 @@ include("includes/functions.php");
 
 $handle = fopen("ips.txt", "w");
 
-foreach (dbFetchRows("SELECT * FROM `ipv4_networks`"))
+foreach (dbFetchRows("SELECT * FROM `ipv4_networks`") as $data)
 {
   $cidr = $data['ipv4_network'];
   list ($network, $bits) = explode("/", $cidr);
diff --git a/html/includes/graphs/device/sla.inc.php b/html/includes/graphs/device/sla.inc.php
index 161582a5ac..4f97345482 100644
--- a/html/includes/graphs/device/sla.inc.php
+++ b/html/includes/graphs/device/sla.inc.php
@@ -28,7 +28,7 @@ $rrd_options .= " DEF:rtt=$rrd_file:rtt:AVERAGE ";
 $rrd_options .= " VDEF:avg=rtt,AVERAGE ";
 $rrd_options .= " LINE1:avg#CCCCFF:'".str_pad('Average',$descr_len-3)."':dashes";
 $rrd_options .= " GPRINT:rtt:AVERAGE:%4.1lf".$unit."\\\l ";
-$rrd_options .= " LINE1:rtt#CC0000:'" . rrdtool_escape($descr,$descr_len-3)) . "'";
+$rrd_options .= " LINE1:rtt#CC0000:'" . rrdtool_escape($descr,$descr_len-3) . "'";
 $rrd_options .= " GPRINT:rtt:LAST:%4.1lf".$unit." ";
 $rrd_options .= " GPRINT:rtt:MIN:%4.1lf".$unit." ";
 $rrd_options .= " GPRINT:rtt:MAX:%4.1lf".$unit."\\\l ";
diff --git a/html/pages/bill/pdf_history.inc.php b/html/pages/bill/pdf_history.inc.php
index 66c4f0b232..77eb8c53a9 100644
--- a/html/pages/bill/pdf_history.inc.php
+++ b/html/pages/bill/pdf_history.inc.php
@@ -170,7 +170,7 @@
 
   // Transfer overview
   function transferOverview($bill_id, $history) {
-    global $list_colour_a, $list_colour_b, $config['billing']['base'];
+    global $list_colour_a, $list_colour_b, $config;
 
     $i         = 0;
     $tot       = array();
diff --git a/html/pages/device/edit/services.inc.php b/html/pages/device/edit/services.inc.php
index fa25dc6458..6ef7e0e45d 100644
--- a/html/pages/device/edit/services.inc.php
+++ b/html/pages/device/edit/services.inc.php
@@ -34,7 +34,7 @@ foreach (dbFetchRows("SELECT * FROM `devices` ORDER BY `hostname`") as $dev);
 
 if ($updated) { print_message("Device Settings Saved"); }
 
-if (dbFetchCell("SELECT COUNT(*) from `services` WHERE `device_id` = ?" array($device['device_id'])) > '0')
+if (dbFetchCell("SELECT COUNT(*) from `services` WHERE `device_id` = ?", array($device['device_id'])) > '0')
 {
   $i = "1";
   foreach (dbFetchRows("select * from services WHERE device_id = ? ORDER BY service_type", array($device['device_id'])) as $service)
@@ -43,7 +43,7 @@ if (dbFetchCell("SELECT COUNT(*) from `services` WHERE `device_id` = ?" array($d
   }
 }
 
-if($existform
+if($existform)
 {
   echo('<div style="float: left;">');
   echo("
diff --git a/html/pages/device/loadbalancer/ace_vservers.inc.php b/html/pages/device/loadbalancer/ace_vservers.inc.php
index 0676830141..65032709c8 100644
--- a/html/pages/device/loadbalancer/ace_vservers.inc.php
+++ b/html/pages/device/loadbalancer/ace_vservers.inc.php
@@ -15,7 +15,7 @@ $sep = "";
 foreach ($menu_options as $option => $text)
 {
   if ($_GET['type'] == $option) { echo("<span class='pagemenu-selected'>"); }
-  echo('<a href="'.generate_url($vars, array('type' => $option)).'">'.$text.'</a>')
+  echo('<a href="'.generate_url($vars, array('type' => $option)).'">'.$text.'</a>');
   if ($_GET['type'] == $option) { echo("</span>"); }
   echo(" | ");
 }
@@ -33,7 +33,7 @@ foreach ($graph_types as $type => $descr)
   echo("$type_sep");
   if ($_GET['opte'] == $type) { echo("<span class='pagemenu-selected'>"); }
   echo('<a href="device/device=' . $device['device_id'] . '/tab=routing/type=loadbalancer_vservers/graphs/'.$type.'/">'.$descr.'</a>');
-  echo('<a href="'.generate_url($vars, array('type' => 'loadbalancer_ace_vservers')).'">'.$text.'</a>')
+  echo('<a href="'.generate_url($vars, array('type' => 'loadbalancer_ace_vservers')).'">'.$text.'</a>');
   if ($_GET['opte'] == $type) { echo("</span>"); }
 
   $type_sep = " | ";
diff --git a/scripts/mailscanner.php b/scripts/mailscanner.php
index 9a23e50193..53e5412f97 100644
--- a/scripts/mailscanner.php
+++ b/scripts/mailscanner.php
@@ -56,7 +56,7 @@
 	}
 	
 	function clearStats($mailstats) {
-	    if (file_exists($mailstats) {
+	    if (file_exists($mailstats)) {
 		$fp	= fopen($mailstats, 'w');
 		fwrite($fp, "mess_recv:0\n");
 		fwrite($fp, "mess_rejected:0\n");