From: Kees Monshouwer <mind04@monshouwer.org>
Date: Sat, 7 Sep 2013 12:08:56 +0000 (+0200)
Subject: avoid division by 0, and make coverity happy, or not...
X-Git-Tag: rec-3.6.0-rc1~449^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f5cb7e61b22309865cba7706e224d4ac282e6fdb;p=pdns

avoid division by 0, and make coverity happy, or not...
---

diff --git a/pdns/ws.cc b/pdns/ws.cc
index 2f0ce8a5a..6506b4c91 100644
--- a/pdns/ws.cc
+++ b/pdns/ws.cc
@@ -122,13 +122,14 @@ void printtable(ostringstream &ret, const string &ringname, const string &title,
 
   ret<<"<table class=\"data\">";
   int printed=0;
+  int total=max(1,tot);
   for(vector<pair<string,unsigned int> >::const_iterator i=ring.begin();limit && i!=ring.end();++i,--limit) {
-    ret<<"<tr><td>"<<i->first<<"</td><td>"<<i->second<<"</td><td align=right>"<< StatWebServer::makePercentage(i->second*100.0/tot)<<"</td>"<<endl;
+    ret<<"<tr><td>"<<i->first<<"</td><td>"<<i->second<<"</td><td align=right>"<< StatWebServer::makePercentage(i->second*100.0/total)<<"</td>"<<endl;
     printed+=i->second;
   }
   ret<<"<tr><td colspan=3></td></tr>"<<endl;
   if(printed!=tot)
-    ret<<"<tr><td><b>Rest:</b></td><td><b>"<<tot-printed<<"</b></td><td align=right><b>"<< StatWebServer::makePercentage((tot-printed)*100.0/tot)<<"</b></td>"<<endl;
+    ret<<"<tr><td><b>Rest:</b></td><td><b>"<<tot-printed<<"</b></td><td align=right><b>"<< StatWebServer::makePercentage((tot-printed)*100.0/total)<<"</b></td>"<<endl;
 
   ret<<"<tr><td><b>Total:</b></td><td><b>"<<tot<<"</b></td><td align=right><b>100%</b></td>";
   ret<<"</table></div>"<<endl;