Bug 201296 - x11-libs/qt-4.3* < 4.3.2-r1 emul-linux-x86-qtlibs < 20071210 QSslSocket missing SSL certificate verification (CVE-2007-5965)
Bug#: 201296 Product:  Gentoo Security Version: unspecified Platform: All
OS/Version: Linux Status: RESOLVED Severity: minor Priority: P2
Resolution: FIXED Assigned To: security@gentoo.org Reported By: rbu@gentoo.org
Component: Vulnerabilities
URL:  http://trolltech.com/company/newsroom/announcements/press.2007-12-21.2182567220
Summary: x11-libs/qt-4.3* < 4.3.2-r1 emul-linux-x86-qtlibs < 20071210 QSslSocket missing SSL certificate verification (CVE-2007-5965)
Keywords:  
Status Whiteboard: A4 [noglsa]
Opened: 2007-12-04 23:31 0000
Description:   Opened: 2007-12-04 23:31 0000
Thiago Macieira of Trolltech wrote:
  Qt 4 has a potential vulnerability in QSslSocket, which might cause a the 
  certificate verification in SSL connections not to be performed. As a 
  consequence, code using QSslSocket might be mislead into thinking the 
  certificate was verified correctly when it actually failed in one or more 
  criterea

  To solve the issue, apply the following patch that is attached.

  The next maintenance release of Qt 4 will have the patch included.

  Versions affected: 4.3.0, 4.3.1 and 4.3.2

------- Comment #1 From Robert Buchholz 2007-12-04 23:33:08 0000 -------
Created an attachment (id=137760) [details]
qsslsocket-fix.patch

Upstream propsed patch

------- Comment #2 From Robert Buchholz 2007-12-04 23:35:43 0000 -------
We're handling this confidential as I am not aware of a coordinated release
date yet. Caleb, please do not commit the patch yet. If you want to, you can
prepare an ebuild and attach it to this bug.

However, since this issue is of a low impact, my advise would be to go normal
stabling process via arch teams once this is public.

------- Comment #3 From Caleb Tennis 2007-12-05 12:53:30 0000 -------
The patch looks pretty harmless, so I won't bother with attaching an ebuild. 
I'll just wait for the announcement or release notification, and throw it into
portage at that time.

------- Comment #4 From Robert Buchholz 2007-12-05 17:09:38 0000 -------
"Qt 4.3.3, due out today, is not affected by this issue. It affects 
only 4.3.0, 4.3.1 and 4.3.2."

So we can bump the ebuild in the tree before disclosure.

------- Comment #5 From Caleb Tennis 2007-12-05 17:20:50 0000 -------
I got my commercial Qt today, but I'm not sure if we want to do that with the
open source one when it's out in a few hours.  Namely, we don't know what else
was "fixed" in 4.2.2 -> 4.2.3.  I vote to just revbump 4.2.2 with the patch. 
In fact, if you want we can bump it in portage with the patch before the
disclosure and not make public mention of the reason for the patch until
disclosure.  Thoughts?

------- Comment #6 From Robert Buchholz 2007-12-05 19:30:02 0000 -------
QT 4.3.3 contains this fix and probably some other patches. Feel free to
include this patch into 4.3.2 and we'll handle prestabling in this bug.

------- Comment #7 From Caleb Tennis 2007-12-05 23:52:47 0000 -------
qt-4.3.2-r1 has been committed with this patch.

------- Comment #8 From Robert Buchholz 2007-12-06 00:09:26 0000 -------
Adding arch security liaisons (plus opfer and armin76) and Chris for releng.

Please test and mark stable x11-libs/qt-4.3.2-r1.
Target keywords : "alpha amd64 hppa ia64 mips ppc ppc64 sparc x86"

------- Comment #9 From Christian Faulhammer 2007-12-06 07:49:53 0000 -------
On x86 I get this, but it goes on fine.

rm -f *~ core *.core
g++ -c -pipe -O2 -Wall -W  -I../../../mkspecs/linux-g++ -I. -I. -o
ptrsizetest.o ptrsizetest.cpp
ptrsizetest.cpp: In function ‘int main(int, char**)’:
ptrsizetest.cpp:18: error: ‘PointerSize’ is not a member of
‘QPointerSizeTest<4>’
make: *** [ptrsizetest.o] Error 1
Pointer size: 4

------- Comment #10 From Caleb Tennis 2007-12-06 11:23:14 0000 -------
That warning is fine, I believe.  It's just part of their system checks.  The
output probably should be supressed.

------- Comment #11 From Raúl Porcel 2007-12-06 12:18:24 0000 -------
Why not 4.3.3?

------- Comment #12 From Caleb Tennis 2007-12-06 12:45:23 0000 -------
If you want to stablize 4.3.3, then by all means go for it.  But it has a lot
more "bug fixes" than just this particular issue, and since it's been in
portage for only a day now I wasn't comfortable with requesting it for
stabilization.

------- Comment #13 From Christian Faulhammer 2007-12-06 16:04:21 0000 -------
x86 stable for 4.3.2-r1

------- Comment #14 From Raúl Porcel 2007-12-06 16:46:04 0000 -------
alpha/ia64/sparc stable for 4.3.2-r1

------- Comment #15 From Markus Rothe 2007-12-07 13:59:33 0000 -------
ppc64 stable (qt-4.3.2-r1)

------- Comment #16 From Tobias Scherbaum 2007-12-07 14:01:05 0000 -------
ppc stable

------- Comment #17 From Jeroen Roovers 2007-12-07 16:15:06 0000 -------
Stable for HPPA.

------- Comment #18 From Robert Buchholz 2007-12-10 13:16:53 0000 -------
amd64 stable, last arch.

This is ready for GLSA decision. I tend to vote yes.

------- Comment #19 From Robert Buchholz 2007-12-10 16:23:01 0000 -------
taco, please merge this into a new qt emul.

------- Comment #20 From Peter Weller 2007-12-11 00:28:43 0000 -------
Bumped the emul ebuild with new Qt, not yet stable though.

------- Comment #21 From Peter Weller 2007-12-11 22:01:13 0000 -------
app-emulation/emul-linux-x86-qtlibs-20071210 stable on amd64

------- Comment #22 From Robert Buchholz 2007-12-30 19:21:11 0000 -------
public via $URL

I vote NO on this bug.

------- Comment #23 From Pierre-Yves Rofes 2007-12-30 19:35:31 0000 -------
no too, closing.