From 2dd53861dd8005f7175f81cea971f7eaddbecc89 Mon Sep 17 00:00:00 2001
From: Stephen Hemminger <shemminger@vyatta.com>
Date: Tue, 21 Feb 2012 16:11:32 -0800
Subject: Fix validation of value 0

Confusion over if ($value) vs if (defined($value)) in argument processing
lead to 0 always being allowed.
---
 scripts/vyatta-qos-util.pl | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

(limited to 'scripts/vyatta-qos-util.pl')

diff --git a/scripts/vyatta-qos-util.pl b/scripts/vyatta-qos-util.pl
index 6dae105..1d4e754 100755
--- a/scripts/vyatta-qos-util.pl
+++ b/scripts/vyatta-qos-util.pl
@@ -56,10 +56,10 @@ GetOptions(
     "time=s"            => \$timeval,
 ) or usage();
 
-getPercent($percent)		if $percent;
-getPercentOrRate($percentrate)	if $percentrate;
-getRate($rate)			if $rate;
-getBurstSize($burst)		if $burst;
-getProtocol($protocol)		if $protocol;
-getDsfield($dscp)		if $dscp;
-getTime($timeval)		if $timeval;
+getPercent($percent)		if defined($percent);
+getPercentOrRate($percentrate)	if defined($percentrate);
+getRate($rate)			if defined($rate);
+getBurstSize($burst)		if defined($burst);
+getProtocol($protocol)		if defined($protocol);
+getDsfield($dscp)		if defined($dscp);
+getTime($timeval)		if defined($timeval);
-- 
cgit v1.2.3