comparison view/View.cpp @ 1020:69c4f8403ebf colourschemes

Fix integer overflow in getFrameForX

 
 #include <unistd.h>
 
 //#define DEBUG_VIEW 1
 //#define DEBUG_VIEW_WIDGET_PAINT 1
-
 
 View::View(QWidget *w, bool showProgress) :
     QFrame(w),
     m_centreFrame(0),
     m_zoomLevel(1024),

 }
 
 sv_frame_t
 View::getFrameForX(int x) const
 {
-    int z = m_zoomLevel;
+    sv_frame_t z = m_zoomLevel; // nb not just int, or multiplication may overflow
     sv_frame_t frame = m_centreFrame - (width()/2) * z;
 
+    frame = (frame / z) * z; // this is start frame
+    frame = frame + x * z;
+
 #ifdef DEBUG_VIEW_WIDGET_PAINT
-    SVDEBUG << "View::getFrameForX(" << x << "): z = " << z << ", m_centreFrame = " << m_centreFrame << ", width() = " << width() << ", frame = " << frame << endl;
-#endif
-
-    frame = (frame / z) * z; // this is start frame
-    return frame + x * z;
+    cerr << "View::getFrameForX(" << x << "): z = " << z << ", m_centreFrame = " << m_centreFrame << ", width() = " << width() << ", frame = " << frame << endl;
+#endif
+
+    return frame;
 }
 
 double
 View::getYForFrequency(double frequency,
 		       double minf,