fix for the segfaults when displaying videos (it just had sometimes

invalid inputimages when repainting)

1 files changed, 2 insertions, 0 deletions

diff --git a/noncore/multimedia/opieplayer2/xinevideowidget.cpp b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
index 1ac9277..9e86041 100644
--- a/noncore/multimedia/opieplayer2/xinevideowidget.cpp
+++ b/noncore/multimedia/opieplayer2/xinevideowidget.cpp
@@ -1,304 +1,306 @@
 
 /*
                             This file is part of the Opie Project
 
                              Copyright (c)  2002 Robert Griebl <sandman@handhelds.org>
                              Copyright (c)  2002 Holger Freyther <zecke@handhelds.org>
               =.
             .=l.
            .>+-=
  _;:,     .>    :=|.         This program is free software; you can
 .> <`_,   >  .   <=          redistribute it and/or  modify it under
 :`=1 )Y*s>-.--   :           the terms of the GNU General Public
 .="- .-=="i,     .._         License as published by the Free Software
  - .   .-<_>     .<>         Foundation; either version 2 of the License,
      ._= =}       :          or (at your option) any later version.
     .%`+i>       _;_.
     .i_,=:_.      -<s.       This program is distributed in the hope that
      +  .  -:.       =       it will be useful,  but WITHOUT ANY WARRANTY;
     : ..    .:,     . . .    without even the implied warranty of
     =_        +     =;=|`    MERCHANTABILITY or FITNESS FOR A
   _.=:.       :    :=>`:     PARTICULAR PURPOSE. See the GNU
 ..}^=.=       =       ;      Library General Public License for more
 ++=   -.     .`     .:       details.
  :     =  ...= . :.=-
  -.   .:....=;==+<;          You should have received a copy of the GNU
   -_. . .   )=.  =           Library General Public License along with
     --        :-=`           this library; see the file COPYING.LIB.
                              If not, write to the Free Software Foundation,
                              Inc., 59 Temple Place - Suite 330,
                              Boston, MA 02111-1307, USA.
 
 */
 
 #include <qimage.h>
 #include <qdirectpainter_qws.h>
 #include <qgfx_qws.h>
 #include <qsize.h>
 #include <qapplication.h>
 
 #include <qpe/resource.h>
 
 #include "xinevideowidget.h"
 
 
 // 0 deg rot: copy a line from src to dst (use libc memcpy)
 
 // 180 deg rot: copy a line from src to dst reversed
 
 /*
  * This code relies the len be a multiply of 16bit
  */
 static inline void memcpy_rev ( void *_dst, void *_src, size_t len )
 {
 
     /*
      * move the source to the end
      */
     char *src_c = static_cast<char*>(_src) + len;
 
     /*
      * as we copy by 16bit and not 8bit
      * devide the length by two
      */
     len >>= 1;
 
     short int* dst = static_cast<short int*>( _dst );
     short int* src = reinterpret_cast<short int*>( src_c );
 
     /*
      * Increment dst after assigning
      * Decrement src before  assigning becase we move backwards
      */
     while ( len-- )
         *dst++ = *--src;
 
 }
 
 // 90 deg rot: copy a column from src to dst
 
 static inline void memcpy_step ( void *_dst, void *_src, size_t len, size_t step )
 {
     short int *dst = static_cast<short int*>( _dst );
     short int *src = static_cast<short int*>( _src );
 
     len >>= 1;
 
     /*
      * Copy 16bit from src to dst and move to the next address
      */
     while ( len-- ) {
         *dst++ = *src;
         src = reinterpret_cast<short int*>(reinterpret_cast<char*>(src)+step);
     }
 }
 
 // 270 deg rot: copy a column from src to dst reversed
 
 static inline void memcpy_step_rev ( void *_dst, void *_src, size_t len, size_t step )
 {
     len >>= 1;
 
     char *src_c = static_cast<char*>( _src ) + (len*step);
     short int* dst = static_cast<short int*>( _dst );
     short int* src = reinterpret_cast<short int*>( src_c );
 
     while ( len-- ) {
         src_c -= step;
         src = reinterpret_cast<short int*>( src_c );
         *dst++ = *src;
     }
 }
 
 
 XineVideoWidget::XineVideoWidget ( QWidget* parent, const char* name )
 	: QWidget ( parent, name, WRepaintNoErase | WResizeNoErase )
 {
 	setBackgroundMode ( NoBackground );
 
 	m_logo              = 0;
 	m_buff              = 0;
 	m_bytes_per_line_fb = qt_screen-> linestep ( );
 	m_bytes_per_pixel   = ( qt_screen->depth() + 7 ) / 8;
 	m_rotation          = 0;
 }
 
 
 XineVideoWidget::~XineVideoWidget ( )
 {
 	delete m_logo;
 }
 
 void XineVideoWidget::clear ( )
 {
 	m_buff = 0;
 	repaint ( false );
 }
 
 QSize XineVideoWidget::videoSize() const
 {
     QSize s = size();
     bool fs = ( s == qApp->desktop()->size() );
 
     // if we are in fullscreen mode, do not rotate the video
     // (!! the paint routine uses m_rotation + qt_screen-> transformOrientation() !!)
     m_rotation = fs ? - qt_screen->transformOrientation() : 0;
 
     if ( fs && qt_screen->isTransformed() )
         s = qt_screen->mapToDevice( s );
 
     return s;
 }
 
 void XineVideoWidget::paintEvent ( QPaintEvent * )
 {
 	if ( m_buff == 0 ) {
 		QPainter p ( this );
 		p. fillRect ( rect ( ), black );
 		if ( m_logo )
 			p. drawImage ( 0, 0, *m_logo );
 	}
 	else {
 		// Qt needs to be notified which areas were really updated .. strange
 		QArray <QRect> qt_bug_workaround_clip_rects;
 
 		{
 			QDirectPainter dp ( this );
 
 			int rot = dp. transformOrientation ( ) + m_rotation; // device rotation + custom rotation
 
 			uchar *fb = dp. frameBuffer ( );
 			uchar *frame = m_buff;
 
 			// where is the video frame in fb coordinates
 			QRect framerect = qt_screen-> mapToDevice ( QRect ( mapToGlobal ( m_thisframe. topLeft ( )), m_thisframe. size ( )), QSize ( qt_screen-> width ( ), qt_screen-> height ( )));
 
 			qt_bug_workaround_clip_rects. resize ( dp. numRects ( ));
 
 			for ( int i = dp. numRects ( ) - 1; i >= 0; i-- ) {
 				const QRect &clip = dp. rect ( i );
 
 				qt_bug_workaround_clip_rects [ i ] = qt_screen-> mapFromDevice ( clip, QSize ( qt_screen-> width ( ), qt_screen-> height ( )));
 
 				uchar *dst = fb + ( clip. x ( ) * m_bytes_per_pixel ) + ( clip. y ( ) * m_bytes_per_line_fb ); 	// clip x/y in the fb
 				uchar *src = frame;
 
 				// Adjust the start the source data based on the rotation (xine frame)
 				switch ( rot ) {
 					case  0: src += ((( clip. x ( ) - framerect. x ( )) * m_bytes_per_pixel ) + (( clip. y ( ) - framerect. y ( )) * m_bytes_per_line_frame )); break;
 					case  1: src += ((( clip. y ( ) - framerect. y ( )) * m_bytes_per_pixel ) + (( clip. x ( ) - framerect. x ( )) * m_bytes_per_line_frame ) + (( framerect. height ( ) - 1 ) * m_bytes_per_pixel )); break;
 					case  2: src += ((( clip. x ( ) - framerect. x ( )) * m_bytes_per_pixel ) + (( clip. y ( ) - framerect. y ( )) * m_bytes_per_line_frame ) + (( framerect. height ( ) - 1 ) * m_bytes_per_line_frame )); break;
 					case  3: src += ((( clip. y ( ) - framerect. y ( )) * m_bytes_per_pixel ) + (( clip. x ( ) - framerect. x ( )) * m_bytes_per_line_frame )); break;
 					default: break;
 				}
 
 				// all of the following widths/heights are fb relative (0deg rotation)
 
 				uint leftfill = 0;   // black border on the "left" side of the video frame
 				uint framefill = 0;  // "width" of the video frame
 				uint rightfill = 0;  // black border on the "right" side of the video frame
 				uint clipwidth = clip. width ( ) * m_bytes_per_pixel;  // "width" of the current clip rect
 
 				if ( clip. left ( ) < framerect. left ( ))
 					leftfill = (( framerect. left ( ) - clip. left ( )) * m_bytes_per_pixel ) <? clipwidth;
 				if ( clip. right ( ) > framerect. right ( ))
 					rightfill = (( clip. right ( ) - framerect. right ( )) * m_bytes_per_pixel ) <? clipwidth;
 
 				framefill = clipwidth - ( leftfill + rightfill );
 
 				for ( int y = clip. top ( ); y <= clip. bottom ( ); y++ ) {
 					if (( y < framerect. top ( )) || ( y > framerect. bottom ( ))) {
 						// "above" or "below" the video -> black
 						memset ( dst, 0, clipwidth );
 					}
 					else {
 						if ( leftfill )
 							memset ( dst, 0, leftfill ); // "left" border -> black
 
 						if ( framefill ) { // blit in the video frame
 							// see above for an explanation of the different memcpys
 
 							switch ( rot ) {
 								case  0: memcpy ( dst + leftfill, src, framefill & ~1 ); break;
 								case  1: memcpy_step ( dst + leftfill, src, framefill, m_bytes_per_line_frame ); break;
 								case  2: memcpy_rev ( dst + leftfill, src, framefill ); break;
 								case  3: memcpy_step_rev ( dst + leftfill, src, framefill, m_bytes_per_line_frame ); break;
 								default: break;
 							}
 						}
 						if ( rightfill )
 							memset ( dst + leftfill + framefill, 0, rightfill ); // "right" border -> black
 					}
 
 					dst += m_bytes_per_line_fb; // advance one line in the framebuffer
 
 					// advance one "line" in the xine frame data
 					switch ( rot ) {
 						case  0: src += m_bytes_per_line_frame;	break;
 						case  1: src -= m_bytes_per_pixel;      break;
 						case  2: src -= m_bytes_per_line_frame; break;
 						case  3: src += m_bytes_per_pixel;      break;
 						default: break;
 					}
 				}
 			}
 		}
 
 		{
 			// QVFB hack by Martin Jones
 			// We need to "touch" all affected clip rects with a normal QPainter in addition to the QDirectPainter
 
 			QPainter p ( this );
 
 			for ( int i = qt_bug_workaround_clip_rects. size ( ) - 1; i >= 0; i-- ) {
 				p. fillRect ( QRect ( mapFromGlobal ( qt_bug_workaround_clip_rects [ i ]. topLeft ( )), qt_bug_workaround_clip_rects [ i ]. size ( )), QBrush ( NoBrush ));
 			}
 		}
 	}
 }
 
 
 QImage *XineVideoWidget::logo ( ) const
 {
 	return m_logo;
 }
 
 
 void XineVideoWidget::setLogo ( QImage* logo )
 {
 	delete m_logo;
 	m_logo = logo;
 }
 
 void XineVideoWidget::setVideoFrame ( uchar* img, int w, int h, int bpl )
 {
 	bool rot90 = (( -m_rotation ) & 1 );
 
 	if ( rot90 ) { // if the rotation is 90 or 270 we have to swap width / height
 		int d = w;
 		w = h;
 		h = d;
 	}
 
 	m_lastframe = m_thisframe;
 	m_thisframe. setRect (( width ( ) - w ) / 2, ( height ( ) - h ) / 2, w , h );
 
 	m_buff                 = img;
 	m_bytes_per_line_frame = bpl;
 
 	// only repaint the area that *really* needs to be repainted
 
 	repaint ((( m_thisframe & m_lastframe ) != m_lastframe ) ? m_lastframe : m_thisframe, false );
+	// ensure that we always have a valid frame!
+	m_buff = 0;
 }
 
 void XineVideoWidget::resizeEvent ( QResizeEvent * )
 {
     emit videoResized( videoSize() );
 }
 
 
 void XineVideoWidget::mouseReleaseEvent ( QMouseEvent * /*me*/ )
 {
 	emit clicked();
 }