From 0eca7ca8d325375643bf3d1f498345565fef24e0 Mon Sep 17 00:00:00 2001
From: Stephen Hemminger <stephen.hemminger@vyatta.com>
Date: Thu, 28 May 2009 16:12:13 -0700
Subject: Handle percent without % sign

Bug 4465
When using percent for netem, allow value without percent
---
 lib/Vyatta/Qos/Util.pm | 23 +++++++++++------------
 1 file changed, 11 insertions(+), 12 deletions(-)

diff --git a/lib/Vyatta/Qos/Util.pm b/lib/Vyatta/Qos/Util.pm
index 7f5454d..151b1d3 100644
--- a/lib/Vyatta/Qos/Util.pm
+++ b/lib/Vyatta/Qos/Util.pm
@@ -113,18 +113,17 @@ sub getPercent {
     my $percent = shift;
     my ( $num, $suffix ) = get_num($percent);
 
-    ( $suffix eq '%' )
-      or die "$percent incorrect suffix (expect %)\n";
-    defined $num
-      or die "$percent is not a valid percent bandwidth (not a number)\n";
-    ( $num >= 0 )
-      or die
-      "$percent is not a acceptable percent bandwidth (negative value)\n";
-    ( $num <= 100 )
-      or die
-      "$percent is not a acceptable percent bandwidth (greater than 100%)\n";
-
-    return $num;
+    if (defined $suffix && $suffix ne '%' ) {
+	die "$percent incorrect suffix (expect %)\n";
+    } elsif (! defined $num) {
+	die "$percent is not a valid percent (not a number)\n";
+    } elsif ( $num < 0 ) {
+	die "$percent is not a acceptable percent (negative value)\n";
+    } elsif ( $num > 100 ) {
+	die "$percent is not a acceptable percent (greater than 100%)\n";
+    } else {
+	return $num;
+    }
 }
 
 # Default time units for tc are usec.
-- 
cgit v1.2.3