mirror of https://github.com/apache/cloudstack.git
bug 6485: if tap device exists, include them in the rules
This commit is contained in:
parent
696f3f5fcb
commit
f6c9c27b76
|
|
@ -1674,6 +1674,13 @@ def default_network_rules(session, args):
|
|||
return 'false'
|
||||
|
||||
vif = "vif" + domid + ".0"
|
||||
tap = "tap" + domid + ".0"
|
||||
vifs = [vif]
|
||||
try:
|
||||
util.pread2(['ifconfig', tap])
|
||||
vifs.append(tap)
|
||||
except:
|
||||
pass
|
||||
|
||||
delete_rules_for_vm_in_bridge_firewall_chain(vm_name)
|
||||
|
||||
|
|
@ -1695,19 +1702,22 @@ def default_network_rules(session, args):
|
|||
util.pread2(['iptables', '-F', vmchain])
|
||||
|
||||
try:
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-out', vif, '-j', vmchain])
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', vif, '-j', vmchain])
|
||||
for v in vifs:
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-out', v, '-j', vmchain])
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', v, '-j', vmchain])
|
||||
util.pread2(['iptables', '-A', vmchain, '-m', 'state', '--state', 'RELATED,ESTABLISHED', '-j', 'ACCEPT'])
|
||||
#allow dhcp
|
||||
util.pread2(['iptables', '-A', vmchain, '-p', 'udp', '--dport', '67:68', '--sport', '67:68', '-j', 'ACCEPT'])
|
||||
#don't let vm spoof its ip address
|
||||
util.pread2(['iptables', '-A', vmchain, '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', vif, '--source', vm_ip, '-j', 'RETURN'])
|
||||
for v in vifs:
|
||||
util.pread2(['iptables', '-A', vmchain, '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', v, '--source', vm_ip, '-j', 'RETURN'])
|
||||
util.pread2(['iptables', '-A', vmchain, '-j', 'DROP'])
|
||||
except:
|
||||
util.SMlog("Failed to program default rules for vm " + vm_name)
|
||||
return 'false'
|
||||
|
||||
default_ebtables_rules(vm_name, vif, vm_ip, vm_mac)
|
||||
for v in vifs:
|
||||
default_ebtables_rules(vm_name, v, vm_ip, vm_mac)
|
||||
|
||||
if write_rule_log_for_vm(vmName, vm_id, vm_ip, domid, '_initial_', '-1') == False:
|
||||
util.SMlog("Failed to log default network rules, ignoring")
|
||||
|
|
@ -1782,16 +1792,27 @@ def network_rules_for_rebooted_vm(session, vmName):
|
|||
return True
|
||||
|
||||
vif = "vif" + curr_domid + ".0"
|
||||
tap = "tap" + domid + ".0"
|
||||
vifs = [vif]
|
||||
try:
|
||||
util.pread2(['ifconfig', tap])
|
||||
vifs.append(tap)
|
||||
except:
|
||||
pass
|
||||
vmchain = '-'.join(vm_name.split('-')[:-1])
|
||||
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-out', vif, '-j', vmchain])
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', vif, '-j', vmchain])
|
||||
for v in vifs:
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-out', v, '-j', vmchain])
|
||||
util.pread2(['iptables', '-A', 'BRIDGE-FIREWALL', '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', v, '-j', vmchain])
|
||||
|
||||
#change antispoof rule in vmchain
|
||||
try:
|
||||
delcmd = "iptables -S " + vmchain + " | grep physdev-in | sed 's/-A/-D/'"
|
||||
inscmd = "iptables -S " + vmchain + " | grep physdev-in | sed -r 's/vif[0-9]+.0/" + vif + "/' | sed 's/-A/-I/'"
|
||||
inscmd = "iptables -S " + vmchain + " | grep physdev-in | grep vif | sed -r 's/vif[0-9]+.0/" + vif + "/' | sed 's/-A/-I/'"
|
||||
inscmd2 = "iptables -S " + vmchain + " | grep physdev-in | grep tap | sed -r 's/tap[0-9]+.0/" + tap + "/' | sed 's/-A/-I/'"
|
||||
|
||||
ipts = []
|
||||
for cmd in [delcmd, inscmd]:
|
||||
for cmd in [delcmd, inscmd, inscmd2]:
|
||||
cmds = util.pread2(['/bin/bash', '-c', cmd]).split('\n')
|
||||
cmds.pop()
|
||||
for c in cmds:
|
||||
|
|
@ -2010,6 +2031,13 @@ def network_rules(session, args):
|
|||
return 'false'
|
||||
|
||||
vif = "vif" + domid + ".0"
|
||||
tap = "tap" + domid + ".0"
|
||||
vifs = [vif]
|
||||
try:
|
||||
util.pread2(['ifconfig', tap])
|
||||
vifs.append(tap)
|
||||
except:
|
||||
pass
|
||||
vm_name = '-'.join(vm_name.split('-')[:-1])
|
||||
vmchain = vm_name
|
||||
|
||||
|
|
@ -2078,7 +2106,8 @@ def network_rules(session, args):
|
|||
|
||||
|
||||
util.pread2(['iptables', '-A', vmchain, '-p', 'udp', '--dport', '67:68', '--sport', '67:68', '-j', 'ACCEPT'])
|
||||
util.pread2(['iptables', '-I', vmchain, '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', vif, '--source', vm_ip, '-j', 'RETURN'])
|
||||
for v in vifs:
|
||||
util.pread2(['iptables', '-I', vmchain, '-m', 'physdev', '--physdev-is-bridged', '--physdev-in', v, '--source', vm_ip, '-j', 'RETURN'])
|
||||
util.pread2(['iptables', '-A', vmchain, '-j', 'DROP'])
|
||||
|
||||
if write_rule_log_for_vm(vmName, vm_id, vm_ip, domid, signature, seqno) == False:
|
||||
|
|
|
|||
Loading…
Reference in New Issue