haproxy-devel, fix carp check settings for pfSense 2.2

Conflicts:
	config/haproxy-devel/haproxy_global.php

2 files changed, 24 insertions, 22 deletions

diff --git a/config/haproxy-devel/haproxy.inc b/config/haproxy-devel/haproxy.inc
index fd4262a5..bc2b0ac8 100644
--- a/config/haproxy-devel/haproxy.inc
+++ b/config/haproxy-devel/haproxy.inc
@@ -1537,6 +1537,22 @@ function haproxy_plugin_certificates($pluginparams) {
 	return $result;
 }
 
+function haproxy_carpipismaster($ip) {
+	global $config;
+	foreach($config['virtualip']['vip'] as $carp) {
+		if ($carp['mode'] != "carp")
+			continue;
+		$ipaddress = $carp['subnet'];
+		if ($ipaddress != $ip)
+			continue;
+
+		$carp_int = "{$carp['interface']}_vip{$carp['vhid']}";
+		$status = get_carp_interface_status($carp_int);
+		return ($status == "MASTER");
+	}
+	return null;
+}
+
 function haproxy_check_run($reload) {
 	global $config, $g, $haproxy_run_message;
 
@@ -1549,8 +1565,8 @@ function haproxy_check_run($reload) {
 
 	if(isset($a_global['enable'])) {
 		if (isset($a_global['carpdev'])) {
-			$status = get_carp_interface_status($a_global['carpdev']);
-			if ($status != "MASTER") {
+			$status = haproxy_carpipismaster($a_global['carpdev']);
+			if (!$status) {
 				if (haproxy_is_running()) {
 					log_error("Stopping haproxy on CARP backup.");
 					//exec("/bin/pkill -F /var/run/haproxy.pid haproxy");//doesnt work for multiple pid's in a pidfile
diff --git a/config/haproxy-devel/haproxy_global.php b/config/haproxy-devel/haproxy_global.php
index 60c2fd9d..978d778d 100755
--- a/config/haproxy-devel/haproxy_global.php
+++ b/config/haproxy-devel/haproxy_global.php
@@ -271,26 +271,12 @@ function enable_change(enable_change) {
 					Carp monitor
 				</td>
 				<td class="vtable">
-					<select name="carpdev" class="formfld">
-					<option value="disabled" <?php if (!isset($pconfig['carpdev'])) echo "selected"; ?>>
-						disabled
-					</option>
-				<?php
-					if(is_array($config['virtualip']['vip'])) {
-					foreach($config['virtualip']['vip'] as $carp):
-						if ($carp['mode'] != "carp") continue;
-						$ipaddress = $carp['subnet'];
-						$carp_int = trim(find_carp_interface($ipaddress));
-				?>
-					<option value="<?=$carp_int;?>"
-					 <?php if (isset($pconfig['carpdev']) && $carp_int == $pconfig['carpdev']) echo "selected"; ?>>
-						<?=$carp_int;?> (<?=$ipaddress;?>)
-					</option>
-				<?php
-					endforeach;
-					}
-				?>
-					</select>
+					<?php
+					$vipinterfaces = array();
+					$vipinterfaces[] = array('ip' => '', 'name' => 'Disabled');
+					$vipinterfaces += haproxy_get_bindable_interfaces($ipv="ipv4,ipv6", $interfacetype="carp");
+					echo_html_select('carpdev',$vipinterfaces, $pconfig['carpdev'],"No carp interfaces pressent");
+					?>				
 					<br/>
 					Monitor carp interface and only run haproxy on the firewall which is MASTER.
 				</td>