[PATCH mogilefs 3/7] Some DBD may report a - rather than undefined, so validate that it is numeric before using it.

Robin H. Johnson robbat2 at gentoo.org
Tue Jun 5 13:38:11 UTC 2007


From: Robin H. Johnson <robbat2 at gentoo.org>

Signed-off-by: Robin H. Johnson <robbat2 at gentoo.org>
---
 server/lib/MogileFS/Worker/Query.pm |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/server/lib/MogileFS/Worker/Query.pm b/server/lib/MogileFS/Worker/Query.pm
index abeaada..456382c 100644
--- a/server/lib/MogileFS/Worker/Query.pm
+++ b/server/lib/MogileFS/Worker/Query.pm
@@ -876,7 +876,7 @@ sub cmd_get_paths {
         my $dev = $dmap->{$devid};
         my $util = $dev->observed_utilization;
 
-        if (defined($util)) {
+        if (defined($util) and $util =~ /\A\d+\Z/) {
             $weight = 102 - $util;
             $weight ||= 100;
         } else {
-- 
1.5.2



More information about the mogilefs mailing list