From 0d14872adcb767a729808618a1385ad4bdafb277 Mon Sep 17 00:00:00 2001 From: Klaus Schmidinger Date: Sat, 18 May 2013 12:43:41 +0200 Subject: [PATCH] Fixed an endless loop in the DrawEllipse() functions for very small ellipses --- CONTRIBUTORS | 1 + HISTORY | 4 +++- osd.c | 10 +++++----- 3 files changed, 9 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTORS b/CONTRIBUTORS index 2b1f8df4..3f8cfb90 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -3146,6 +3146,7 @@ Zoran Turalija Stefan Braun for reporting an endless loop in cTextWrapper::Set() in case the given Width is smaller than one character + for reporting an endless loop in the DrawEllipse() functions for very small ellipses Jochen Dolze for changing cThread::SetIOPriority() from "best effort class" to "idle class" in order diff --git a/HISTORY b/HISTORY index 9fa5f952..3e66a707 100644 --- a/HISTORY +++ b/HISTORY @@ -7795,7 +7795,7 @@ Video Disk Recorder Revision History improve overall performance when an editing process is running (thanks to Jochen Dolze). -2013-05-02: Version 2.0.2 +2013-05-18: Version 2.0.2 - Fixed multiple occurrences of the same directory in the recordings list in case there are directories that only differ in non-alphanumeric characters (was broken by @@ -7808,3 +7808,5 @@ Video Disk Recorder Revision History one single sequence. - Fixed an error message when parsing SCR values in diseqc.conf. - Fixed an unexpected RCS version tag in the newplugin script. +- Fixed an endless loop in the DrawEllipse() functions for very small ellipses (reported + by Stefan Braun). diff --git a/osd.c b/osd.c index cc652b62..45d4419c 100644 --- a/osd.c +++ b/osd.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: osd.c 2.38 2013/02/14 15:50:19 kls Exp $ + * $Id: osd.c 2.38.1.1 2013/05/18 12:41:48 kls Exp $ */ #include "osd.h" @@ -639,8 +639,8 @@ void cBitmap::DrawEllipse(int x1, int y1, int x2, int y2, tColor Color, int Quad case 8: cy = y1; rx /= 2; break; default: ; } - int TwoASquare = 2 * rx * rx; - int TwoBSquare = 2 * ry * ry; + int TwoASquare = max(1, 2 * rx * rx); + int TwoBSquare = max(1, 2 * ry * ry); int x = rx; int y = 0; int XChange = ry * ry * (1 - 2 * rx); @@ -1380,8 +1380,8 @@ void cPixmapMemory::DrawEllipse(const cRect &Rect, tColor Color, int Quadrants) case 8: cy = y1; rx /= 2; break; default: ; } - int TwoASquare = 2 * rx * rx; - int TwoBSquare = 2 * ry * ry; + int TwoASquare = max(1, 2 * rx * rx); + int TwoBSquare = max(1, 2 * ry * ry); int x = rx; int y = 0; int XChange = ry * ry * (1 - 2 * rx);