From d3209ca62403b22974964874f40186cb12d3baff Mon Sep 17 00:00:00 2001 From: Tom Eastep Date: Fri, 8 Aug 2014 08:15:26 -0700 Subject: [PATCH] Correct handling of a physical name in the provider INTERFACE column Signed-off-by: Tom Eastep --- Shorewall/Perl/Shorewall/Providers.pm | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/Shorewall/Perl/Shorewall/Providers.pm b/Shorewall/Perl/Shorewall/Providers.pm index 4ad8d859b..49d81e4d0 100644 --- a/Shorewall/Perl/Shorewall/Providers.pm +++ b/Shorewall/Perl/Shorewall/Providers.pm @@ -454,7 +454,14 @@ sub process_a_provider( $ ) { require_capability 'REALM_MATCH', "Configuring multiple providers through one interface", "s"; } - fatal_error "Unknown Interface ($interface)" unless known_interface( $interface ); + my $interfaceref = known_interface( $interface ); + + fatal_error "Unknown Interface ($interface)" unless $interfaceref; + # + # Switch to the logical name if a physical name was passed + # + $interface = $interfaceref->{name}; + fatal_error "A bridge port ($interface) may not be configured as a provider interface" if port_to_bridge $interface; my $physical = get_physical $interface;