Skip to content

Commit

Permalink
Fix another race in the filtered touch implementation
Browse files Browse the repository at this point in the history
The lastPoints is used outside of the mutex-locked area, so of course
it cannot be a reference.

Change-Id: Ic69f9acd643366f8086e96b026a47d5de40cd77b
Reviewed-by: Laszlo Agocs <[email protected]>
  • Loading branch information
sletta committed Mar 3, 2017
1 parent f874b6e commit 74bb656
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -883,7 +883,7 @@ void QEvdevTouchScreenHandlerThread::filterAndSendTouchPoints()
}

QList<QWindowSystemInterface::TouchPoint> points = m_handler->d->m_touchPoints;
const QList<QWindowSystemInterface::TouchPoint> &lastPoints = m_handler->d->m_lastTouchPoints;
QList<QWindowSystemInterface::TouchPoint> lastPoints = m_handler->d->m_lastTouchPoints;

m_handler->d->m_mutex.unlock();

Expand Down

0 comments on commit 74bb656

Please sign in to comment.