Add range_check method to QIntC

Jay Berkenbilt
1 parent 24196c08
Showing 3 changed files with 16 additions and 19 deletions
include/qpdf/BufferInputSource.hh
include/qpdf/QIntC.hh
libqpdf/BufferInputSource.cc
@@ -54,8 +54,6 @@ class BufferInputSource: public InputSource
     virtual void unreadCh(char ch);
  
   private:
-    static void range_check(qpdf_offset_t cur, qpdf_offset_t delta);
-
     class Members
     {
         friend class BufferInputSource;
@@ -222,6 +222,20 @@ namespace QIntC // QIntC = qpdf Integer Conversion
     {
         return IntConverter<T, unsigned long long>::convert(i);
     }
+
+    template <typename T>
+    void range_check(T const& cur, T const& delta)
+    {
+        if ((delta > 0) &&
+            ((std::numeric_limits<T>::max() - cur) < delta))
+        {
+            std::ostringstream msg;
+            msg.imbue(std::locale::classic());
+            msg << "adding " << delta << " to " << cur
+                << " would cause an integer overflow";
+            throw std::range_error(msg.str());
+        }
+    }
 };
  
 #endif // QINTC_HH
@@ -102,21 +102,6 @@ BufferInputSource::tell()
 }
  
 void
-BufferInputSource::range_check(qpdf_offset_t cur, qpdf_offset_t delta)
-{
-    if ((delta > 0) &&
-        ((std::numeric_limits<qpdf_offset_t>::max() - cur) < delta))
-    {
-        std::ostringstream msg;
-        msg.imbue(std::locale::classic());
-        msg << "seeking forward from " << cur
-            << " by " << delta
-            << " would cause an overflow of the offset type";
-        throw std::range_error(msg.str());
-    }
-}
-
-void
 BufferInputSource::seek(qpdf_offset_t offset, int whence)
 {
     switch (whence)
@@ -126,12 +111,12 @@ BufferInputSource::seek(qpdf_offset_t offset, int whence)
 	break;
  
       case SEEK_END:
-        range_check(this->m->max_offset, offset);
+        QIntC::range_check(this->m->max_offset, offset);
 	this->m->cur_offset = this->m->max_offset + offset;
 	break;
  
       case SEEK_CUR:
-        range_check(this->m->cur_offset, offset);
+        QIntC::range_check(this->m->cur_offset, offset);
 	this->m->cur_offset += offset;
 	break;