diff --git a/usr/local/www/carp_status.php b/usr/local/www/carp_status.php
index 1faebf0818..9524aef838 100644
--- a/usr/local/www/carp_status.php
+++ b/usr/local/www/carp_status.php
@@ -212,7 +212,7 @@ include("head.inc");
 								$icon = "<img {$align} src=\"/themes/".$g['theme']."/images/icons/icon_pass_d.gif\" alt=\"backup\" />";
 							} else if($status == "INIT") {
 								$icon = "<img {$align} src=\"/themes/".$g['theme']."/images/icons/icon_log.gif\" alt=\"init\" />";
-							} else
+							} else {
 								$icon = "";
 							}
 						}
diff --git a/usr/local/www/services_unbound.php b/usr/local/www/services_unbound.php
index 33dbe46edc..decb25d70a 100644
--- a/usr/local/www/services_unbound.php
+++ b/usr/local/www/services_unbound.php
@@ -281,7 +281,7 @@ function show_advanced_dns() {
 									<select id="active_interface" name="active_interface[]" multiple="multiple" size="<?php echo $size; ?>">
 										<option value="all" <?php if (empty($pconfig['active_interface']) || empty($pconfig['active_interface'][0]) || in_array("all", $pconfig['active_interface'], true)) echo 'selected="selected"'; ?>>All</option>
 										<?php
-											foreach ($interface_addresses as $laddr => ldescr):
+											foreach ($interface_addresses as $laddr => $ldescr):
 												$selected = "";
 												if (in_array($laddr, $pconfig['active_interface']))
 													$selected = 'selected="selected"';
@@ -302,7 +302,7 @@ function show_advanced_dns() {
 									<select id="outgoing_interface" name="outgoing_interface[]" multiple="multiple" size="<?php echo $size; ?>">
 										<option value="" <?php if (empty($pconfig['outgoing_interface']) || empty($pconfig['outgoing_interface'][0])) echo 'selected="selected"'; ?>>All</option>
 										<?php
-											foreach ($interface_addresses as $laddr):
+											foreach ($interface_addresses as $laddr => $ldescr):
 												$selected = "";
 												if (in_array($laddr, $pconfig['outgoing_interface']))
 												$selected = 'selected="selected"';
diff --git a/usr/local/www/xmlrpc.php b/usr/local/www/xmlrpc.php
index 49fad44ad2..ec6d458d36 100755
--- a/usr/local/www/xmlrpc.php
+++ b/usr/local/www/xmlrpc.php
@@ -213,9 +213,9 @@ function restore_config_section_xmlrpc($raw_params) {
 			foreach ($config['virtualip']['vip'] as $vipindex => $vip) {
 				if ($vip['mode'] == "carp")
 					$oldvips["{$vip['interface']}_vip{$vip['vhid']}"] = "{$vip['password']}{$vip['advskew']}{$vip['subnet']}{$vip['subnet_bits']}{$vip['advbase']}";
-				else if ($vip['mode'] == "ipalias" && (substr($vip['interface'], 0, 4) == '_vip') || strpos($vip['interface'], "lo0")))
+				else if ($vip['mode'] == "ipalias" && (substr($vip['interface'], 0, 4) == '_vip') || strpos($vip['interface'], "lo0"))
 					$oldvips[$vip['subnet']] = "{$vip['interface']}{$vip['subnet']}{$vip['subnet_bits']}";
-				else if (($vip['mode'] == "ipalias" || $vip['mode'] == 'proxyarp') && !(substr($vip['interface'], 0, 4) == '_vip') || strpos($vip['interface'], "lo0")))
+				else if (($vip['mode'] == "ipalias" || $vip['mode'] == 'proxyarp') && !(substr($vip['interface'], 0, 4) == '_vip') || strpos($vip['interface'], "lo0"))
 					$vipbackup[] = $vip;
 			}
 		}