From 975bee12a871a4f09def1b7c9069d5ae1f766d2c Mon Sep 17 00:00:00 2001
From: Kees Monshouwer <mind04@monshouwer.org>
Date: Mon, 26 Aug 2013 23:26:43 +0200
Subject: [PATCH]  fix ringbuffer size decrease

---
 pdns/statbag.cc | 32 +++++++++++++++++---------------
 1 file changed, 17 insertions(+), 15 deletions(-)

diff --git a/pdns/statbag.cc b/pdns/statbag.cc
index 83421ff75f..430c048299 100644
--- a/pdns/statbag.cc
+++ b/pdns/statbag.cc
@@ -189,26 +189,28 @@ void StatRing::resize(unsigned int newsize)
     return;
   Lock l(d_lock);
 
-  if(newsize>d_size) {
-    d_size=newsize;
-    d_items.resize(d_size);
-    return;
-  }
-
   // this is the hard part, shrink
-  int startpos=d_pos-newsize;
-  int rpos;
-  vector<string>newring;
-  for(unsigned int i=startpos;i<d_pos;++i) {
-    rpos= i;
+  if(newsize<d_size) {
+    unsigned int startpos=0;
+    if (d_pos>newsize)
+      startpos=d_pos-newsize;
+
+    vector<string>newring;
+    for(unsigned int i=startpos;i<d_pos;++i) {
+      newring.push_back(d_items[i%d_size]);
+    }
 
-    newring.push_back(d_items[rpos%d_size]);
+    d_items=newring;
+    d_size=newring.size();
+    d_pos=min(d_pos,newsize);
   }
-  d_items=newring;
-  d_size=newsize;
-  d_pos=d_size-1;
 
+  if(newsize>d_size) {
+    d_size=newsize;
+    d_items.resize(d_size);
+  }
 }
+
 StatRing::~StatRing()
 {
   // do not clean up d_lock, it is shared
-- 
2.47.2