diff options
author | Bjoern Erik Nilsen <bjorn.nilsen@nokia.com> | 2009-05-06 14:32:01 (GMT) |
---|---|---|
committer | Bjoern Erik Nilsen <bjorn.nilsen@nokia.com> | 2009-05-06 15:17:59 (GMT) |
commit | 67c2b69d45eb43078b7e4193ced6cd513602483c (patch) | |
tree | d36450519b6e7f00157f4cb77b9b687666797e9c /src/gui | |
parent | 85f6350812d5a22c496c66f4f91ea8e36e454561 (diff) | |
download | Qt-67c2b69d45eb43078b7e4193ced6cd513602483c.zip Qt-67c2b69d45eb43078b7e4193ced6cd513602483c.tar.gz Qt-67c2b69d45eb43078b7e4193ced6cd513602483c.tar.bz2 |
Crash in QWidget::render when passing an untransformed QPixmap painter.
The crash only occurred on Windows and X11 when running with
-graphicssystem raster. The reason is that the actual paint device
in QRasterPaintEngine::begin() is changed to pixmap->data->buffer(),
which means QPaintEngine::paintDevice() returns something else than
what it was told to paint on (see cb0c899b56b84154f69ddc545991bc6ded96ab01)
The root of the problem, however, was that we used a weird condition
(painter->worldMatrixEnabled(), added in 345072b9 for Qt 4.4) to find
the target device. We did that because the shared painter was completely
different in 4.4. We refactored it in 4.5.0, and we can only trust
QPaintEngine::paintDevice to be the target device.
Auto-test included.
Task-number: 252837
Reviewed-by: Trond
Diffstat (limited to 'src/gui')
-rw-r--r-- | src/gui/kernel/qwidget.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/kernel/qwidget.cpp b/src/gui/kernel/qwidget.cpp index fb9c8cb..b96d8b3 100644 --- a/src/gui/kernel/qwidget.cpp +++ b/src/gui/kernel/qwidget.cpp @@ -4805,7 +4805,7 @@ void QWidget::render(QPainter *painter, const QPoint &targetOffset, Q_ASSERT(engine); QPaintEnginePrivate *enginePriv = engine->d_func(); Q_ASSERT(enginePriv); - QPaintDevice *target = painter->worldMatrixEnabled() ? engine->paintDevice() : painter->device(); + QPaintDevice *target = engine->paintDevice(); Q_ASSERT(target); // Render via a pixmap when dealing with non-opaque painters or printers. |